modified eCommerce Shopsoftware 2.0.7.2 rev 14622 veröffentlicht

[noRiddle (revilonetz)]

"der Wahnsinn" ?, hoffentlich nicht.

Den Eintrag in der sessions.php  hatte ich gar nicht gesehen.
Der macht aber auch nur Sinn wenn ebenfalls session.gc_divison  bestimmt wird.
ini_set()  geht im Übrigen nicht auf jedem Server, abhängig von dessen Konfig.

However muß ja alles abgefangen werden und somit ist, wie bereits gesagt, die "defined() OR define()"-Lösung die richtige.

Gruß,
noRiddle

[Tomcraft]

Danke dir!

Grüße

Torsten

[peter111]

Moin in die Runde,
unser 2.0.7.2 Shop läuft unter PHP 8.1 bringt aber immer wieder die gleichen Warnungen.
Leider kann ich die warnings nicht selbst herbeiführen und finde auch keinen Grund dafür!?

z.B. bei offensichtlich nicht mehr vorhandenen Seiten:

Code: PHP  [Auswählen]

Undefined variable $main_content in File: /.../quiltshop/shop_content.php on Line: 116  {}      {}

Ebenfalls häufig im Zusammenhang mit:

Code: PHP  [Auswählen]

Undefined array key "products_weight" in File: .../quiltshop/includes/classes/shopping_cart.php on Line: 526    {}
Undefined array key "products_id"  
Undefined array key "oID" in File usw{}

Ich habe die betroffenen Dateien bereits mehrfach geprüft und keine Abweichungen von den Originaldateien feststellen können. Kann es sein, dass ich ein Changeset übersehen habe?
Die 8 von Torsten empfohlenen Changesets habe ich eingepflegt.
 Gruß Peter

[Karl1]

Hallo Peter,
erstes passiert, wenn in shop_content.php die Variable $language_not_found auf "true" gesetzt ist.
Das passiert, wenn man versucht eine Contentseite mit falsche Sprache aufzurufen, z.B.:

Code: PHP  [Auswählen]

https://dein_shop/shop_content.php?coID=2&language=foo

Deine zweiten Warnung ist seltsam:

Code: PHP  [Auswählen]

... includes/classes/shopping_cart.php on Line: 526

Bei mir ist in Zeile 526 ein schließende geschweifte Klammer, die eigentlich keine Warnung erzeugt.

Gruß Karl

[peter111]

Hallo Karl,
danke für die Hinweise.
Ich habe nun festgestellt, dass die warning mit dem "Undefined variable $main_content" (fast) immer von Seiten herrühren, die nicht mehr existieren! Offensichtlich haben Kunden oder Bots alte Seiten verlinkt.
Falls das der einzige Grund ist sollte es gefixt werden, denn es ist lästig.
In der /includes/modules/default.php habe ich in Zeile 147 (irgendwo aus dem Forum) folgende Änderung:

Code: PHP  [Auswählen]

// Warning Problem bei leeren Untercategorien
        if(isset($categories_content)) {
                $default_smarty->assign('module_content', $categories_content);
        }

Ich hatte vermutet, dass das die Lösung für die warnings sei - ist es aber wohl nicht?!

Bei der zweiten warning (Asche auf mein Haupt) habe ich verschwiegen, dass der Shop dezimale Mengen verarbeiten kann und es daher einige Erweiterungen der /includes/classes/shopping_cart.php gibt.
Daher stimmen die Zeilennummern nicht mehr.
Ich denke es macht Sinn, dass der Entwickler des Moduls (Timo) das Teil nochmals anschaut.
Ich melde mich dann wieder.
Dank und Gruß
Peter

[Timm]

Lass den Timo in Ruhe, der hat keine Zeit.

Die Fehlermeldungen kommen auch in Shops ohne dezimale Mengen.

Poste einfach die korrekte Zeilennummer der Original Datei. Ich kenne die nicht, da ich neben den dezimalen Mengen noch den persistenten Warenkorb drin habe.

Gruß Timm

P.S. Und die Fehlermeldungen bitte dann komplett mit Backtraces und auch da mit angepassten Zeilennummern, wenn die Dateien angepasst wurden.

[peter111]

Moin,
meine shopping_cart passt laut Entwickler.
Blöd ist auch, dass diese warnings nicht täglich zu finden sind !?
Ich habe die warnings mal auszugsweise aufbereitet (solch eine lange Liste liest sonst niemand)

Code: PHP  [Auswählen]

2022-12-06 06:57:32]    [warning]       [modified]      [pid:9673]      WARNING found for URL: /Patchworkstoffe/Weihnachtsstoffe/Patchworkstoff-Makower-Wonderland-Haeuser::4702.html   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Trying to access array offset on value of type null in File: /.../quiltshop/includes/classes/shopping_cart.php on Line: 664     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #0 - /.../quiltshop/templates/bootstrap4/source/boxes/shopping_cart.php called at Line 30     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #1 - /.../quiltshop/templates/bootstrap4/source/boxes.php called at Line 193  {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #2 - /.../quiltshop/product_info.php called at Line 44        {}      {}

-- Zeile in meiner shopping_cart - im Original 635

Code: PHP  [Auswählen]

     $products_array[(int)$products_id]['products_tax_class_id'] = $xtPrice->xtc_get_tax_class($products_id, $products_array[(int)$products_id]['products_tax_class_id']);

Code: PHP  [Auswählen]

[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      WARNING found for URL: /Patchworkstoffe/Weihnachtsstoffe/Patchworkstoff-Makower-Wonderland-Haeuser::4702.html   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Undefined array key "products_id" in File: /.../quiltshop/includes/classes/shopping_cart.php on Line: 507       {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #0 - /.../quiltshop/templates/bootstrap4/source/boxes/shopping_cart.php called at Line 30     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #1 - /.../quiltshop/templates/bootstrap4/source/boxes.php called at Line 193  {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #2 - /.../quiltshop/product_info.php called at Line 44        {}      {}

-- Zeile in meiner shopping_cart - im Original 481

Code: PHP  [Auswählen]

       && $xtPrice->get_content_type_product($product['products_id']) == 'virtual'

Code: PHP  [Auswählen]

[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      WARNING found for URL: /Patchworkstoffe/Weihnachtsstoffe/Patchworkstoff-Makower-Wonderland-Haeuser::4702.html   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Undefined array key "products_id" in File: /.../quiltshop/includes/classes/shopping_cart.php on Line: 513       {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #0 - /.../quiltshop/templates/bootstrap4/source/boxes/shopping_cart.php called at Line 30     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #1 - /.../quiltshop/templates/bootstrap4/source/boxes.php called at Line 193  {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #2 - /.../quiltshop/product_info.php called at Line 44        {}      {}

-- Zeile in meiner shopping_cart - im Original 487

Code: PHP  [Auswählen]

    $products_price = $xtPrice->xtcGetPrice($product['products_id'], false, $qty, $product['products_tax_class_id'], $product['products_price']);

Code: PHP  [Auswählen]

[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      WARNING found for URL: /Patchworkstoffe/Weihnachtsstoffe/Patchworkstoff-Makower-Wonderland-Haeuser::4702.html   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Undefined array key "products_price" in File: /.../quiltshop/includes/classes/shopping_cart.php on Line: 513    {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #0 - /.../quiltshop/templates/bootstrap4/source/boxes/shopping_cart.php called at Line 30     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #1 - /.../quiltshop/templates/bootstrap4/source/boxes.php called at Line 193  {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #2 - /.../quiltshop/product_info.php called at Line 44        {}      {}

-- Zeile in meiner shopping_cart - im Original 487

Code: PHP  [Auswählen]

 $products_price = $xtPrice->xtcGetPrice($product['products_id'], false, $qty, $product['products_tax_class_id'], $product['products_price']);

Code: PHP  [Auswählen]

[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      WARNING found for URL: /Patchworkstoffe/Weihnachtsstoffe/Patchworkstoff-Makower-Wonderland-Haeuser::4702.html   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Undefined array key "products_weight" in File: /.../quiltshop/includes/classes/shopping_cart.php on Line: 526   {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #0 - /.../quiltshop/templates/bootstrap4/source/boxes/shopping_cart.php called at Line 30     {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #1 - /.../quiltshop/templates/bootstrap4/source/boxes.php called at Line 193  {}      {}
[2022-12-06 06:57:32]   [warning]       [modified]      [pid:9673]      Backtrace #2 - /.../quiltshop/product_info.php called at Line 44        {}      {}

-- Zeile in meiner shopping_cart - im Original 497

Code: PHP  [Auswählen]

 $this->weight += ($qty * $product['products_weight']);

und so geht es weiter mit "products_id"  "products_price".

Bin ich eigentlich der Einzige mit diesen warnings?
Gruß Peter

[EDIT Tomcraft 09.12.2022: Beitrag formatiert.]

[Karl1]

Hallo Peter,
vorweg, ich konnte deine warnings nachstellen.

Betreffend des älteren Posts mit shop_content.php - hier kannst du den Code in Zeile 116 so ändern:

Code: PHP  [Auswählen]

if (isset($main_content)) $smarty->assign('main_content', $main_content);

Jetzt aber zu dem letzten Post betreffend der shopping_cart.php.
Diese Warnungen treten nur auf, wenn beim Aufruf des Warenkorbes keine "products_id" vorhanden ist.
Es stellt sich die Frage, wie kann im Warenkorb die product_id fehlen?
Dies ist der Fall, wenn ein Produkt gelöscht wird und gleichzeitig ein angemeldeter Kunde das Produkt in seinem Warenkorb hat.
Aktualisiert dieser den Warenkorb wird die Warnung in die Logs geschrieben.

Hat ein abgemeldeter Kunde einen Artikel in seinem Warenkorb, der zwischenzeitig gelöscht wurde, wird dieser Artikel nicht mehr im Warenkorb angezeigt. Es erscheinen keine Einträge in den Logs.

Ich nehme an, dass man die Warnungen umgehen könnte, wenn man in der "class shoppingCart" das vorhandensein der product_id abfragt.

Gruß Karl

[Timm]

@Karl
Moin

Kannst du das in den Demoshops nachstellen? Ich konnte das gerade nicht. Laptop und Telefon haben aber auch gerade die gleiche IP.

Wäre ja schlimm, wenn Artikel aus dem Warenkorb eines abgemeldeten Kunden rausfliegen, nur weil jemand anderes den Artikel löscht. Oder hab ich dich da falsch verstanden.

Peter hatte mir die Meldungen schonmal geschickt im Oktober, da wir beide dezimale Mengen haben und er dachte, dass es daran liegt. Da habe ich geschrieben, dass das scheinbar gefixed wurde. Ich hatte die Fehlermeldungen selber auch und hatte sie dann wohl auch im Demoshop gesehen und irgendwann nicht mehr. Deshalb meine Antwort an ihn. Ich war die letzte Zeit selbst sehr beschäftigt mit meinem Shop und Modulupdates, deshalb kann ich mich nicht mehr genau dran erinnern.

@Torsten
Kennst du die passenden changelogs, sofern es welche gibt?

[Viol]

Ich habe das so verstanden, dass der Shopbetreiber einen Artikel gelöscht hat, nachdem ein Kunde den im Warenkorb hatte. Meldet der sich wieder an, wird ihm der verlassene Warenkorb mit einem Artikel angezeigt, den es aber nicht mehr gibt.

[Timm]

Macht Sinn 

[peter111]

@Karl,
ich habe die shop_content.php geändert und werde beobachten, ob jetzt Ruhe ist.

Ich habe die Tabelle  customers_basket geprüft. Dort ist die product_id  überall vorhanden.
Danke für die Hilfe
Peter

[Karl1]

Hallo Timm,
ich konnte die Warnungen auch im Dev-Shop nachstellen, steht jetzt in den Logs.

Scheinbar wurde ich falsch verstanden, ich versuche es nochmal.

- Admin ist im Adminbereich angemeldet.
- Kunde ist zeitgleich im Shop unterwegs und legt sich Artikel 1 in den Warenkorb.
- Admin löscht im Adminbereich Artikel 1.
- Kunde schaut sich den Warenkorb an.

Gruß Karl

[noRiddle (revilonetz)]

@peter111
Ich würde dringend davon abraten wegen Warnings im Core-Code Änderungen zu machen.
Ich verstehe auch nicht mit welcher Leichtfertigkeit das immer wieder getan wird,
nachdem sich die Entwickler die Mühe mit den Hooks und den Klassenerweiterungen gemacht haben.
Ausnahme sollte nur sein wenn man eine Erweiterung benötigt die nicht update-sicher möglich ist (was im Einzelfall noch zu prüfen wäre).

Da alle Warnings hier, bzw. im Bug-Tracker, gemeldet werden werden sie in der nächsten Version behoben sein.
Warum also im Code rumfummeln und beim nächsten Update dann wieder einen Riesenaufwand haben ?

Gruß,
noRiddle

[kotzbrocken]

Moin,

leider hab ich momentan auch immer wieder Warnungen in der "mod_warning_admin"-Logdatei.

Code: PHP  [Auswählen]

[2022-12-09 15:49:35]   [warning]       [modified]      [pid:20828]     Undefined array key "parameters" in File: /shop/admin/includes/extra_menu.php on Line: 97       {}      {}
[2022-12-09 15:49:35]   [warning]       [modified]      [pid:20828]     Backtrace #0 - /shop/admin/includes/column_left.php called at Line 49   {}      {}
[2022-12-09 15:49:35]   [warning]       [modified]      [pid:20828]     Backtrace #1 - /shop/admin/includes/column_left.php called at Line 219  {}      {}
[2022-12-09 15:49:35]   [warning]       [modified]      [pid:20828]     Backtrace #2 - /shop/admin/includes/header.php called at Line 237       {}      {}
[2022-12-09 15:49:35]   [warning]       [modified]      [pid:20828]     Backtrace #3 - /shop/admin/start.php called at Line 147 {}      {}
[2022-12-09 15:49:40]   [warning]       [modified]      [pid:20828]     WARNING found for URL: /shop/admin/logs.php     {}      {}

Kann mir da bitte jemand auf die Sprünge helfen, was das (Undefined array key "parameters") genau bedeutet?

Beste Grüße
Michou