- Fix sig replacement (#1486538) + fix checking for separator existance in sig

program/js/app.js

@@ -2331,19 +2331,19 @@ function rcube_webmail()
       // remove the 'old' signature
       if (show_sig && this.env.identity && this.env.signatures && this.env.signatures[this.env.identity]) {
         sig = this.env.signatures[this.env.identity].is_html ? this.env.signatures[this.env.identity].plain_text : this.env.signatures[this.env.identity].text;
-        
-        if (sig.indexOf('-- ') != 0)
+
+        sig = sig.replace(/\r\n/, '\n');
+        if (!sig.match(/^--[ -]\n/)
           sig = sig_separator + '\n' + sig;
 
         p = this.env.sig_above ? message.indexOf(sig) : message.lastIndexOf(sig);
         if (p >= 0)
           message = message.substring(0, p) + message.substring(p+sig.length, message.length);
       }
-
       // add the new signature string
       if (show_sig && this.env.signatures && this.env.signatures[id]) {
         sig = this.env.signatures[id]['is_html'] ? this.env.signatures[id]['plain_text'] : this.env.signatures[id]['text'];
-        if (sig.indexOf('-- ') != 0)
+        if (!sig.match(/^--[ -]\n/)
           sig = sig_separator + '\n' + sig;
 
         if (this.env.sig_above) {
@@ -2412,12 +2412,12 @@ function rcube_webmail()
       if (this.env.signatures[id]) {
         if (this.env.signatures[id].is_html) {
           sig = this.env.signatures[id].text;
-          if (this.env.signatures[id].plain_text.indexOf('-- ') != 0)
+          if (!this.env.signatures[id].plain_text.match(/^--[ -]\n/)
             sig = sig_separator + '<br />' + sig;
         }
         else {
           sig = this.env.signatures[id].text;
-          if (sig.indexOf('-- ') != 0)
+          if (!sig.match(/^--[ -]\n/)
             sig = sig_separator + '\n' + sig;
           sig = '<pre>' + sig + '</pre>';
         }