Changeset 6641 for trunk/expressoMail1_2

Timestamp:

06/26/12 14:59:52 (12 years ago)

Author:

acoutinho

Message:

Ticket #2906 - Verificar mensagem de erro ao usuario durante sinalizacao de mensagens

Location:

trunk/expressoMail1_2/js

Files:

• draw_api.js (modified) (1 diff)
• search.js (modified) (1 diff)

trunk/expressoMail1_2/js/draw_api.js

@@ -1760 +1760 @@
                                         .html('<img src="../prototype/modules/mail/img/flagEditor.png">');
                                         
-                                        switch( fail )
-                                        {
-                                            case '#FollowupflagMessageIdError': {
-                                                    $('#td_message_followup_' + messageClickedId + ', ' + 
-                                                    'tr[role="'+messageClickedId+'_'+msg_folder+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");
-                                                    alert("Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém o atributo message-id."); break;
-                                            }
-                                            case '#FollowupflagParamsError': {
-                                                    $('#td_message_followup_' + messageClickedId + ', ' + 
-                                                    'tr[role="'+messageClickedId+'_'+msg_folder+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");
-                                                    alert("Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém todos os atributos necessários."); break;
-                                            }
-                                            default : {
-                                                    $('#td_message_followup_' + messageClickedId + ', ' + 
-                                                    'tr[role="'+messageClickedId+'_'+msg_folder+'"] #td_message_followup_search_' + messageClickedId).attr('title', get_lang('Follow up')).find(".flag-edited").css("background", headers_msgs.followupflagged.backgroundColor);
-                                                    updateCacheFollowupflag(messageClickedId, msg_folder, true);
-                                            }
-                                        }
+                                        
+                                        if(fail != 'success'){
+                                            $('#td_message_followup_' + messageClickedId + ', ' + 
+                                            'tr[role="'+messageClickedId+'_'+msg_folder+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");
+                                            MsgsCallbackFollowupflag[fail]();
+                                        }else{
+                                            $('#td_message_followup_' + messageClickedId + ', ' + 
+                                            'tr[role="'+messageClickedId+'_'+msg_folder+'"] #td_message_followup_search_' + messageClickedId).attr('title', get_lang('Follow up')).find(".flag-edited").css("background", headers_msgs.followupflagged.backgroundColor);
+                                            updateCacheFollowupflag(messageClickedId, msg_folder, true);
+                                        }                               
                                 });
                                 

trunk/expressoMail1_2/js/search.js

@@ -586 +586 @@
                                                         
                                                         if (fail) {
-                                                                if(current_folder == mailbox){
-                                                                        $('#td_message_followup_' + messageClickedId + ', ' + 
-                                                                        'tr[role="'+messageClickedId+'_'+mailbox+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");       
-                                                                }else{                                                          
-                                                                        $('tr[role="'+messageClickedId+'_'+mailbox+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");             
-                                                                }
-                                                                
-                                                                switch( fail )
-                                                                {
-                                                                    case '#FollowupflagMessageIdError': alert("Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém o atributo message-id."); break;
-                                                                    case '#FollowupflagParamsError': alert("Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém todos os atributos necessários."); break;
-                                                                }
-                                                                
-                                                                return false;
+                                                            
+                                                            var isCurrentFolder = current_folder == mailbox ? '#td_message_followup_' + messageClickedId + ', ' : '';   
+                                                            $(isCurrentFolder + 'tr[role="'+messageClickedId+'_'+mailbox+'"] #td_message_followup_search_' + messageClickedId).find(".flag-edited").css("background", "#CCCCCC");
+                                                            MsgsCallbackFollowupflag[fail]();
+                                                            return false;
                                                         }