Fehler beim manuellen Löschen von Artikeln in der Bestellung

[floh]

Hi,

habe eben folgendes Scenario erlebt:

Kunde bestellt mehrere Positionen (Artikel). Ein Artikel wurde inzwischen vom Vorlieferanten ohne unseres Wissens aus dem Sortiment genommen.

1. Als Admin angemeldet,
2. Im Adminbereich auf "Bestellungen" gegangen
3. Markierte Bestellung auf "Bearbeiten" geklickt
4. Bestellung geht auf und oben Links wieder auf "Bearbeiten" gegangen
5. Im folgenden Auswahlmenü: "Artikel und Artikeloptionen bearbeiten und einfügen" mit Button "Bearbeiten" gewählt
6. Alle bestellten Positionen werden sauber aufgelistet. Geht ich nun auf den gewünschten Artikel und möchte ihn löschen, bekomme ich den SQL-Fehler:

Code: [Auswählen]

1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'WHERE products_id = 0' at line 1

UPDATE products SET products_quantity = products_quantity + WHERE products_id = 0

[XT SQL Error]Der Artikel wird nicht beim ersten Mal aus der Bestellung gelöscht. Lösche ich hingegen den kompletten Auftrag, geht das ohne zu mucken...  

Achja, bevor ich es noch vergesse: modified eCommerce Shopsoftware 1.05, php-Version: 5.2.0.8, mysql-Version: 5.0.32

Gruss

floh



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

[franky_n]

Hallo floh,

versuche mal folgendes.

In admin/orders_edit.php in Z.415 das ändern:

Code: PHP  [Auswählen]

    xtc_db_query("UPDATE ".TABLE_PRODUCTS." SET products_quantity = products_quantity + ".xtc_db_input($_POST['del_qty'])." WHERE products_id = " . (int)$_POST['del_pID']);

in das:

Code: PHP  [Auswählen]

    xtc_db_query("UPDATE ".TABLE_PRODUCTS." SET products_quantity = products_quantity + ".xtc_db_input($_POST['del_qty'])." WHERE products_id = '" . (int)$_POST['del_pID'] . "'");

Viele Grüße

Franky

[franky_n]

Hallo floh,

noch was.
In der admin/orders_edit_products.php nach Z.68:

Code: PHP  [Auswählen]

echo xtc_draw_hidden_field('opID', $order->products[$i]['opid']);

das einfügen:

Code: PHP  [Auswählen]

echo xtc_draw_hidden_field('del_pID', $order->products[$i]['id']);

Viele Grüße

Franky

[Tomcraft]

Oder einfach die funktionierenden Dateien aus dem Trunk nehmen.

Grüße

Torsten

[floh]

Torsten Du Scherzkeks..... Ich hatte an diesen Dateien noch nie und rein gar nichts geändert. Es sind noch die Original-Dateien aus dem Downloadpacket der 1.05..... Wollte dass nur mal als Bug melden.

Gruß

floh

[web28]

[...] Wollte dass nur mal als Bug melden.
[...]

Hallo floh,

das ist tatsächlich ein Bug, danke für den Hinweis. Wurde aber bereits für die neue Version gefixt.

Kann im Trunk Demoshop gerne getestet werden.

Gruss Web28

[Tomcraft]

Und genau deshalb meinte ich doch auch, dass du die Dateien aus dem Trunk nehmen sollst, denn dort ist der Fehler behoben.

War also kein Scherz.

Grüße

Torsten

[tinchenkatrinchen]

im trunk orders_edit_products.php Trunk fehlt das hidden field

Code: PHP  [Auswählen]

echo xtc_draw_hidden_field('old_qty', $order->products[$i]['qty']);

dadurch wird der artikelbestand falsch oder gar nicht geändert, wenn man die stückzahl ändert in der bestellung.

[Tomcraft]

Danke für den Hinweis, wir haben das in r1825 korrigiert.

Grüße

Torsten