XML output validerer ikke korrekt

Maa vi se lidt mere af XML'en ?

Gerne en komplet XML snippet som illusterer problemet.

Komplet xml.

<?xml version="1.0" encoding="UTF-8"?><rss version="2.0"><channel><title>mctraef.dk</title><link>http://www.mctraef.dk</link><description>MCtræf i hele Danmark</description><item><title><![CDATA[MC træf, Færch Torvet 18-06-2009 17:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=15"></link><description></description></item><item><title><![CDATA[Højeruplund Traktørsted 21-06-2009 10:30]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=23"></link><description></description></item><item><title><![CDATA[Niverød MC 21-06-2009 11:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=32"></link><description></description></item><item><title><![CDATA[Torvet i Hillerød 21-06-2009 11:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=33"></link><description></description></item><item><title><![CDATA[Fælleskørsel fra BikX 21-06-2009 12:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=34"></link><description></description></item></channel></rss>

Selvom det nok er ret uoverskueligt :-)

Ekspertens syntaksfavning hjalp mig. Det er et " for meget efter 15.

Og dog. Stadig samme fejl. XML se nu således ud:

<?xml version="1.0" encoding="UTF-8"?><rss version="2.0"><channel><title>mctraef.dk</title><link>http://www.mctraef.dk</link><description>MCtræf i hele Danmark</description><item><title><![CDATA[MC træf, Færch Torvet 18-06-2009 17:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=15</link><description></description></item><item><title><![CDATA[Højeruplund Traktørsted 21-06-2009 10:30]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=23</link><description></description></item><item><title><![CDATA[Niverød MC 21-06-2009 11:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=32</link><description></description></item><item><title><![CDATA[Torvet i Hillerød 21-06-2009 11:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=33</link><description></description></item><item><title><![CDATA[Fælleskørsel fra BikX 21-06-2009 12:00]]></title><link>http://www.mctraef.dk/index.php?inc=traef&id=34</link><description></description></item></channel></rss>

Dit XML er ikke synderligt pænt... Men det er jo ikke det det handler om. Du har to fejl, lige nu der kunne give fejl:

Tomme elementer skal altid afslutte sig selv. Dit description element skal skrives som <description /> når du ikke har noget indhold i.

& tegn skal skrives som &amp;
Derfor vil dine link tags give fejl, som de ser ud nu.
Du skal skrive:
http://www.mctraef.dk/index.php&amp;inc=traef&amp;id=15

Ja se, det lettede :-)

Hvad skal jeg gøre for at mit rss blive pænt?
Jeg hjar fulgt w3schools' guide

Jeg forhastede mig med linket, jeg fik også oversat ? til et &, hvilket ikke var meningen:
http://www.mctraef.dk/index.php?inc=traef&amp;id=15

Hvordan laver du XML'en?

Via noget php-kode med udtræk fra en MySQL.

...
ja, men hvordan?
Bruger du DOM, SimpleXML eller skriver du skidtet manuelt?

At escape alle links virkede for mig:

<?xml version="1.0" encoding="UTF-8"?>
<rss version="2.0">
    <channel>
        <title>mctraef.dk</title>
        <link>http://www.mctraef.dk</link>
        <description>MCtræf i hele Danmark</description>
        <item>
            <title><![CDATA[MC træf, Færch Torvet 18-06-2009 17:00]]></title>
            <link><![CDATA[http://www.mctraef.dk/index.php?inc=traef&id=15]]></link>
            <description></description>
        </item>
        <item>
            <title><![CDATA[Højeruplund Traktørsted 21-06-2009 10:30]]></title>
            <link><![CDATA[http://www.mctraef.dk/index.php?inc=traef&id=23]]></link>
            <description></description>
        </item>
        <item>
            <title><![CDATA[Niverød MC 21-06-2009 11:00]]></title>
            <link><![CDATA[http://www.mctraef.dk/index.php?inc=traef&id=32]]></link>
            <description></description>
        </item>
        <item>
            <title><![CDATA[Torvet i Hillerød 21-06-2009 11:00]]></title>
            <link><![CDATA[http://www.mctraef.dk/index.php?inc=traef&id=33]]></link>
            <description></description>
        </item>
        <item>
            <title><![CDATA[Fælleskørsel fra BikX 21-06-2009 12:00]]></title>
            <link><![CDATA[http://www.mctraef.dk/index.php?inc=traef&id=34]]></link>
            <description></description>
        </item>
    </channel>
</rss>

Ovenstående kode validerede fint.

Jeg skriver det manuelt.

print '<?xml version="1.0" encoding="UTF-8"?>';
print '<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">';
print '<channel>';
print '<atom:link href="http://www.mctraef.dk/traeffeed.php" rel="self" type="application/rss+xml" />';
print '<title>mctraef.dk</title>';
print '<link>http://www.mctraef.dk</link>';
print '<description>'.utf8_encode("MCtræf i hele Danmark").'</description>';
$hent_naeste = "select *, traef.id AS t_id from traef_event, traef where traef_event.traef_id = traef.id AND starttid > '".date("U")."'order by starttid limit 10";
$koer_hent_naeste = mysql_query($hent_naeste) or die(mysql_error());
while($row=mysql_fetch_array($koer_hent_naeste)){
print '<item>';
print '<title><![CDATA['.utf8_encode($row['navn'].' '.date("d-m-Y H:i", $row["starttid"])).']]></title>';
print '<description><![CDATA['.utf8_encode($row['beskrivelse']).']]></description>';
print '<guid>http://www.mctraef.dk/index.php?inc=traef&amp;id='.$row["t_id"].'</guid>';
print '</item>';
}
print '</channel>';
print '</rss>';

Har rettet lidt til som du kan se. Endeligt resultat: http://www.mctraef.dk/traeffeed.php

Problematikken i det der er at du reelt set får alting til at stå på en enkelt linie. Det vil sige at når du vil fejlsøge i din XML vil du altid få at vide at fejlen ligger i linie 1. Ikke særligt behjælpeligt i en linie på 3000 tegn.

Hvis du får det sat pænt op, vil du også hurtigt kunne finde fejl i din XML og du vil også have et bedre overblik.

Istedet for at printe dit output ud bør du lægge det i en variabel og derefter kan du bruge DOM for at 'forskønne det':

---------------------------------
$rss = '<?xml version="1.0" encoding="UTF-8"?>';
$rss .= '<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">';
$rss .= '<channel>';
$rss .= '<atom:link href="http://www.mctraef.dk/traeffeed.php" rel="self" type="application/rss+xml" />';
$rss .= '<title>mctraef.dk</title>';
$rss .= '<link>http://www.mctraef.dk</link>';
$rss .= '<description>'.utf8_encode("MCtræf i hele Danmark").'</description>';
$hent_naeste = "select *, traef.id AS t_id from traef_event, traef where traef_event.traef_id = traef.id AND starttid > '".date("U")."'order by starttid limit 10";
$koer_hent_naeste = mysql_query($hent_naeste) or die(mysql_error());
while($row=mysql_fetch_array($koer_hent_naeste))
{
    $rss .= '<item>';
    $rss .= '<title><![CDATA['.utf8_encode($row['navn'].' '.date("d-m-Y H:i", $row["starttid"])).']]></title>';
    $rss .= '<description><![CDATA['.utf8_encode($row['beskrivelse']).']]></description>';
    $rss .= '<guid>http://www.mctraef.dk/index.php?inc=traef&amp;id='.$row["t_id"].'</guid>';
    $rss .= '</item>';
}
$rss .= '</channel>';
$rss .= '</rss>';

$doc = new DOMDocument('1.0');
$doc->preserveWhiteSpace = false;
$doc->loadXML($rss);
$doc->formatOutput = true;
$data = $doc->saveXML();

echo $data;
---------------------------------

Prøv det...

Ja, det blev jo unægteligt noget kønnere :-)

Hvad mener validatoren med dette: http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fwww.mctraef.dk%2Ftraeffeed.php

guid element?

Det betyder at du burde have et element der hedder guid der unikt identificerer en item.

Typisk vil <link> pege på den specifikke nyhed og derfor kan du i princippet lade <guid> indeholde det samme.

Men nogle vælger jo at liste de fulde nyheder på en og samme side og linker derfor til den samme side i alle items->link elementerne. Så kan man med fordel bruge <guid> elementet til at adskille nyhederne i feedet.

Godt så. Så er det vist tid til svar og point.

Jamen, svar fra mig, så :)