Seite 1 von 2
W3: 5 Validate-Fehler, könnt ihr mir helfen?
Verfasst: 29.12.2005 10:37
von Diesel
Hi,
Hatte mich gestern eifrig daran gemacht, mein Board W3 konform zu machen, habe von den ehemalig 26 Fehlern auch schon einiges fixen können, aber 5 sind jetzt noch übrig.
http://validator.w3.org/check?uri=http: ... osting.de/
Könnt ihr mir sagen, was ich bei diesen Fehlern im Quelltext verändern muss?
Lieben Dank!
Verfasst: 29.12.2005 11:21
von CMPro
Error Line 279 column 65: there is no attribute "BACKGROUND".
verwende anstelle von "background"
die geschichte mit dem "<br />", da versuch mal das / wegzulassen
danach mal schauen was noch für fehler auftreten
Verfasst: 29.12.2005 11:34
von S2B
Wie bitte? Das / bei <br /> weglassen?
Der <br />-Fehler und auch der letzte Fehler kommen daher, dass der Page Generation Time-MOD HTML-Code nach </body></html> einfügt, was nicht erlaubt ist.
Beim obersten Fehler könnte ich mir vorstellen, dass er von den nic.de.vu-Standardheadern kommt. Die Meta- und Link-Tags werden nämlich dort nicht korrekt mit / geschlossen.
Edit: Das zweite ist Quatsch, sorry. *duck*
Verfasst: 29.12.2005 11:42
von CMPro
mag ja sein das es dahingehört..aber das mit dem "versuch" soll ja eigentlich zeigen das ich selber auch nicht sicher bin / war
Verfasst: 29.12.2005 11:43
von Diesel
S2B hat geschrieben:Wie bitte? Das / bei <br /> weglassen?
Der <br />-Fehler und auch der letzte Fehler kommen daher, dass der Page Generation Time-MOD HTML-Code nach </body></html> einfügt, was nicht erlaubt ist.
[/i]
4 Fehler haben wa noch...
Also, die letzten 3 kommen alle dadurch, das es nach den beiden Abschlusselementen "</body></html>" eingefügt ist, sehe ich jetzt auch gerade, wo mans mal direkt anschaut.
Wie müsste man den Mod den umcoden, damit es richtig klappt? Dann wäre nur noch ein Fehler bis zu Perfektion...
Der Mod hat sich in der "page_tail.php" eingebettet, hab die entsprechende Stelle gepostet:
Code: Alles auswählen
/* Un-comment the line below to restrict Admins only to view page generation info */
//if( ($userdata['session_logged_in']) and ($userdata['user_level'] == ADMIN) )
{
$gzip_text = ($board_config['gzip_compress']) ? 'GZIP enabled' : 'GZIP disabled';
$debug_text = (DEBUG == 1) ? 'Debug on' : 'Debug off';
$excuted_queries = $db->num_queries;
Tja, und der Fehler in Zeile 15, müsste man noch grübeln...
Verfasst: 29.12.2005 12:00
von S2B
Der Teil muss vor $template->pparse(); ausgegeben werden.
Verfasst: 29.12.2005 12:38
von Diesel
Hi,
Beim eincoden vor
sind die Validate-Fehler zwar weg, aber die Generate-Stats werden vor dem Footer (Oder eher gesagt gleich am Anfang) ausgeworfen.
Könnte man das so umcoden, das alles im Footer erstmal aufgeworfen wird und sich erst dann die Stats blicken lassen? Das wäre die beste Variante.
Aktuell sieht das ganze so aus:
Code: Alles auswählen
//
// Show the overall footer.
//
$admin_link = ( $userdata['user_level'] == ADMIN ) ? '<a href="admin/index.' . $phpEx . '?sid=' . $userdata['session_id'] . '">' . $lang['Admin_panel'] . '</a><br /><br />' : '';
$template->set_filenames(array(
'overall_footer' => ( empty($gen_simple_header) ) ? 'overall_footer.tpl' : 'simple_footer.tpl')
);
$template->assign_vars(array(
'PHPBB_VERSION' => '2' . $board_config['version'],
'TRANSLATION_INFO' => (isset($lang['TRANSLATION_INFO'])) ? $lang['TRANSLATION_INFO'] : ((isset($lang['TRANSLATION'])) ? $lang['TRANSLATION'] : ''),
'ADMIN_LINK' => $admin_link)
);
$template->pparse('overall_footer');
/* Un-comment the line below to restrict Admins only to view page generation info */
//if( ($userdata['session_logged_in']) and ($userdata['user_level'] == ADMIN) )
{
$gzip_text = ($board_config['gzip_compress']) ? 'GZIP enabled' : 'GZIP disabled';
$debug_text = (DEBUG == 1) ? 'Debug on' : 'Debug off';
$excuted_queries = $db->num_queries;
$mtime = microtime();
$mtime = explode(" ",$mtime);
$mtime = $mtime[1] + $mtime[0];
$endtime = $mtime;
$gentime = round(($endtime - $starttime), 4);
$sql_time = round($db->sql_time, 4);
$sql_part = round($sql_time / $gentime * 100);
$php_part = 100 - $sql_part;
echo '<br /><div style="font-family: Verdana; font-size: 10px; color: #444444; letter-spacing: -1px" align="center">Seite generiert in '. $gentime .' Sekunden (PHP: '. $php_part .'% - SQL: '. $sql_part .'%) und SQL-Abfragen: '. $excuted_queries .'</div>';
}
//
// Close our DB connection.
//
$db->sql_close();
Verfasst: 29.12.2005 12:54
von S2B
Code: Alles auswählen
/* Un-comment the line below to restrict Admins only to view page generation info */
//if( ($userdata['session_logged_in']) and ($userdata['user_level'] == ADMIN) )
{
$gzip_text = ($board_config['gzip_compress']) ? 'GZIP enabled' : 'GZIP disabled';
$debug_text = (DEBUG == 1) ? 'Debug on' : 'Debug off';
$excuted_queries = $db->num_queries;
$mtime = microtime();
$mtime = explode(" ",$mtime);
$mtime = $mtime[1] + $mtime[0];
$endtime = $mtime;
$gentime = round(($endtime - $starttime), 4);
$sql_time = round($db->sql_time, 4);
$sql_part = round($sql_time / $gentime * 100);
$php_part = 100 - $sql_part;
$pagegen = '<br /><div style="font-family: Verdana; font-size: 10px; color: #444444; letter-spacing: -1px" align="center">Seite generiert in '. $gentime .' Sekunden (PHP: '. $php_part .'% - SQL: '. $sql_part .'%) und SQL-Abfragen: '. $excuted_queries .'</div>';
}
$template->assign_vars(array(
'PHPBB_VERSION' => '2' . $board_config['version'],
'TRANSLATION_INFO' => (isset($lang['TRANSLATION_INFO'])) ? $lang['TRANSLATION_INFO'] : ((isset($lang['TRANSLATION'])) ? $lang['TRANSLATION'] : ''),
'PAGEGEN' => $pagegen,
'ADMIN_LINK' => $admin_link)
);
$template->pparse('overall_footer');
und in der overall_footer.tpl:
an die Stelle, an der die Page Generation Time angezeigt werden soll.
Verfasst: 29.12.2005 13:26
von Diesel
Vielen Dank, das hat mir sehr geholfen!
Jetzt bleiben noch 2 Fehler im Dokument übrig, habt ihr dazu noch Ideen?
Verfasst: 29.12.2005 18:15
von IPB_Flüchtling
@Diesel:
Auch auf die Gefahr hin, Dir den Spaß zu verderben:
http://validator.w3.org/check?uri=http% ... ype=Inline
Liebe Grüße,
IPB_Flüchtling