Beiträge von info@biketeile-service.de

Zitat

PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in /var/www/vhosts/httpdocs/seo-2.4/inc/cseo_db.inc.php on line 158, referer:

Dieser Bug ist mit deinem Problem nichts zu tun. Ich gebe dir aber Recht, es ist einfach ärgerlich. Für solchen Preis, den man für Shop bezahlt hat, könnte der Entwickler besser seinen Job machen.

Fast alle Konstanten, die in commerce_seo_url.php (und in den einbezogenen Skripte) verwendet werden, werden erst in application_top.php definiert/initialisiert.
application_top.php, wie man aus dem commerce_seo_url.php entnehmen kann, nirgendwo einbezogen wurde. Den äquivalenten Ersatz dafür hat man auch nicht realisiert. Wir haben also "jede Menge" undeklarierte Konstanten:

PHP Notice: Use of undefined constant MODULE_COMMERCE_SEO_URL_LOWERCASE - assumed 'MODULE_COMMERCE_SEO_URL_LOWERCASE' in commerce_seo_url.php on line 37
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 158
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 164
PHP Notice: Use of undefined constant DB_CACHE - assumed 'DB_CACHE' in inc/cseo_db.inc.php on line 234
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 158
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 164
PHP Notice: Use of undefined constant DB_CACHE - assumed 'DB_CACHE' in inc/cseo_db.inc.php on line 234
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 158
PHP Notice: Use of undefined constant STORE_DB_TRANSACTIONS - assumed 'STORE_DB_TRANSACTIONS' in inc/cseo_db.inc.php on line 164
PHP Notice: Use of undefined constant DB_CACHE - assumed 'DB_CACHE' in inc/cseo_db.inc.php on line 234
PHP Notice: Use of undefined constant MODULE_COMMERCE_SEO_404_HANDLING - assumed 'MODULE_COMMERCE_SEO_404_HANDLING' in commerce_seo_url.php on line 111

v2.4.13. Bug bestätigt.

Zitat

denn der Fehler kommt aus der default.php

product_listing.php

Am Beispiel des TS:
http://www.testshop.tierland.net/latanis-parasi…e=0&per_site=-0

Zitat

nach ein paar Stunden funktioniert dann alles wieder normal.

Weil per_site in einer Sessionvariable gespeichert wird. Man muss sich abmelden und wieder anmelden, dann es ist wieder ok. Oder man ruft das URL mit korrekten Parameter auf (http://www.testshop.tierland.net/latanis-parasi…ge=1&per_site=1).

Die Ursache liegt daran, dass der Entwickler die Eingabe nicht validiert.
product_listing.php, from line 24
class.split_page_results.php, from line 98

Zugang zu den einigen Seiten des Adminbereiches ist (durch Konfiguration) auch für nicht Admin möglich (etwa Mitarbeiter, die nur Artikeln einpflegen).
Allerdings habe ich allgemein gesagt. mysql_real_escape_string muss beim Programmierer "im Blut" sein.

Das ist offensichtlicher fetter Bug. Solche Fehler führen früher oder später zum SQL injection attack.
Lies mal mysql_real_escape_string und korrigiere.

product_ask_a_question.php, line 97:

$module_smarty->assign('ALL_OK', PRODUCT_ASK_A_QUESTION_SUCCESS);
xtc_redirect(xtc_href_link(FILENAME_PRODUCT_INFO, 'products_id=' . $_POST['products_id']) . '#paq');

Quizfrage: Sieht der Besucher irgendwann, dass seine Produktfrage erfolgreich abgeschickt wurde?
Ich verstehe, dass dies eine "Lösung" für

$smarty = new Smarty;
...
require_once (DIR_FS_INC . 'cseo_get_mail_body.inc.php');

war. (Und warum konnte man das originale Smarty einfach nicht in der Zwischenvariable sichern und anschließend wiederherstellen?..)
Andererseits cseo_get_mail_body.inc.php ist einfach Murks. includes/classes/Smarty_3/cseo_plugins beinhaltet einen schwachen Fehlversuch es richtig zu machen.
Die Doku zum Smarty resources beschreibt, wie man die resources anbindet, um so was "require_once (DIR_FS_INC . 'cseo_get_mail_body.inc.php');" zu vermeiden.

http://plussupport.commerce-seo.de/showthread.php…ull=1#post12451

item.php, line 51:
global $order, $total_count;
Globale variable $total_count ist nicht initialisiert.

Mögliche Lösung:
shipping_estimate.php, line 20:
$total = $_SESSION['cart']->show_total();
erweitern so
$total = $_SESSION['cart']->show_total();
$total_count = $_SESSION['cart']->count_contents();

Fehlt sowohl in English als auch in Deutsch
[ATTACH=CONFIG]601[/ATTACH]

Es gibt eine im "logfiles"-Verzeichnis im Shop.
Es gibt auch eine Errorlog-Datei beim Hoster.

Zitat

Also es hat sich erledigt, Hoster hat Backup zurückrestored

Wenn Backup auch Webspace beinhaltet, dann kann natürlich die Log-Datei des Shops leer sein.

Zeige log-Datei.

Und "Selbstabholung (Selbstabholung der Ware in unserer Geschäftsstelle.): 0,00 EUR" sieht für dich nicht komisch aus?
Wäre evtl. besser für die versandkostenfreie Lieferung einen "schöneren" (treffsichereren) Namen zu vergeben, der aber eindeutig sagen würde, dass die Versandkosten 0,00 EUR nicht wegen irgendeines Fehlers entstanden sind, sondern durch den Shopbesitzerabsicht?

main_categories_list.php, lines 68,70
function htmlentities

Zitat

Wie bei htmlspecialchars() kann über das optionale dritte Argument encoding die bei der Konvertierung zu verwendende Zeichenkodierung festgelegt werden. Standardwert ist ISO-8859-1 in PHP Versionen vor 5.4.0 und UTF-8 in PHP 5.4.0 und neuer. Auch wenn das Argument optional ist, wird dennoch empfohlen eine für den Code richtige Kodierung anzugeben.

ot_shipping.php

function ot_shipping() {
    ...
    $this->title = MODULE_ORDER_TOTAL_SHIPPING_TITLE;
    ...
}


function process() {
    ...
    if (($pass == true) && ( ($order->info['total'] - $order->info['shipping_cost']) >= $xtPrice->xtcFormat(MODULE_ORDER_TOTAL_SHIPPING_FREE_SHIPPING_OVER, false, 0, true))) {
        $order->info['shipping_method'] = $this->title;
    ...
}

Gibt es irgendeinen wichtigen Grund, warum shipping_method auf den Titel des Zusammenfassungsmodules gesetzt wird?
Normalerweise steht im shipping_method der Versandartname, z. B. Deutsche Post (Versand nach AT: (1 x 3 kg)).
Bei der versandkostenfreien Lieferung habe ich erwartet "Versandkostenfrei". Tatsächlich ist einfacheres "Versandkosten" dort zu finden.

Zitat

Ich deute mal "zu anstrengend" mit zu groß?

Korrekt.
Geht die Änderung in das nächste Update rein? Oder nur ich finde es zu groß?

Siehe screenshot
[ATTACH=CONFIG]591[/ATTACH]

function xtcCalculateCurr($price) {
    return $this->currencies[$this->actualCurr]['value'] * $price;
}

$xtPrice->xtcCalculateCurr($quote['methods'][0]['cost'], $quote['tax'])
Teste zuerst selbst

Versandkosten: 10.00 Einheiten
CHF Kurs: 2.50 (nicht realer Kurs, nur zum Zeigen)

$quote['methods'][0]['cost'] = $xtPrice->xtcCalculateCurr($quote['methods'][0]['cost']);
$quote['methods'][0]['cost'] hat 25.00 CHF (hier ist der Betrag richtig)
'VALUE' => $xtPrice->xtcFormat(((isset($quote['tax']) && $quote['tax'] > 0) ? $xtPrice->xtcAddTax($quote['methods'][0]['cost'], $quote['tax']) : (!empty($quote['methods'][0]['cost']) ? $quote['methods'][0]['cost'] : '0')), true)
'VALUE' hat 62.50 CHF
'VALUE' wird auf dem Frontend angezeigt.
$total += ((isset($quote['tax']) && $quote['tax'] > 0) ? $xtPrice->xtcAddTax($quote['methods'][0]['cost'], $quote['tax']) : (!empty($quote['methods'][0]['cost']) ? $quote['methods'][0]['cost'] : '0'));
$total wird um 62.50 CHF erhöht, was falsch ist, weil Versandkosten nur 25.00 CHF beträgt.

Wo korrekt?
Warenkorb:
[ATTACH=CONFIG]587[/ATTACH]
Kasse:
[ATTACH=CONFIG]586[/ATTACH]

$quote['methods'][0]['cost'] = $xtPrice->xtcCalculateCurr($quote['methods'][0]['cost']);
$total += ((isset($quote['tax']) && $quote['tax'] > 0) ? $xtPrice->xtcAddTax($quote['methods'][0]['cost'], $quote['tax']) : (!empty($quote['methods'][0]['cost']) ? $quote['methods'][0]['cost'] : '0'));

xtcCalculateCurr wird 2 Mal aufgerufen (siehe fettmarkiert). Meinst du, dass es richtig ist?

Zitat

eine Änderung bringt hier keine ersichtliche Änderung

Sorry, ich habe es nicht verstanden. Was meinen Sie damit?

Zitat

der dessen code "sauber" ist, werfe den ersten Stein!

Es ist klar, dass es keinen absoluten sauberen Code gibt. Man muss jedoch immer danach streben, da der saubere Code die blöden Fehler vermeidet.