pagination not valid xhtml

Hi Folks
since two days I try to find where I could change & in the url call to & which makes the validation fail. the code generated in pagination looks like this

Thanks in advance

EDIT: Added code tags

Hi,

try by editing this row in maxigallery class (line 740)




:-)

Hi heliotrope

this seems to be ok, it looks like this already with &

return $modx->makeUrl($docid, $docalias, join(’&’,$urlstring));

thanks anyway.

here ist the site if you wanna have a look at the code

http://server8.ispace.ch/~admin21/index.php?id=7

for the time being I solved it with this plugin

$js = ’&page’;
$modx->documentOutput = str_replace($js,’&page’,&$modx->documentOutput);

but I don’t think this is really the best solution. So if anyone has a better solution to solve the probem in within MaxiGallery, I would really appreciate it.

It’s a problem in $modx->makeUrl function, it uses & chars instead of &amp.

so I guess I will have to use my solution until the next release ist out right?

Yes, although I think that it’s not fixed in next release either..

EDIT: Try to change in maxigallery.class.php line 740 from:

return $modx->makeUrl($docid, $docalias, join(’&’,$urlstring));

to:

return $modx->makeUrl($docid, $docalias, join(’&’,$urlstring));

in the download I got from the repository the maxigallery.class.php
is already set to
return $modx->makeUrl($docid, $docalias, join(’&’,$urlstring));
in line 740.

well so I will keep my solution as long as I don’t get any problems with the site

thanks anyway

Quote from: bbeer at Mar 11, 2007, 01:31 AM

in the download I got from the repository the maxigallery.class.php
is already set to
return $modx->makeUrl($docid, $docalias, join(’&’,$urlstring));
in line 740.

Umm.. no it’s not.

I checked again
in my installation it is set to & I guess I did it after the typ of heliotrope before. However there was no change the & was not displayed as &