order_total.php, print_order.php - und andere kleinere Bugs

[metoo]

Hallo,

bin bei meiner Suche nicht fündig geworden, daher gehe ich mal davon aus, dass die Sachen hier noch nicht gepostet wurden:

1. includes/classes/order_total.php (Zeile 255 / 256)

Code: PHP  [Auswählen]

for ($i = 0; $i <$size; $i ++) {                                                        $output_string .= '              <tr>'."\n".'                <td align="right" class="main">'.$GLOBALS[$class]->output[$i]['title'].'</td>'."\n".'                <td align="right" class="main">'.$GLOBALS[$class]->output[$i]['text'].'</td>'."\n".'              </tr>';
     }
 

Hier werden z.B. in der checkout_confirmation im Bestellablauf Tabellenbestandteile gestartet, ohne dass eine entsprechende Tabelle existiert. Dadurch landen die Zwischenbeträge alle in einer Zeile.

Lösungsansatz: Ich habe das mal durch folgenden Code ersetzt (wobei auf die entsprechenden Abmessungen des Templates geachtet werden sollte).

Code: PHP  [Auswählen]

for ($i = 0; $i <$size; $i ++) {
                                                $output_string .= '<div style="text-align:right; float:left; width:570px;">'.$GLOBALS[$class]->output[$i]['title'].'</div><div style="text-align:right; float:left; width:100px;"> '.$GLOBALS[$class]->output[$i]['text'].'</div><div style="clear:both;"></div>'."\n";
                                        }
 

2. templates/xtc5/modules/print_order.html (Zeile

Code: PHP  [Auswählen]

        <link rel="stylesheet" type="text/css" href="templates/xtc5/stylesheet.css" />
 

templates/xtc5/modules/print_order.html (Zeile 19)

Code: PHP  [Auswählen]

<td style="vertical-align:top; text-align:right;"><img src="templates/xtc5/img/logo.gif" alt="{$store_name}" /></td>
 

Hier ist das Template statisch gesetzt und "templates/xtc5/" müsste durch {$tpl_path} ersetzt werden. Kann sonst zu Problemen führen, wenn man den Template-Ordner umbenennt.

3. includes/header.php (Zeile 38)

Code: PHP  [Auswählen]

        <link rel="shortcut icon" href="favicon.ico" type="image/x-icon" />
 

Hier müsste der Domainname mit in den Link zum Favicon rein, da sonst in jedem "Unterordner" nach einem Favicon gesucht wird (insbesondere bei den Suma-URLs).

4. checkout_payment

Bei ausgeschaltetem Javascript und fehlendem Haken bei den AGB wird folgende Fehlermeldung ausgegeben:
"* Sofern Sie unsere Allgemeinen Gesch&auml;ftsbedingungen nicht akzeptieren,<br /> k&ouml;nnen wir Ihre Bestellung bedauerlicherweise nicht entgegennehmen!<br /><br />"

Ich nehme mal an, dass die Meldung von der SSEQlib zerlegt wird, hab mir das aber noch nicht näher angeschaut und noch keine Lösung.

MfG,
Dennis



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

[metoo]

Bei dem oberen Beitrag zu Punkt 4 wird die Meldung nicht so dargestellt, wie Sie im Shop dargestellt wird, also erneuter Versuch.

So sieht's da im Shop aus:

Code: PHP  [Auswählen]

&#x2a;&#x20;Sofern&#x20;Sie&#x20;unsere&#x20;Allgemeinen&#x20;Gesch&auml&#x3b;ftsbedingungen&#x20;nicht&#x20;akzeptieren&#x2c;&#x3c;br&#x20;&#x2f;&#x3e;&#x20;k&ouml&#x3b;nnen&#x20;wir&#x20;Ihre&#x20;Bestellung&#x20;bedauerlicherweise&#x20;nicht&#x20;entgegennehmen&#x21;&#x3c;br&#x20;&#x2f;&#x3e;&#x3c;br&#x20;&#x2f;&#x3e;
 

[metoo]

Hat beim 2. Versuch leider auch nicht geklappt, also nun als Bild:

  [ Für Gäste sind keine Dateianhänge sichtbar ]

[Tomcraft]

Zu 1.) Probelm ist bereits behoben worden! -> http://www.modified-shop.org/forum/topic.php?id=1422#post-10619
Zu 2.) print_order.html und print_product_info.html werden angepast!
Hier sind die Links wie folgt zu setzen:

Code: PHP  [Auswählen]

<link rel="stylesheet" type="text/css" href="{php}echo DIR_WS_CATALOG;{/php}templates/{php}echo CURRENT_TEMPLATE;{/php}/stylesheet.css" />
 

und:

Code: PHP  [Auswählen]

<td style="vertical-align:top; text-align:right;"><img src="{php}echo DIR_WS_CATALOG;{/php}templates/{php}echo CURRENT_TEMPLATE;{/php}/img/logo.gif" alt="{$store_name}" /></td>
 

Zu 3.) Wen stört das?!
Alternativ:

Code: PHP  [Auswählen]

<link rel="shortcut icon" href="<?php echo DIR_WS_CATALOG.'favicon.ico'; ?>" type="image/x-icon" />
 

Noch besser, favicon.ico ins Template auslagern!

Code: PHP  [Auswählen]

<link rel="shortcut icon" href="<?php echo DIR_WS_CATALOG.'templates/'.CURRENT_TEMPLATE.'/favicon.ico'; ?>" type="image/x-icon" />
 

Zu 4.) Momentan kein Support von mir wegen der SSEQ-LIB, sorry...

[DeineAgentur]

3. includes/header.php (Zeile 38)

Code: PHP  [Auswählen]

        <link rel="shortcut icon" href="favicon.ico" type="image/x-icon" />
 

Hier müsste der Domainname mit in den Link zum Favicon rein, da sonst in jedem "Unterordner" nach einem Favicon gesucht wird (insbesondere bei den Suma-URLs).

Da würde folgendes Abhilfe schaffen:

Code: PHP  [Auswählen]

<link rel="shortcut icon" href="<? echo (($request_type == 'SSL') ? HTTPS_SERVER : HTTP_SERVER).DIR_WS_CATALOG;?>favicon.ico" type="image/x-icon" />
 

Gruß Josef

[Tomcraft]

Wieso absolut, wenn relativ reicht?

Was gefällt dir nicht an:

Code: PHP  [Auswählen]

<link rel="shortcut icon" href="<?php echo DIR_WS_CATALOG.'favicon.ico'; ?>" type="image/x-icon" />
 

???

Grüße

Torsten

[DeineAgentur]

Finde Absolute für diverse dinge einfach besser. War ja kein Tadel sondern ein Vorschlag.

[Tomcraft]

Okidoki... im neuen Paket wird es trotzdem mit relativer Pfadangabe drin stehen. ÄTSCH!

[mantis]

@Tomcraft

finde den relative Pfad auch Robuster, und ein absoluter Pfad hat keine ersichtlichen Vorteile.

Gruss
mantis

[mizzy]

Hallo Tomcraft,

danke für die Codes, hab ich gleich mal nach Deinen Vorschlägen eingebaut.

Schönes Wochenende Euch

V.G. Micha

[Tomcraft]

Freut mich... In der nächsten Shop-Version werden die fixes auch integriert sein.

Grüße

Torsten