Changeset 5476

Timestamp:

02/06/12 11:20:51 (12 years ago)

Author:

cristiano

Message:

Ticket #2480 - Adequações no parser de e-mail - Tratamento do HTML

Location:

trunk

Files:

• expressoMail1_2/inc/class.imap_functions.inc.php (modified) (2 diffs)
• expressoMail1_2/js/draw_api.js (modified) (3 diffs)
• library/CssToInlineStyles (added)
• library/CssToInlineStyles/css_to_inline_styles.php (added)

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

@@ -1827 +1827 @@
 
         function replace_special_characters($body) 
-        {
-            // Suspected TAGS!
-            // $tag_list = Array('blink','object','meta','html','link','frame','iframe','layer','ilayer','plaintext','script','style','img','applet','embed','head','frameset','xml','xmp');
-            // remove MS Office's proprietary tag
-            //$body = mb_ereg_replace('<!\-\-\[if [^!]* mso .*\]>.*<!\[endif\]\-\->', '', $body);
-            // Layout problem: Change html elements
-            // with absolute position to relate position, CASE INSENSITIVE.
+        {               
+            if(trim($body) === '') return;
+            
             $body = str_ireplace('POSITION: ABSOLUTE;','', $body);
+            $body = str_ireplace('<o:p>&nbsp;</o:p>','<br />', $body);//Qubra de linha do MSO
+            $body = preg_replace('/<(meta|base|link|html|\/html)[^>]*>/i', '', $body);
+            
+            require_once('../library/CssToInlineStyles/css_to_inline_styles.php');
+            $cssToInlineStyles = new CSSToInlineStyles($body);
+            $cssToInlineStyles->setUseInlineStylesBlock(true);
+            $cssToInlineStyles->setCleanup(TRUE);
+            $body = $cssToInlineStyles->convert(); //Converte as tag style em inline styles
 
             ///--------------------------------// 
@@ -1841 +1845 @@
                 'blink', 'object', 'frame', 'iframe',
                 'layer', 'ilayer', 'plaintext', 'script',
-                'applet', 'embed', 'frameset', 'xml', 'xmp'
+                'applet', 'embed', 'frameset', 'xml', 'xmp','style','head'
             );
 
             foreach ($tag_list as $index => $tag) 
                 $body = @mb_eregi_replace("<$tag\\b[^>]*>(.*?)</$tag>", '', $body);
-            
-            $body = preg_replace('/<(meta|base|link)[^>]*>/i', '', $body);
-
-            //try to wrap CSS code instead of remove STYLE tags
-            require_once('../library/csstidy/class.csstidy.php');
-            $css = new csstidy();
-            $css->set_cfg('preserve_css', false);
-
-            $regs_found = array();
-            $tags_found = preg_match_all("@<style[^>]*>.*?</style[^>]*>@si", $body, $regs_found);
-            $wrapper_class = 'ExpressoCssWrapper' . time();
-                        
-            foreach ($regs_found as $k => &$v) {
-                foreach ($v as $kk => $vv) {
-                    $n_start = strpos($vv, '>') + 1;
-                    $n_length = strrpos($vv, '<') - $n_start;
-                    $bf_innerHTML = substr($vv, $n_start, $n_length);
-                    $bf_innerHTML = mb_ereg_replace('<!--', '', $bf_innerHTML);
-                    $bf_innerHTML = mb_ereg_replace('-->', '', $bf_innerHTML);
-                    
-                    $css->parse($bf_innerHTML);
-
-                    $prefix = ".$wrapper_class ";
-                    if (isset($css->css[41]) && count($css->css[41] > 0))
-                        foreach ($css->css[41] as $key => $value) {
-                            //explode multiple selectors per block 
-                            $selectors = explode(',', $key);
-
-                            foreach ($selectors as $selector) {
-                                if (ereg('\*', $key)) {
-                                    //skip selecto '*' 
-                                    continue;
-                                }
-
-                                $selector = eregi_replace('[^#\.]*body.*', '', $selector);
-                                $css->css[41][$prefix . trim($selector)] = $value;
-                            }
-                            unset($css->css[41][$key]);
-                        }
-
-                    $body = str_replace($vv, '<style>' . $css->print->plain() . '</style>', $body);
-                }
+
+            // Malicious Code Remove
+            $dirtyCodePattern = "/(<([\w]+[\w0-9]*)(.*)on(mouse(move|over|down|up)|load|blur|change|error|click|dblclick|focus|key(down|up|press)|select)([\n\ ]*)=([\n\ ]*)[\"'][^>\"']*[\"']([^>]*)>)(.*)(<\/\\2>)?/misU";
+            preg_match_all($dirtyCodePattern, $body, $rest, PREG_PATTERN_ORDER);
+            foreach ($rest[0] as $i => $val) {
+                if (!(preg_match("/javascript:window\.open\(\"([^'\"]*)\/index\.php\?menuaction=calendar\.uicalendar\.set_action\&cal_id=([^;'\"]+);?['\"]/i", $rest[1][$i]) && strtoupper($rest[4][$i]) == "CLICK" )) //Calendar events
+                    $body = str_replace($rest[1][$i], "<" . $rest[2][$i] . $rest[3][$i] . $rest[7][$i] . ">", $body);
             }
-            
-        // Malicious Code Remove
-        $dirtyCodePattern = "/(<([\w]+[\w0-9]*)(.*)on(mouse(move|over|down|up)|load|blur|change|error|click|dblclick|focus|key(down|up|press)|select)([\n\ ]*)=([\n\ ]*)[\"'][^>\"']*[\"']([^>]*)>)(.*)(<\/\\2>)?/misU";
-        preg_match_all($dirtyCodePattern, $body, $rest, PREG_PATTERN_ORDER);
-        foreach ($rest[0] as $i => $val) {
-            if (!(preg_match("/javascript:window\.open\(\"([^'\"]*)\/index\.php\?menuaction=calendar\.uicalendar\.set_action\&cal_id=([^;'\"]+);?['\"]/i", $rest[1][$i]) && strtoupper($rest[4][$i]) == "CLICK" )) //Calendar events
-                $body = str_replace($rest[1][$i], "<" . $rest[2][$i] . $rest[3][$i] . $rest[7][$i] . ">", $body);
-        }
-
-        /*
-         * Remove deslocamento a esquerda colocado pelo Outlook.
-         * Este delocamento faz com que algumas palavras fiquem escondidas atras da barra lateral do expresso. 
-         */
-        $body = mb_ereg_replace("(<p[^>]*)(text-indent:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
-        $body = mb_ereg_replace("(<p[^>]*)(margin-right:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
-        $body = mb_ereg_replace("(<p[^>]*)(margin-left:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
-        //--------------------------------------------------------------------------------------------//        
-        //Remoção de tags <span></span> para correção de erro no firefox
-        //Comentado pois estes replaces geram erros no html da msg, não se pode garantir que o os </span></span> sejam realmente os fechamentos dos <span><span>.
-        //Caso realmente haja a nescessidade de remover estes spans deve ser repensado a forma de como faze-lo.
-        //              $body = mb_eregi_replace("<span><span>","",$body);
-        //              $body = mb_eregi_replace("</span></span>","",$body);
-        //Correção para compatibilização com Outlook, ao visualizar a mensagem
-        $body = mb_ereg_replace('<!--\[', '<!-- [', $body);
-        $body = mb_ereg_replace('&lt;!\[endif\]--&gt;', '<![endif]-->', $body);
-
-        return "<div class=\"$wrapper_class\"><span>" . $body . '</span></div>';
+
+            /*
+            * Remove deslocamento a esquerda colocado pelo Outlook.
+            * Este delocamento faz com que algumas palavras fiquem escondidas atras da barra lateral do expresso. 
+            */
+            $body = mb_ereg_replace("(<p[^>]*)(text-indent:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
+            $body = mb_ereg_replace("(<p[^>]*)(margin-right:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
+            $body = mb_ereg_replace("(<p[^>]*)(margin-left:[^>;]*-[^>;]*;)([^>]*>)", "\\1\\3", $body);
+            //--------------------------------------------------------------------------------------------//    
+            //Remoção de tags <span></span> para correção de erro no firefox
+            //Comentado pois estes replaces geram erros no html da msg, não se pode garantir que o os </span></span> sejam realmente os fechamentos dos <span><span>.
+            //Caso realmente haja a nescessidade de remover estes spans deve ser repensado a forma de como faze-lo.
+            //          $body = mb_eregi_replace("<span><span>","",$body);
+            //          $body = mb_eregi_replace("</span></span>","",$body);
+            //Correção para compatibilização com Outlook, ao visualizar a mensagem
+            $body = mb_ereg_replace('<!--\[', '<!-- [', $body);
+            $body = mb_ereg_replace('&lt;!\[endif\]--&gt;', '<![endif]-->', $body);
+            $body  = preg_replace("/<p[^\/>]*>([\s]?)*<\/p[^>]*>/", '', $body); //Remove paragrafos vazios (evita duplo espaçamento em emails do MSO)
+           
+            return  $body ;
     }
         

trunk/expressoMail1_2/js/draw_api.js

@@ -2933 +2933 @@
         _body.id = 'body_' + ID;
         _body.innerHTML = newBody;
-        //_body.style.fontSize = '16px';
+        _body.style.marginLeft = '5px';
 
         var _elements = _body.getElementsByTagName( '*' );
@@ -3758 +3758 @@
                         save_msg(ID);
                 };
-                $("#save_message_options_"+ID).button({ disabled: false });
+                $("#save_message_options_"+ID).button({disabled: false});
         };
 
@@ -3918 +3918 @@
         $(".rich-button").click(function(){
                 //TO-DO : Se o usuário clicar varias vezes no botão, a função de iniciar o ckeditor se perde e morre.
-                $(this).button({ disabled: true });
+                $(this).button({disabled: true});
                 if($(this).find("span:first-child").html() == get_lang("Rich text")){
                         $(this).find("span:first-child").html(get_lang("Simple Text"));