Page 1 of 1

XHMTL 1.0 / CSS Compatible Output

PostPosted: Mon Feb 12, 2007 1:00 am
by Spunkmeyer
Hey guys, I was just recently going through a coranto installation on my site http://www.advocatusdiaboli.ca, and wanted to add the ISAY addon to allow people to comment on the site. Unfortunately, the link to the comments uses the following code:

Code: Select all
<a href="/path to isay/iSay.cgi?Page=Comments&ID=<Field: newsid>">Comments (<?php include("/path to isay/iSay_GetCommentCount.cgi?ID=<Field: newsid>"); ?>)</a>


Now, this works great unless you're trying to validate your page as XHTML 1.0. "ID=" generates a cannot generate system identifier for general entity "ID" error.

I think if we're gonna do a new version of Coranto, all the XHTML/CSS output should be validated. There's no point in using plain HTML anymore. Output for tables should be divs, etc.

Oh, on a side note, anyone know how i would make that link valid? XHTML?

PostPosted: Mon Feb 12, 2007 1:34 am
by Dale Ray
Do you have a page on a site you can point us at?

I think you would need to modify iSay to use a different name for the parameter, ID is used in HTML tags.

PostPosted: Mon Feb 12, 2007 1:53 am
by Spunkmeyer
Well its not that I cant get it working, its just that that link produces that error, and I don't know enough about the code to go digging through it and modify it...

PostPosted: Mon Feb 12, 2007 9:50 am
by SrNupsen
It is the ampersand (&) character. To validate, you have to use &amp; in your URL instead of &.

PostPosted: Tue Feb 13, 2007 9:08 am
by Spunkmeyer
Ah cool, thanks man, I'll give it a shot.

Re: XHMTL 1.0 / CSS Compatible Output

PostPosted: Sat May 26, 2007 11:12 am
by Whatrevolution
I had to fix that, to embed iSay in my pages. It was mostly a matter of search & replace. The tables, could be rewritten to divs as you said, easily, since they are in iSay_Config.dat already. Error reports are an exception, as is the <html></html> etc. for iSay.cgi output.

http://coranto.org/forum/viewtopic.php? ... rt=15#2107