Changeset 1319

Timestamp:

08/13/09 11:34:34 (15 years ago)

Author:

amuller

Message:

Ticket #621 - Melhoria no desanexar marcando como lida/

Location:

trunk/expressoMail1_2

Files:

• inc/class.imap_functions.inc.php (modified) (4 diffs)
• js/main.js (modified) (3 diffs)

trunk/expressoMail1_2/inc/class.imap_functions.inc.php

@@ -315 +315 @@
                         $this->mbox = $this->open_mbox($params["folder"]);
                 $return["status"] = true;
-                $return["msg_num"] = $params["msg_num"];
                 $header = "";
                 
@@ -321 +320 @@
                 foreach($headertemp as $head) {//Se eu colocar todo o header do email dá pau no append, então procuro apenas o que interessa.
                         $head1 = explode(":",$head);
-                        if ( (strtoupper($head1[0]) == strtoupper("to")) || 
-                                        (strtoupper($head1[0]) == strtoupper("from")) || 
-                                        (strtoupper($head1[0]) == strtoupper("subject")) ||
-                                        (strtoupper($head1[0]) == strtoupper("date")) )
+                        if ( (strtoupper($head1[0]) == "TO") || 
+                                        (strtoupper($head1[0]) == "FROM") || 
+                                        (strtoupper($head1[0]) == "SUBJECT") ||
+                                        (strtoupper($head1[0]) == "DATE") )
                                 $header .= $head."\r\n";
                 }
@@ -333 +332 @@
                                                              para que o método decodeBody do expresso possa trazer tudo certinho*/
                 
-                $status = @imap_append($this->mbox,
+                $status = imap_append($this->mbox,
                                 "{".$this->imap_server.":".$this->imap_port.$this->imap_options."}".$params["folder"],
                                         $header.
@@ -341 +340 @@
                                                         $msg->encoding[$params["msg_num"]][0], $msg->charset[$params["msg_num"]][0]
                                                         )                                       
-                                        )); //Append do novo email, só com header e conteúdo sem anexos.
-                
-                if(!$status) {
+                                        ), "\\Seen"); //Append do novo email, só com header e conteúdo sem anexos.
+                
+                if(!$status)
+                {
                         $return["status"] = false;
                         $return["msg"] = lang("error appending mail on delete attachments");
                 }
-        
-                @imap_delete($this->mbox, imap_msgno($this->mbox, $params["msg_num"])); //Insere o novo e-mail sem anexos.
-                @imap_expunge($this->mbox); //Deleta o e-mail com anexos
+                else
+                {
+                        $status = imap_status($this->mbox, "{".$this->imap_server.":".$this->imap_port."}".$params['folder'], SA_UIDNEXT);
+                        $return['msg_no'] = $status->uidnext - 1; 
+                        imap_delete($this->mbox, imap_msgno($this->mbox, $params["msg_num"]));
+                        imap_expunge($this->mbox);
+                }
                 
                 return $return;

trunk/expressoMail1_2/js/main.js

@@ -165 +165 @@
 
         var call_back = function(data) {
-                                        if(!data.status) { //Em caso de erros, o status vem como false e o texto do erro vem em msg.
-                                                alert(data.msg);
-                                        }
-                                        else {
-                                                mail_msg = Element("tbody_box");
-                                                msg_to_delete = Element(data.msg_num);
-                                                if (msg_to_delete) {
-                                                        if ( (msg_to_delete.style.backgroundColor != '') && (preferences.use_shortcuts == '1') )
-                                                                select_msg('null', 'down');
-                                                        mail_msg.removeChild(msg_to_delete);                                                                                            
-                                                }//Remove o email com anexos da lista de emails.
-                                                write_msg(get_lang("Attachments removed"));//indica que os anexos foram removidos
-                                                refresh(); //dá um refresh na lista de emails, para que apareça o novo email sem anexos
-                                                delete_border(data.msg_num+'_r','false');//fecha a tela do email já removido.
-                                        }
-                                };
+                if(!data.status) {
+                        alert(data.msg);
+                }
+                else {
+                        msg_to_delete = Element(msg_num);
+                        change_tr_properties(msg_to_delete, data.msg_no);
+                        msg_to_delete.childNodes[1].innerHTML = "";
+                        write_msg(get_lang("Attachments removed"));
+                        delete_border(msg_num+'_r','false'); //close email tab
+                }
+        };
         if (confirm(get_lang("delete all attachments confirmation")))
-                cExecute ("$this.imap_functions.remove_attachments&folder="+folder+"&msg_num="+msg_num, call_back); //chamada em ajax para a função de remover anexos.
+                cExecute ("$this.imap_functions.remove_attachments&folder="
+                                +folder+"&msg_num="+msg_num, call_back);
 }
 function watch_changes_in_msg(border_id)
@@ -1590 +1586 @@
 }
 function change_tr_properties(tr_element, newUid, newSubject){
-        var td_who = document.getElementById('td_who_'+tr_element.id);
-        td_who.nextSibling.innerHTML = newSubject;
+        message_id=tr_element.id;
+        var td_who = document.getElementById('td_who_'+message_id);
+        if (typeof(newSubject) != 'undefined')
+                td_who.nextSibling.innerHTML = newSubject;
         tr_element.id = newUid;
 
@@ -1599 +1597 @@
         };
         for (var i=2; i < 10; i++){
-                tr_element.childNodes[i].id = tr_element.childNodes[i].id.replace(message_id,newUid);
+                if (typeof(tr_element.childNodes[i].id) != "undefined")
+                        tr_element.childNodes[i].id = tr_element.childNodes[i].id.replace(message_id,newUid);
                 tr_element.childNodes[i].onclick = openNewMessage;
         }