W3: 5 Validate-Fehler, könnt ihr mir helfen?

[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!

[CMPro]

Error Line 279 column 65: there is no attribute "BACKGROUND".

verwende anstelle von "background"

Code: Alles auswählen

style="background-image: url('pfadzurgrafik');"

die geschichte mit dem "<br />", da versuch mal das / wegzulassen

danach mal schauen was noch für fehler auftreten

[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*

[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

[Diesel]

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...

[S2B]

Der Teil muss vor $template->pparse(); ausgegeben werden.

[Diesel]

Hi,
Beim eincoden vor

Code: Alles auswählen

$template->pparse('overall_footer');

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();

[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:

Code: Alles auswählen

{PAGEGEN}

an die Stelle, an der die Page Generation Time angezeigt werden soll.

[Diesel]

Vielen Dank, das hat mir sehr geholfen!

Jetzt bleiben noch 2 Fehler im Dokument übrig, habt ihr dazu noch Ideen?

[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