[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squirrelmail-plugins
Subject:    Re: [SM-PLUGINS] Newmail plugin bug + fix
From:       "Tomas Kuliavas" <tokul () users ! sourceforge ! net>
Date:       2004-06-23 13:46:26
Message-ID: 62000.213.190.44.96.1088010655.squirrel () 213 ! 190 ! 44 ! 96
[Download RAW message or body]

> Dear all,
>
> Don't know if this has been fixed yet or not, but here goes.
>
> SM1.4.2a introduces a bug into the generic newmail plugin - specifically
> the audio notification part.
>
> Briefly, the <embed> tag on the left side has a '\' sprinkled where it
> should not be in the resulting HTML. This is, of course, an invalid
> syntax and so the sound never plays. I think it was something like:
> <EMBED SRC=\"https://....
>
> The fix for this is to make a change to the Newmail setup.php. Line 214
> should become:
> echo "<EMBED SRC=\"".htmlspecialchars($newmail_media)."\" HIDDEN=TRUE
> AUTOSTART=TRUE>\n";
>
> p dont think (et al), if this hasn't already been fixed, can this change
> be committed (presuming my PHP coding isn't that bad!)?

http://cvs.sf.net/viewcvs.py/squirrelmail/squirrelmail/plugins/newmail/setup.php?r1=1.17.2.2&r2=1.17.2.3



-------------------------------------------------------
This SF.Net email sponsored by Black Hat Briefings & Training.
Attend Black Hat Briefings & Training, Las Vegas July 24-29 - 
digital self defense, top technical experts, no vendor pitches, 
unmatched networking opportunities. Visit www.blackhat.com
--
squirrelmail-plugins mailing list
List Address: squirrelmail-plugins@lists.sourceforge.net
List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-plugins
http://squirrelmail.org/cvs
[prev in list] [next in list] [prev in thread] [next in thread]