v6.2.3 Email Signature not loading

[jvddevine]

I just want to formally thank markv for the extra footwork in pressing this issue and I want to echo your comment on the "Sugar 6.4.0 Beta 6 is now available to test" thread, wherein you state "STOP adding features and START solving bugs". This email signature is only part of the problems I have begun to endure with the email client in upgraded versions. For example, using a commercial tool (Outlander) to archive my emails from Outlook I get a pile of poo stored in the contact record full of formatting tags and other useless junk. Arrrghhh!

I don't know why they changed it so dramatically, because the upgrade really made it troublesome to deal with and as I said early in this thread - I essentially had no troubles with the email client until this last upgrade - or was the one before last ? (ha ha ha)

[beschaeftigt.de]

I found the solution for this bug.

Just go to include/javascript/sugar_grp_quickcomp.js in funktion setSignature (line 3927) to look like this:

if(idx){ → if(idx!==null || idx!== undefined) {

then everything is fine :-) Worked for me in 6.2.4

[feijoa]

Sorry is this an insert a line into the code or revising a line a line?
I am using Version 6.4.0 and I'm not which line this pertains to. Could you possible paste the code around it?

Thanks

[irrbert]

Hi everyone, i have the same problem, also editing of email draft shows no text or signature.
The JS Error is coming :

ReferenceError: isReplyForward is not defined [http://XXX/include/javascript/sugar_...s.js?c=1:5171]

Hope this will fixed soon.

Best redards, Robert.

[beschaeftigt.de]

Sure :-)

 /**     * Writes out the signature in the email editor     */    setSignature : function(idx) {        if (!tinyMCE)            return false;        var hide = document.getElementById('setEditor' + idx).checked;        SE.composeLayout.renderTinyMCEToolBar(idx,hide);        //wait for signatures to load before trying to set them        if (!SE.composeLayout.signatures) {            setTimeout("SE.composeLayout.setSignature(" + idx + ");", 1000);            return;        }
        if(idx!==null || idx!== undefined) { //BUGFIX for Signature Problem if(idx){            var sel = document.getElementById('signatures' + idx);        } else {            var sel = document.getElementById('signature_id');            idx = SE.tinyInstances.currentHtmleditor;        }
        //Ensure that the tinyMCE html has been rendered. 
  

Sorry is this an insert a line into the code or revising a line a line?
I am using Version 6.4.0 and I'm not which line this pertains to. Could you possible paste the code around it?

Thanks