nicht valide Templates

Enthält archivierte Beiträge von Hotschi's Board zum Download MOD.
Posten ist hier nicht möglich, bitte das Supportforum dafür verwenden!
Dungeonwatcher

nicht valide Templates

Beitrag von Dungeonwatcher »

Hi! 8)

Da ich ein dunkles Thema (FiBlack) in meinem Forum nutze und somit auch die Eingabefelder weiße Schrift auf dunklem Hintergrund haben, fiel mir sofort auf, das die Templates einige kleine Unschönheiten aufweisen. Will sagen, ein Teil der Eingabefelder war korrekt weiß auf dunkel aber ein Teil leider nicht. Also bin ich beigegangen und habe diese Unschönheit korrigiert und gleichmal den Validator www.validome.org bemüht. :roll:

Der meckerte mich gleich wieder voll. Mittlerweile habe ich fast alle Admin Templates entsprechend korrigiert und werde sie hier dann auch hochladen. Allerdings wiedersetzt sich "dl_ext_blacklist_body.tpl" meinen Bemühungen. Grund hierfür ist dieser Abschnitt:

Code: Alles auswählen

  <!-- BEGIN extention_row -->
  <tr>
    <td width="50%" class="{extention_row.ROW_CLASS}" align="center"><span class="nav">{extention_row.EXTENTION}</span></td>
    <td width="50%" class="{extention_row.ROW_CLASS}" align="center"><input type="checkbox" name="extention[]" value="{extention_row.EXTENTION}" /></td>
  </tr>
  <!-- END extention_row -->
Der fertige html-Quelltext sieht nämlich so aus:

Code: Alles auswählen

  <tr>
    <td width="50%" class="" align="center"><span class="nav">asp</span></td>
    <td width="50%" class="" align="center"><input type="checkbox" name="extention[]" value="asp" /></td>
  </tr>
Es wird extention_row.ROW_CLASS nicht gefunden. :roll:

Wer weiss hier die Lösung?

Bye/2

oxpus

Beitrag von oxpus »

Aber nur bei jedem 2.?

Liegt hieran:

Code: Alles auswählen

		$row_class = (($i % 2)) ? $theme['td_class1'] : $theme['th_class2'];
Diese Zeile ist in der dl_mod/admin/dl_admin_ext_blacklist.php zu finden.
In dieser Zeile muss th_class2 gegen td_class2 getauscht werden.

Ich korrigiere den MOD umgehend...

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:Aber nur bei jedem 2.?
Jupp. Super, jetzt leuchtet Validome.org schön in grün. 8)
Ich korrigiere den MOD umgehend...
Wenn du schon dabei bist, dann korrigiere bitte auch mal die dl_modcp.php und ersetze

Code: Alles auswählen

$file_traffic_range_kb = '"';
mit

Code: Alles auswählen

$file_traffic_range_kb = '';
Da hat sich nämlich ein " reingeschlichen.

Gehe ich recht in der Annahme, das die Templates gem. XHTML 1.0 geschrieben sind?

oxpus

Beitrag von oxpus »

Nein, nur blankes HTML.
Selbst das phpBB wäre nicht XHTML-fähig, daher erfolgt auch keine strikte Anpassung an diesen Standard.

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:Nein, nur blankes HTML.
Das dann aber ziemlich inkonsequent. :wink:

oxpus

Beitrag von oxpus »

Ich sagte schon: Nach den Vorgaben der phpbb-Group.
Und wenn Du nicht immer nur Behauptungen, sondern endlich auch mal konkrete "Fehler" posten würdest, könnte ich der Diskussion gerne weiter beiwohnen!

Dungeonwatcher

Beitrag von Dungeonwatcher »

Nun gut.:

fast immer <br /> statt <br> bzw. <hr /> statt <hr>

die view_dl_cat_body.tpl

Code: Alles auswählen

<td class="{downloads.ROW_CLASS}" {downloads.ROW_SPAN}><span class="genmed">{downloads.CAT_DESC}</td>
hier fehlt das abschliessende <span>

- dl_footer.tpl

Code: Alles auswählen

</tr>
<table>
<br />
hier gehört ein </table> hin

Code: Alles auswählen

<td align="center"><span class="gensmall">{total_stat.TOTAL_STAT}</td>
hier fehlt das abschliessende <span>

usw. Das betrifft sämtliche Templates. Jage einfach mal die DL Seiten durch einen Validator deiner Wahl. :wink:

oxpus

Beitrag von oxpus »

fast immer <br /> statt <br> bzw. <hr /> statt <hr>
Falsch, das ist dann XML-kompatibel und somit korrekt ;)
Ich schrieb ja schon, daß ich mich an die phpbb.com-Vorgaben halte :D

view_dl_cat_body.tpl ist nun gefixt.

dl_footer.tpl ebenso gefixt.
Das betrifft sämtliche Templates. Jage einfach mal die DL Seiten durch einen Validator deiner Wahl.
Wenn ich mal viel Zeit dazu hätte...

Aber zum Thema fehlende </span>:
Nach den HTML-Regeln der w3c ist ein fehlender Tag zwar ein Fehler, er ist aber dann nicht mehr kritisch, wenn eine Tabellen-Zelle geschlossen wird.
Daher kein dragisches Ding, nur springen immer wieder alle Validatoren "wie wild" darauf an und schreiben "kritisch", was es einfach nicht ist.
Aber okay, ich korrigiere es und damit ist es wieder korrekter...

Und btw:
Wenn schon ein Validator, dann doch bitte das Original vom w3c-Konsortium, da hier eben die Regeln zentral aufgestellt und nachgehalten werden.
Alle anderen Validatoren halte ich nicht für korrekt und zweifele deren Arbeitsweisen an...

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:
fast immer <br /> statt <br> bzw. <hr /> statt <hr>
Falsch, das ist dann XML-kompatibel und somit korrekt ;)
Ich schrieb ja schon, daß ich mich an die phpbb.com-Vorgaben halte :D
Du sprachst eben noch nur blankes HTML, und darin ist es nicht korrekt.
view_dl_cat_body.tpl ist nun gefixt.

dl_footer.tpl ebenso gefixt.
Super.
Das betrifft sämtliche Templates. Jage einfach mal die DL Seiten durch einen Validator deiner Wahl.
Wenn ich mal viel Zeit dazu hätte...
Du solltest sie dir wirklich mal nehmen. :wink:
Aber zum Thema fehlende </span>:
Nach den HTML-Regeln der w3c ist ein fehlender Tag zwar ein Fehler, er ist aber dann nicht mehr kritisch, wenn eine Tabellen-Zelle geschlossen wird.
Daher kein dragisches Ding, nur springen immer wieder alle Validatoren "wie wild" darauf an und schreiben "kritisch", was es einfach nicht ist.
Du hast zwar recht, aber mich stört das ungemein. :roll:
Aber okay, ich korrigiere es und damit ist es wieder korrekter...
SUUUPER.

Wenn ich den letzten noch vorhanden Fehler bei mir korrigiert habe, dann schicke ich dir die Templates per PM. Jetzt geht's in die Federn um morgen früh um 04:00 Uhr wieder los zu legen. :?

oxpus

Beitrag von oxpus »

Öhm, das w3c-Konsortium schlägt vor, bei HTML-Seiten alle Tags analog dem XML-Bestimmungen mit einem Leer-Element (also wird dann zu />) zu schliessen, um hier beide Standards zu vereinen.
Also sofern ein Browser XML Seiten verarbeitet, ist das sogar vorgeschrieben.
Nur fallen leider alle Validatoren darauf rein, mit dem w3c-Validator gibts hier aber komischerweise selten Probleme ;)

Und "blankes" HTML gibts auch in vielen Versionen ;)

Ja klar, wenn Du die Templates durch hast, aktualisiere ich die gerne im MOD.
Muss dann eh eine neue Version machen, da auch noch andere Dateien geändert wurden (bislang nur stille Updates gemacht).

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:Öhm, das w3c-Konsortium schlägt vor, bei HTML-Seiten alle Tags analog dem XML-Bestimmungen mit einem Leer-Element (also wird dann zu />) zu schliessen, um hier beide Standards zu vereinen.
Du sagst es: "schlägt vor"!
Also sofern ein Browser XML Seiten verarbeitet, ist das sogar vorgeschrieben.
Woher soll ich wissen welche Browser zum Einsatz kommen?
Nur fallen leider alle Validatoren darauf rein, mit dem w3c-Validator gibts hier aber komischerweise selten Probleme ;)
Nun aber hoppla, wenn selbst der hauseigene Validator rumblubbert:

[...]
- Line 293 column 6: document type does not allow element "TABLE" here.
- Line 294 column 4: document type does not allow element "BR" here; assuming missing "CAPTION" start-tag.
- Line 329 column 4: document type does not allow element "HR" here; missing one of "APPLET", "OBJECT", "MAP", "IFRAME", "BUTTON" start-tag.
- Line 338 column 180: end tag for "SPAN" omitted, but its declaration does not permit this.
[...]
Und "blankes" HTML gibts auch in vielen Versionen ;)
Stimmt, nur nicht als XHTML. :wink:
Ja klar, wenn Du die Templates durch hast, aktualisiere ich die gerne im MOD.
Momentan hakelt es an der dl_footer.tpl:

- ...end tag for element "form" which is not open.

Da steige ich noch niocht so ganz durch. :?

oxpus

Beitrag von oxpus »

Wir können jetzt auch tagelang so weiter um die Standards reden, ich werde mich aber weiterhin an die Vorgaben von phpbb.com halten und HTML modden, welches XML kompatibel ist (abgesehen von, naja, "gelegentlichen Flüchtigskeitsfehlern").
Und da subSilver mit der Seiteneinleitung

Code: Alles auswählen

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
daher kommt, die mit dem phpBB veröffentlicht wird und dazu aber Ale Tag-Elemente XML-like (also "/>") geschlossen werden sollen, halte ich mich dran.

Und alles andere korrigiere ich gerne, wenn ich es genannt bekomme, bzw. selber entdeckt.
Lass uns also damit die Diskussion über Standards beenden und auf die echten Fehler beschränken.
Alles andere ist nur eher Spam und bringt nichts...

Dann noch schnell zur dl_footer.tpl:
Ist schon seltsam, da kein Form enthalten ist, was direkt im Template zu prüfen wäre :?

Und ein letztes:
Stimmt, nur nicht als XHTML.
Laut w3c gibt es 5 XHTML-Standards und eins dazu erweitert mit MathML.
Soviel zum Thema "Vielfalt" ;)

Dungeonwatcher

Beitrag von Dungeonwatcher »

Ein verschlafenes Moin Moin! 8)
oxpus hat geschrieben:Dann noch schnell zur dl_footer.tpl:
Ist schon seltsam, da kein Form enthalten ist, was direkt im Template zu prüfen wäre :?
Ich habe da auch etwas Mist erzählt. Es betrifft dl_footer.php und zwar diesen Block:

Code: Alles auswählen

	* create the overall dl mod jumpbox
	*/
	$dl_jumpbox = '<form method="get" name="dl_jumpbox" action="'.append_sid("downloads.$phpEx?sort_by=$sort_by&order=$order").'" onsubmit="if(this.options[this.selectedIndex].value == -1){ return false; }" /><select name="cat" onchange="if(this.options[this.selectedIndex].value != -1){ forms[\'dl_jumpbox\'].submit() }">';
	$dl_jumpbox .= '<option value="-1">'.$lang['Dl_cat_name'].'</option>';
	$dl_jumpbox .= '<option value="-1">----------</option>';
	$dl_jumpbox .= $dl_mod->dl_dropdown(0, 0, $cat, 'auth_view');
	$dl_jumpbox .= '</select>&nbsp;<input type="submit" value="'.$lang['Go'].'" class="liteoption" /></form>';
Schau bitte mal hier rein: http://www.validome.org/forum/viewtopicp-503-1.htm

oxpus

Beitrag von oxpus »

Hm, ich kann da auch keinen Fehler feststellen, zumal ja die Struktur scheinbar fehlerfrei ist...

Dungeonwatcher

Beitrag von Dungeonwatcher »

Ich bin auch ratlos. :?

Ich werde dieses Rätsel aber noch knacken. 8)

Dafür gibt's noch'n Fehler in der view_dl_cat_body.tpl:

Code: Alles auswählen

<td align="right" class="{downloads.sub.sub.sublevel_row.ROW_CLASS}"><span class="genmed">{downloads.sub.sublevel_row.SUBLEVEL_COUNT}&nbsp;</span></td>
ergibt:

Code: Alles auswählen

<td align="right" class=""><span class="genmed">&nbsp;</span></td>
Hier wird downloads.sub.sub.sublevel_row.ROW_CLASS nicht ausgeführt. Da ist ein sub zuviel im Namen.

Dito in dieser Zeile:

Code: Alles auswählen

<td align="right" class="{subcats.sub.sub.sublevel_row.ROW_CLASS}"><span class="genmed">{subcats.sub.sublevel_row.SUBLEVEL_COUNT}&nbsp;</span></td>

oxpus

Beitrag von oxpus »

Gut, danke, schau ich mir heute Nachmittag an.

EDIT
MOD ist aktualisiert.

Dungeonwatcher

Beitrag von Dungeonwatcher »

Moin mein großer Meister! 8)
oxpus hat geschrieben:Hm, ich kann da auch keinen Fehler feststellen, zumal ja die Struktur scheinbar fehlerfrei ist...
Der Fehler ist gefunden. :D

dl_footer.php:

Code: Alles auswählen

	$dl_jumpbox = '<form method="get" name="dl_jumpbox" action="'.append_sid("downloads.$phpEx?sort_by=$sort_by&order=$order").'" onsubmit="if(this.options[this.selectedIndex].value == -1){ return false; }" /><select name="cat" onchange="if(this.options[this.selectedIndex].value != -1){ forms[\'dl_jumpbox\'].submit() }">';
ersetze durch

Code: Alles auswählen

	$dl_jumpbox = '<form method="get" name="dl_jumpbox" action="'.append_sid("downloads.$phpEx?sort_by=$sort_by&order=$order").'" onsubmit="if(this.options[this.selectedIndex].value == -1){ return false; }"><select name="cat" onchange="if(this.options[this.selectedIndex].value != -1){ forms[\'dl_jumpbox\'].submit() }">';
Hinter dem { return false; }" /> gehört ein >.

Nun gibt es einen Folgefehler in der view_dl_cat_body.tpl:

Code: Alles auswählen

<a href="{downloads.U_CAT_LAST_USER}"class="gensmall">{downloads.CAT_LAST_USER}</a>
ersetze durch

Code: Alles auswählen

<a href="{downloads.U_CAT_LAST_USER}" class="gensmall">{downloads.CAT_LAST_USER}</a>
Hier fehlte zwischen

{downloads.U_CAT_LAST_USER}" und class="gensmall"> ein Leerzeichen. :roll:

Für die "Ungläubigen" :wink: unter uns --> http://dungeon-bbs.homeip.net/forum/downloads.php

Bis zum WoE checke ich die Templates noch mal alle durch, dann kommen sie per PN.

oxpus

Beitrag von oxpus »

Ah gut. Wird korrigiert.
Das mache ich dann aber wirklich erst heute Nachmittag.
Klappt jetzt aktuell schlecht.

Wenn Du dann die Templates durch hast, vergleiche ich die mit denen des MODs und aktualisiere die Version zur 5.0.13.
Damit auch alle anderen valid Templates und gefixte Scripte.

Bis dato schon einmal ein dickes Danke für die akribische Arbeit :!:

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:Ah gut. Wird korrigiert.
Das mache ich dann aber wirklich erst heute Nachmittag.
Klappt jetzt aktuell schlecht.
Keine Hektik.
Wenn Du dann die Templates durch hast, vergleiche ich die mit denen des MODs und aktualisiere die Version zur 5.0.13.
Damit auch alle anderen valid Templates und gefixte Scripte.
So soll es dann sein.
Bis dato schon einmal ein dickes Danke für die akribische Arbeit :!:
Nichts zu danken und Sorry für das generve. :wink:

Jetzt geht's in die Federn und vorpennen für 'ne Nachtschicht... :?

oxpus

Beitrag von oxpus »

Okay, gn8. Wenn man so sagen darf.
Und "generve"?
Nicht doch. Bin ja froh um jeden Bug, der behoben wird.

Dungeonwatcher

Beitrag von Dungeonwatcher »

Gut dann geht's weiter mit der FAQ. ;)

Suche in der deutschen und englischen Fassung alle <br> und ersetze mit <br />.

Suche in beiden Fassungen:

Code: Alles auswählen

<img border="0" src="'.$images['Dl_green'].'">
und ersetze mit:

Code: Alles auswählen

<img border="0" src="'.$images['Dl_green'].'" alt=""  title="" />
W3C = http://dungeon-bbs.homeip.net/forum/faq.php

oxpus

Beitrag von oxpus »

Merci ;)

Dungeonwatcher

Beitrag von Dungeonwatcher »

oxpus hat geschrieben:Merci ;)
:lol: Für heute ist aber Schluß, meine Brötchengeber verlangt nach mir. :(

oxpus

Beitrag von oxpus »

Kein Thema.
Eilt ja nicht.

Dungeonwatcher

Beitrag von Dungeonwatcher »

Moin großer Meister" 8)

Nimm die dl_stat_body.tpl:

suche mehrmals:

Code: Alles auswählen

boader="0"class="forumline"
und ersetze mit:

Code: Alles auswählen

border="0" class="forumline"
Das war wohl eine Kombination aus "dicker Daumen" und "Copie and paste". :wink:

Suche in downloads_body.tpl:

Code: Alles auswählen

<input type="submit" class="liteoption" value="{L_GO}" /></span></td>
und ersetze mit:

Code: Alles auswählen

<input type="submit" class="liteoption" value="{L_GO}" /></td>
Hier ist das </span> zu viel.

Nun die view_dl_body.tpl:

Suche:

Code: Alles auswählen

<form action={download_button.U_DOWNLOAD} method="post" name="download">
und ersetze mit:

Code: Alles auswählen

<form action="{download_button.U_DOWNLOAD}" method="post" name="download">
Da fehlen die " bei action

suche:

Code: Alles auswählen

ecllpadding="0"
und ersetze:

Code: Alles auswählen

cellpadding="0"
die dl_details.php:

suche nach:

Code: Alles auswählen

'EDIT_IMG' => '<img src="'.$images['icon_edit'].'" border="0" />',
ersetze mit:

Code: Alles auswählen

'EDIT_IMG' => '<img src="'.$images['icon_edit'].'" border="0" title="Edit" alt="Edit" />',
suche nach:

Code: Alles auswählen

<img src="'.$images['icon_search'].'" border="0" title="'.$lang['Search'].'" alt="'.$lang['Search'].'"></a>',
und ersetze mit:

Code: Alles auswählen

<img src="'.$images['icon_search'].'" border="0" title="'.$lang['Search'].'" alt="'.$lang['Search'].'" /></a>',
Hier gehört ein /> vor das </a>.

suche nach:

Code: Alles auswählen

'U_DOWNLOAD' => append_sid("downloads.$phpEx?view=load&id=$id"))
und ersetze mit:

Code: Alles auswählen

'U_DOWNLOAD' => append_sid("downloads.$phpEx?view=load&id=$id"))
Hier wurde das view=load& bemängelt.

So jetzt mach ich mir 'n Kaffee. :wink:

Gesperrt