Fehler in create_account.php bzgl. Neukunden Gutschein?

[Torsten Riemer]

Bitte mal prüfen: ​BUG in v2.0.2.2 -> create_account.php ?

Es gibt wohl einen SQL-Fehler:

1062 - Duplicate entry '0' FOR KEY 'idx_coupon_id'

INSERT INTO coupon_email_track (coupon_id, customer_id_sent, sent_firstname, emailed_to, date_sent) VALUES ('', '0', 'Admin', 'mail@example.com', now())

[Ronald Parcinski]

Fehler besteht seit 2.0.0.0

[Ronald Parcinski]

In 10698:

fix #1176

[Volker Strähle]

Sind das nicht 2 Fehler an gleicher Stelle?

1. $insert_id = xtc_db_insert_id(); statt $insert_id = xtc_db_insert_id($insert_query);

2. Ich sehe nicht, warum auf coupon_id in Tabelle coupon_email_track ein unique-index ist. Es gibt doch extra eine Spalte um unique_id um jeden Datensatz eindeutig zu identifizieren.

[Torsten Riemer]

Ich öffne das Ticket nochmal wegen der Frage von webald.

[Ronald Parcinski]

Mit dem Fix wurde das Problem gelöst

Spalte coupon_id ist ein unique-index weil eine Coupon-Id nur einmal vorkommen darf.

Ohne diesen unique-index wäre der Fehler gar nicht aufgefallen.

[Volker Strähle]

Dann ist die Spalte unique-id überflüssig. Zwei unabhängige und eindeutige Indizes sind unnötig.

Allerdings bin ich der Meinung, dass der unique-Index auf coupon_id vom Datenbank-Design her keinen Sinn macht. coupon_id wird in der Tabelle coupons festgelegt und ist dort eindeutig; coupon_email_track ist nur eine davon abhängige Tabelle.

In dieser Tabelle könnte auch das mehrfache Versenden eines Coupons dokumentiert werden (etwa weil der Coupon vom Admin erneut zugesandt werden mußte). Das geht derzeit aufgrund des Index nicht. Hierfür ware u. U. eine zusätzliche Spalte mit einem Flag notwendig, ob dieser Datensatz bei der Berechnung des Kundenguthabens berücksichtigt werden muss.

[Gerhard Waldemair]

Die coupon_id ist ein Autoincrement. Damit wird auch der Verweis auf die Sprache gemacht. Die coupon_code muss eindeutig sein und deshalb gibt es hier einen UNIQUE Index dazu.