Changeset 6641 for trunk/prototype

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/prototype

Files:

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

trunk/prototype/modules/mail/interceptors/Helpers.php

@@ -206 +206 @@
                 
                 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');
         }
         
@@ -222 +220 @@
                  * deste interceptor.
                  */
+                
+           
+            
+            
                 if ($original['URI']['service'] == 'PostgreSQL') return $result;
                 
@@ -267 +269 @@
         //followupflagged:PostgreSQL.after.find
         public function doImapFind( &$uri , &$result , &$criteria , $original ){
-
-                $imap_uri = $uri;
+            
+                $imap_uri = $uri;
                 $imap_uri['service'] = 'Imap';
                 $imap_criteria = $original['criteria'];
@@ -311 +313 @@
                  * Faz a consistência do banco com o imap
                  */
-                 /*
+                /*
                 if ($result) {
                         $idList = array();

trunk/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){

trunk/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;
+                                        
                                 });
                                         

trunk/prototype/services/ImapServiceAdapter.php

@@ -538 +538 @@
                                         
                                         $data['messageId'] = $headers[0]->message_id;
-                                                        
-
-                                        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'])) {