Thema: HTML-Unsauberkeit bei products_info

Igotcha · **am:** 13. November 2010, 11:39:30

Hallo zusammen,

ich habe eine kleine HTML-Unsauberkeit bei der products_info.php, die ich mir nicht erklären kann.

Meine index.php (also Shopstartseite( ist 100% valide, was mir die entsprechenden Tools anzeigen.

Beim Aufruf von Produktdetails (product_info.php) habe ich aktuell 13 HTML-Warnungen.

Die ersten Zeilen des Browsercode meiner (unveränderten) products_info.php sehen so aus:

Code: XML [Auswählen]

// dies sind Leerzeilen im Browsercode (muss ich so darstellen, da diese ansonsten nicht angezeigt werden
//
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" xml:lang="de">
<head>

Ich habe zwei Leerzeilen über der DOCTYPE, wo ich beim besten Willen nicht weiss, wo diese herkommen. Daraus resultieren wahrscheinlich diese 13 HTML-Warnungen, die ich gerne weg haben möchte.

Danke und Grüße

Linkback: https://www.modified-shop.org/forum/index.php?topic=9279.0

Alfred · **Antwort #1 am:** 13. November 2010, 11:46:11

Hallo,

schau mal die php-Dateien durch die per include eingebunden werden.

Gruß

Igotcha · **Antwort #2 am:** 13. November 2010, 12:15:15

Da habe ich gestern mehrere Stunden mit verbracht

Der Unterschied zwischen index.php und products_info.php ist folgender:

Code: PHP [Auswählen]

if ($_GET['products_id']) {
$cat = xtc_db_query("SELECT categories_id FROM ".TABLE_PRODUCTS_TO_CATEGORIES." WHERE products_id='".(int) $_GET['products_id']."'");
$catData = xtc_db_fetch_array($cat);
require_once (DIR_FS_INC.'xtc_get_path.inc.php');
if ($catData['categories_id'])
$cPath = xtc_input_validation(xtc_get_path($catData['categories_id']), 'cPath', '');

}

require (DIR_FS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/source/boxes.php');

// include needed functions
require_once (DIR_FS_INC.'xtc_get_download.inc.php');
//require_once (DIR_FS_INC.'xtc_delete_file.inc.php'); // Hetfield - 2009-08-12 - removed never needed function
require_once (DIR_FS_INC.'xtc_get_all_get_params.inc.php');
require_once (DIR_FS_INC.'xtc_date_long.inc.php');
require_once (DIR_FS_INC.'xtc_draw_hidden_field.inc.php');
require_once (DIR_FS_INC.'xtc_image_button.inc.php');
require_once (DIR_FS_INC.'xtc_draw_form.inc.php');
require_once (DIR_FS_INC.'xtc_draw_input_field.inc.php');
require_once (DIR_FS_INC.'xtc_image_submit.inc.php');

if ($_GET['action'] == 'get_download') {
xtc_get_download((int)$_GET['cID']); // Hetfield - 2009-08-12 - update function call for security
}

include (DIR_WS_MODULES.'product_info.php');

An den ganzen zu inkludierenden Dateien war ich nicht dran.

Bei einer anderen Shopseite, die modified eCommerce Shopsoftware-Standard ist, liegt genau das selbe Problem vor.

Schaut euch mal im Browser die ersten Codezeilen von http://kombas.de/shop/index.php und dann von http://kombas.de/shop/Software/ArticleSpinningWizard::1.html an - auch die zwei Leezeilen am Anfang der products_info.php.

Alfred · **Antwort #3 am:** 15. November 2010, 07:04:08

Zitat von: Igotcha am 13. November 2010, 12:15:15

[...]
An den ganzen zu inkludierenden Dateien war ich nicht dran.
[...]

Hallo,

dort wirst du die Leerzeilen finden.

Gruß

DokuMan · **Antwort #4 am:** 15. November 2010, 07:52:57

Das "könnte" daran liegen, dass z.B. in der /modules/product_info.php ganz unten nach dem schließenden Tag ?> noch eine Leerzeile enthalten ist.

Die zweite Leerzeile steckt dann wohl am Ende in "/inc/xtc_draw_form.inc.php".

Beides ist in der kommenden Version bereits bereinigt.

Igotcha · **Antwort #5 am:** 15. November 2010, 10:37:55

Leider nicht, habe beide Dateien geprüft.

Ist ja kein Beinbruch, aber ein Grund meines Umstiegs von osCommerce auf modified eCommerce Shopsoftware war nun gerade das Layoutchaos in osCommerce und die mangelhafte Validität des generierten Codes dort.

Modulfux · **Antwort #6 am:** 15. November 2010, 11:06:11

Diese beiden Leerzeilen sind aber nicht für die Validierungsfehler verantwortlich.

Hinderlich sind ist eher das <noscript> im head-Bereich.

Gruß
Ronny

Igotcha · **Antwort #7 am:** 15. November 2010, 13:23:00

Ja, das kann u.a. auch daran liegen, ich erhalte folgende Warnungen bei der products_info.php:

Code: [Auswählen]

Result: 0 Fehler / 10 Warnungen

line 96 column 1 - Warnung: <noscript> isn't allowed in <head> elements
line 5 column 1 - Info: <head> previously mentioned
line 96 column 1 - Warnung: inserting implicit <body>
line 97 column 2 - Warnung: <link> isn't allowed in <noscript> elements
line 96 column 1 - Info: <noscript> previously mentioned
line 99 column 69 - Warnung: </head> isn't allowed in <body> elements
line 96 column 1 - Info: <body> previously mentioned
line 100 column 1 - Warnung: discarding unexpected <body>
Info: Doctype given is "-//W3C//DTD XHTML 1.0 Transitional//EN"
Info: Document content looks like XHTML 1.0 Transitional

Markus · **Antwort #8 am:** 15. November 2010, 16:36:00

Hi ...

das liegt nicht an den Dateien welche included werden und auch nicht an den Leerzeilen oben.

Problem 1:
Das noscript im Header ... das muss raus oder aber es muss irgendwie anders abgefangen werden. Was soll das denn bewirken bzw. für was ist das? Navi?

Problem 2:
Du bekommst einen Flashfilm innerhalb der Artikel fast nicht valide. Es gibt da zwar diverse Lösungsansätze ... mir war das aber dann immer zu blöd weil es ein riesiges Gefummel ist.

Problem 3:
Das Piwik-Zeug im Footer ist ausserhalb des schliessenden </body></html>

Problem 4:
In der Hersteller-Infobox ist ein

Code: PHP [Auswählen]

... auch das muss raus

Problem 5:
Das Logo hat keinen alt-Text.

Danach ist bei mir im Validator der Quelltext valide

Markus

Igotcha · **Antwort #9 am:** 15. November 2010, 18:29:04

1. das <noscript> kommt irgendwie aus der Standardinstallation.
2. die Seite war ja nur als Beispiel, da es hier auch vorkommt. Aktuell sitze ich noch an einer XAMPP-Version meines anderen Shop-Relaunches und da ist mir das Problem eigentlich erst aufgefallen.
3. Danke Dir, hab ich angepasst.
4. auch im Standard-Template enthalten.
5. Danke für den Hinweis.

Viele Grüße

Markus · **Antwort #10 am:** 15. November 2010, 18:40:53

Hi ...

und wieviele Fehler bekommst du jetzt noch bzw. ist das Problem gelöst?

Markus

Igotcha · **Antwort #11 am:** 15. November 2010, 19:12:35

Ich habe mir jetzt mal alle Dateien durchsuchen lassen, wo ein "<noscript>" drin ist und bin bei "xtc\templates\xtc5\javascript\general.js.php" im xtc5-Template fündig geworden.

Code: PHP [Auswählen]

<link rel="stylesheet" href="<?php echo 'templates/'.CURRENT_TEMPLATE; ?>/css/no_javascript.css" type="text/css" media="screen" />

Habe jetzt mal die beiden "<noscript>" auskommentiert und voila, alles paletti, auch mit abgeschaltetem JavaScript. Kann jetzt aber nicht weitere Auswirkungen abschätzen, denn da hat sich ja sicher jemand was bei gedacht...

Aber schon cool so ein valider Shop im Vergleich zu osCommerce.

Igotcha · **Antwort #12 am:** 15. November 2010, 20:03:48

Hab den "Haken" gefunden.

Mit auskommentierten "<noscript>" funktioniert die "product_info_tabs_v1.html" nicht mehr. Die Reiter werden nicht mehr angezeigt. Mist...

Modulfux · **Antwort #13 am:** 15. November 2010, 20:19:47

Das die Produkttabs nicht mehr funktionieren ist klar, denn durch das auskommentierte <noscript> wird nun auch zum no-script.css geladen und die überschreibt die anderen Sachen.

Zitat von: Markus am 15. November 2010, 16:36:00

Problem 2:
Du bekommst einen Flashfilm innerhalb der Artikel fast nicht valide. Es gibt da zwar diverse Lösungsansätze ... mir war das aber dann immer zu blöd weil es ein riesiges Gefummel ist.

Da wiederspreche ich vehement. Es gibt eine ganz klare Vorgabe, wie man Flasheinbettungen valide darstellt ohne dass ein Browser meckert. Das ganze auch ohne Riesengefummel.

Gruß
Ronny

Markus · **Antwort #14 am:** 15. November 2010, 20:37:40

Hi ...

@Igotcha ... jetzt kannst du dir überlegen ob du die Fallback-Möglichkeit behalten möchtest wenn jemand Javascript aktiviert hat .. oder ob du lieber einen validen Shop möchtest

@Webkiste ... kannst du mir die "einfache" Lösung mal bitte zeigen? ... auch ich lerne nie aus

Markus