problème sur les liens href dans les mails - solution proposee

[Didg]

Bonjour,

ayant constaté que certains liens href contenus dans des emails étaient mal interprétés, j'ai remarqué que la fonction activate_links($content) du fichier phpgwapi/inc/class.html.inc.php traitait ces liens alors qu'elle ne le devrait pas. Il se trouve que lorsqu'un mail contient a href="http://www.google.fr , la fonction traite la partie http, on se retrouve donc avec un code contenant  2 fois a href= puisque la fonction permet de rendre une URL sous forme d'un lien.
Il me semble que l'erreur vient de la ligne $NotAnchor = '(?<!"|href=|href\s=\s|href=\s|href\s=)'; Le caractère double quote n'est pas interprété comme il faut. Je n'ai pour ma part pas trouvé comme le protéger (si besoin il y a). Je propose par contre l'alternative suivante (3 lignes sont modifiées, elles sont commentées avant leur nouvelle version):
<code>
        static function activate_links($content)
        {
                if (!$content || strlen($content) < 20) return $content;        // performance

                // Exclude everything which is already a link
                //$NotAnchor = '(?<!"|href=|href\s=\s|href=\s|href\s=)';
                $NotAnchor = '(?<!href=|href\s=\s|href=\s|href\s=)';

                // spamsaver emailaddress
                $result = preg_replace('/'.$NotAnchor.'mailto:([a-z0-9._-]+)@([a-z0-9_-]+)\.([a-z0-9._-]+)/i',
                        '\\1 AT \\2 DOT \\3',
                        $content);

                //  First match things beginning with http:// (or other protocols)
                //$Protocol = '(http:\/\/|(ftp:\/\/|https:\/\/))';      // only http:// gets removed, other protocolls are shown
                $Protocol = '(http:\/\/|\shttp:\/\/|ftp:\/\/|https:\/\/)';      // only http:// gets removed, other protocolls are shown

                $Domain = '([\w-]+\.[\w-.]+)';
                $Subdir = '([\w\-\.,@?^=%&;:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
                $Expr = '/' . $NotAnchor . $Protocol . $Domain . $Subdir . '/i';

                $result = preg_replace( $Expr, "$2$3$4", $result );

                //  Now match things beginning with www.
                $NotHTTP = '(?<!:\/\/|" target=\"_blank\">)';   //      avoid running again on http://www links already handled above
                $Domain = 'www(\.[\w-.]+)';
                $Subdir = '([\w\-\.,@?^=%&:\/~\+#]*[\w\-\@?^=%&\/~\+#])?';
                $Expr = '/' . $NotAnchor . $NotHTTP . $Domain . $Subdir . '/i';

              //return preg_replace( $Expr, "$0", $result );
                return preg_replace( $Expr, " $0", $result );

        }
</code>

Je pense qu'il y a moyen de simplifier en travaillant sur les regex.

En espérant que cela puisse dépanner ceux qui avait le même soucis que moi.

----------------------------------------------------------------
Clermont Communauté - egroupware 1.6.002 - ~600 comptes utilisateurs