Changeset 6653 for branches/2.4/prototype

Timestamp:

06/27/12 11:04:30 (12 years ago)

Author:

eduardow

Message:

Ticket #2906 - Verificar mensagem de erro ao usuario durante sinalização de mensagens.

Location:

branches/2.4/prototype

Files:

• modules/mail/interceptors/Helpers.php (modified) (3 diffs)
• modules/mail/js/followupflag.js (modified) (1 diff)
• modules/mail/js/label.js (modified) (1 diff)
• services/ImapServiceAdapter.php (modified) (1 diff)

branches/2.4/prototype/modules/mail/interceptors/Helpers.php

@@ -205 +205 @@
                 
                 if(empty($params['messageId'])) 
-                { 
-                        throw new Exception('#FollowupflagMessageIdError');
-                }else{ 
-                
-                        $params = Controller::create($imap_uri, $params);
-                }
+                    throw new Exception('#FollowupflagMessageIdError'); 
+                else  
+                    $params = Controller::create($imap_uri, $params);
+
                 if (!$params)
-                        throw new Exception('#FollowupflagParamsError');
+                    throw new Exception('#FollowupflagLimitError');
         }
         
@@ -221 +219 @@
                  * deste interceptor.
                  */
+
+
+
+
                 if ($original['URI']['service'] == 'PostgreSQL') return $result;
                 
@@ -310 +312 @@
                  * Faz a consistência do banco com o imap
                  */
-                 /*
+                /*
                 if ($result) {
                         $idList = array();

branches/2.4/prototype/modules/mail/js/followupflag.js

@@ +1 @@
+MsgsCallbackFollowupflag = { 
+                 
+    '#FollowupflagMessageIdError': function(){ 
+        alert('Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém o atributo message-id.'); 
+    }, 
+    '#FollowupflagLimitError': function(){ 
+        alert('Não foi possível sinalizar esta mensagem. \nDetalhes do erro: limite de flags atingido para esta pasta.'); 
+    }, 
+    '#FollowupflagParamsError': function(){ 
+        alert('Não foi possível sinalizar esta mensagem. \nDetalhes do erro: mensagem não contém todos os atributos necessários.'); 
+    } 
+     
+} 
+
 function updateCacheFollowupflag(msgNumber, msgFolder, op){
         if(op){

branches/2.4/prototype/modules/mail/js/label.js

@@ -127 +127 @@
                                 var hasLabel = true;
                                 $.each(data, function(index, value) {
-                                        if(typeof value == 'object'){
-                                                hasLabel = false;
-                                        }
-                                        else {
-                                                hasLabel = value;
-                                        }
+                                
+                                hasLabel = typeof value == 'object' ? false : value;
+
                                 });
                                         

branches/2.4/prototype/services/ImapServiceAdapter.php

@@ -538 +538 @@
                                                         
 
-                                        return ($s) ? $data : array();
+                                        /* 
+                                         * TODO 
+                                         * Verificar erro ao tentar setar uma flag com o limite de flags atingido 
+                                         * onde o status retornado pelo imap_setflag_full é true mesmo não sendo possível 
+                                         * a inserção da flag. 
+                                         */ 
+ 
+                                        return (($s) && (imap_last_error() != 'Too many user flags in mailbox')) ? $data : array();
 
                                 } else if (isset($data['messageId'])) {