GX-Entwicklung

eine reihe sehr unwahrscheinlicher ereignisse oder aber auch eine einfache fehlkonfiguration (zb nach daten uebernahme) kann dazu fuehren das keine neuen rechnungsnummern mehr vergeben werden (verwendung von pdf rechnungen).

szenario 1 (sehr unwahrscheinlich)
in gm_id_starts.php wird die methode GMOrderFormat.set_next_id benutzt um die naechste rechnungsnummer zu setzen. in der methode set_next_id wird auch geprueft ob die id noch vergeben werden kann und in gm_id_starts wird der rueckgabewert entsprechend behandelt. Wenn der shop waehrend nummernkreise / rechnungs nummern format bearbeitet werden nicht offline genommen wird kann es zu einer race condition kommen insofern gleichzeitig mit dem submit in gm_id_starts.php eine neue rechnung erstellt wird. ablauf:
u1: submit gm_id_starts.php
u1: erfolgreiche pruefung ($next_id > $this->get_act_id($type)) in gm_id_starts.php
u2: eine neue rechnungsnummer wird vergeben siehe gm_pdf_order.php
u2: rechnungsnummer wird in tab orders geschrieben
u2: wert des keys GM_NEXT_INVOICE_ID wird aktualisiert
u1: wert des keys GM_NEXT_INVOICE_ID wird aktualisiert (veraltet)

alternativ szenario: daten uebernahme wert des keys GM_NEXT_INVOICE_ID wird nicht mituebernommen.

ab diesem moment ist der wert in der datenbank fuer GM_NEXT_INVOICE_ID nicht mehr "gueltig (zu nieder)". Bei der weiteren generierung von rechnungen wird nun immer die gleiche rechnungsnummer vergeben siehe unten, da immer der aktuelle wert fuer GM_NEXT_INVOICE_ID aus der datenbank bezogen wird, als rechnungsnummer eingesetzt wird und anschlieszend versucht wird den wert fuer GM_NEXT_INVOICE_ID in der datenbank zu aktualisieren ($gmFormat->set_next_id('GM_NEXT_INVOICE_ID', $next_id + 1);). Sollte nun der urspruengliche wert fuer GM_NEXT_INVOICE_ID bereits ungueltig (zu nieder) gewesen sein so ist der um eins erhoehte ebenfalls bereits vergeben, dementsprechend wird der wert fuer GM_NEXT_INVOICE_ID in der datenbank nicht aktualisiert, allerdings wird der rueckgabewert der methode ignoriert und folglich bleibt dies dem user / shop admin verborgen.

Wuerde etwas dagegeben sprechen auf die spalte gm_orders_id einen unique index zu setzen um in zukunft auf nummer sicher zu gehen? Soweit ich den code verstehe ist es nicht angedacht das rechnungsnummern jemals wieder verwendet werden selbst wenn sich das format der rechnungsnummer (gm_order_code) aendert.

gm_pdf_order.php

// -> set id, code only in 'orders.php'
if(empty($_GET['preview']))
{

$gmFormat->update_next_id('GM_NEXT_INVOICE_ID', $next_id, $_GET['oID']);
$gmFormat->update_next_code('GM_NEXT_INVOICE_ID', $gm_orders_code, $_GET['oID']);
$gmFormat->set_next_id('GM_NEXT_INVOICE_ID', $next_id + 1);
}
$order_check['gm_orders_code'] = $gm_orders_code;