Bug in billsafe_2.php

[noRiddle (revilonetz)]

Hallo Community, hallo modified-Team.
Im aktuellen Download-Paket befindet sich folgende Datei: /includes/modules/payment/billsafe_2.php
In dieser Datei wird in Zeile 526 folgendes INSERT in die Tabelle orders_status_history gemacht:

Code: PHP  [Auswählen]

xtc_db_query ('INSERT INTO orders_status_history (orders_status_history_id, orders_id, orders_status_id, date_added, customer_notified, comments) VALUES (NULL, "'.xtc_db_input($insert_id).'", "1", "'.date('Y-m-d H:i:s').'", "0", "'.xtc_db_input($comments).'")');

Da wird in orders_status_history_id als Wert NULL eingegeben.
Das Feld hat jedoch und ein auto_increment und folglich NOT NULL.
orders_status_history_id darf in der INSERT-Query nicht gefüllt werden.

Gruß,
noRiddle

*NACHTRAG*
Ticket wurde erstellt: Ticket #360

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

[Tomcraft]

Danke dir für's melden!

Grüße

Torsten

[Tomcraft]

Fixed in r7256.

Grüße

Torsten

[noRiddle (revilonetz)]

Ich würde außerdem noch vorschlagen zwei Indizes zu setzen, auf orders_id und orders_status_id. Der PRIMARY KEY auf orders_status_history_id kommt ja gar nicht zum tragen, da nach orders_status_history_id an keiner Stelle weder gesucht noch in der WHERE-Clause gefiltert wird.

Denke das wäre sinnvoll.

Gruß,
noRiddle