From: Carsten K. <car...@us...> - 2003-12-11 20:52:10
|
On Thursday, December 11, 2003, at 03:15 pm, Arnaud Fontaine wrote: > Le lun 08/12/2003 =E0 10:57, Arnaud Fontaine a =E9crit : >> On a wiki running PhpWiki 1.3.6, I've a nice little bug : >> >> On some nodes (not all) the diff action adds some nbsp at the very >> beginning of the generated html page. >> >> I first thought it was a problem with the theme I write but it's the >> same with "default". >> >> More : the number of nbsp is not always the same depending of the =20 >> page. >> >> Have a look at =20 >> http://innovation.crao.net/index.php/Accueil?action=3Ddiff > > It seems no one is interested in this bug ... bug when I check the > source code of this page, it starts by : > > =  =20= > ; <?xml version=3D"1.0" encoding=3D"iso-8859-1"?> > > This makes the page not a valid xml page ... and breaks some browsers. > --=20 > Arnaud Fontaine > Jabber: sh...@ra... > ICQ: 3504789 Hi, I already found the bug in lib/diff.php but there are a few other =20 changes I would like make in there before I commit a proper repair to =20= CVS. Here's a patch as a temporary workaround: Index: diff.php =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D RCS file: /cvsroot/phpwiki/phpwiki/lib/diff.php,v retrieving revision 1.44 diff -U2 -r1.44 diff.php --- diff.php 17 Feb 2003 02:17:31 -0000 1.44 +++ diff.php 11 Dec 2003 20:46:39 -0000 @@ -49,5 +49,5 @@ continue; if ($word[0] =3D=3D "\n") { - $this->_group .=3D PrintXML(HTML::raw(' ')); + $this->_group .=3D " "; //new RawXml(' '); = //FIXME: =20 asXML(HTML::raw(' ')) ??? $this->_flushLine($tag); $word =3D substr($word, 1); |