[gelöst] 'null' wird an Mails angehangen

[Harald WSV]

Hallo,

ich habe die Version 2.8.6 installiert, und dabei wird jetzt an jede Mail, die ich über JVerein versende, ein 'null' angehangen. Ist gerade unangenehm aufgefallen, die Mail endete mit 'Vielen Dank für deinen Einsatz.' Daraus wurde dann 'Vielen Dank für deinen Einsatz.null'

schöne Grüße
Harald

[heiner]

Hallo Harald,

verwendest du eine Variable an der Stelle, an der 'null' ausgegeben wird? Evtl. ist der Wert dieser Variablen (z. B. Lesefeld) null.

Heiner

[Harald WSV]

Hallo Heiner,

nein, es ist keine Variable an der Stelle enthalten. Das Problem tritt immer am Ende der Mail auf.

schöne Grüße
Harald

[josa49]

Hallo Harald,

prüfe mal in den Mail-Einstellungen deine Signatur.

Joachim

[Harald WSV]

Hallo Joachim,

habe ich überprüft. Es ist keine Signatur definiert, also auch keine Variable enthalten. Das Problem tritt trotzdem auf.

Auf deinen Hinweis hin habe ich aber testweise eine Signatur eingegeben, damit ist das Problem verschwunden. Auch nachdem ich die Signatur wieder gelöscht habe, bleibt das Problem verschwunden. Offenbar war die Signatur schlicht nicht initialisiert? Auf jeden Fall Danke für den Tip.

schöne Grüße
Harald

[thooge]

Hallo,

ich habe mir den Code angesehen. Es ist tatsächlich so, daß uninitialisierte Maileinstellungen
einen null-Text anhängen.
Ich habe einen Fix angehängt. Zusätzlich wird die Signatur entsprechend RFC 3676 (http://tools.ietf.org/html/rfc3676#section-4.3) erstellt.

Code: Alles auswählen

### Eclipse Workspace Patch 1.0
#P jverein
diff --git src/de/jost_net/JVerein/gui/control/EinstellungControl.java src/de/jost_net/JVerein/gui/control/EinstellungControl.java
index c60c831..fd4c9f3 100644
--- src/de/jost_net/JVerein/gui/control/EinstellungControl.java
+++ src/de/jost_net/JVerein/gui/control/EinstellungControl.java
@@ -1140,7 +1140,7 @@
       return mailsignatur;
     }
     mailsignatur = new TextAreaInput(Einstellungen.getEinstellung()
-        .getMailSignatur(), 1000);
+        .getMailSignatur(false), 1000);
     mailsignatur.setHeight(50);
     return mailsignatur;
   }
diff --git src/de/jost_net/JVerein/gui/control/MailControl.java src/de/jost_net/JVerein/gui/control/MailControl.java
index 091b1ea..d3ba25b 100644
--- src/de/jost_net/JVerein/gui/control/MailControl.java
+++ src/de/jost_net/JVerein/gui/control/MailControl.java
@@ -398,7 +398,7 @@
   {
     final String betr = getBetreffString();
     final String txt = getTxtString()
-        + Einstellungen.getEinstellung().getMailSignatur();
+        + Einstellungen.getEinstellung().getMailSignatur(true);
     BackgroundTask t = new BackgroundTask()
     {
 
diff --git src/de/jost_net/JVerein/rmi/Einstellung.java src/de/jost_net/JVerein/rmi/Einstellung.java
index 19d38a8..b2af838 100644
--- src/de/jost_net/JVerein/rmi/Einstellung.java
+++ src/de/jost_net/JVerein/rmi/Einstellung.java
@@ -345,7 +345,7 @@
 
   public void setImapSentFolder(String imap_sent_folder) throws RemoteException;
 
-  public String getMailSignatur() throws RemoteException;
+  public String getMailSignatur(Boolean separator) throws RemoteException;
 
   public void setMailSignatur(String mailsignatur) throws RemoteException;
 
diff --git src/de/jost_net/JVerein/server/EinstellungImpl.java src/de/jost_net/JVerein/server/EinstellungImpl.java
index d4cdcab..c7862e9 100644
--- src/de/jost_net/JVerein/server/EinstellungImpl.java
+++ src/de/jost_net/JVerein/server/EinstellungImpl.java
@@ -1178,9 +1178,19 @@
   }
 
   @Override
-  public String getMailSignatur() throws RemoteException
+  public String getMailSignatur(Boolean separator) throws RemoteException
   {
-    return (String) getAttribute("mailsignatur");
+    String signatur = (String) getAttribute("mailsignatur");
+    if (signatur == null)
+    {
+      return "";
+    }
+    // Siehe RFC 3676, 4.3. Usenet Signature Convention
+    if (separator && ! signatur.startsWith("-- \n"))
+    {
+      return "-- \n" + signatur;
+    }
+    return signatur;
   }
 
   @Override

Gruß,
Thomas

[heiner]

Ich habe den Patch übernommen.

Heiner