Ungereimtheiten bei "Auftragsbestätigung senden"

[Torsten Riemer]

DokuMan hatte in r1144 die Bestellbearbeitung derart erweitert, dass sowohl die Bestellung an den Kunden, als auch die Bestellung an den Admin erneut versendet werden konnte.
In r1177 hat web28 dieses Changeset wohl nicht komplett revoked, sondern Teile davon übernommen, die es so gar nicht mehr gibt.

Mehrere Anmerkungen:
1.) Der folgende Code ist überflüssig:

define('SEND_BY_ADMIN_PATH', DIR_FS_CATALOG);

2.) Der Button für "BUTTON_ORDER_CONFIRMATION" hat noch die Parameter aus r1144 dran gehängt (&sta=0&stc=1), also:

<a class="button" href="<?php echo xtc_href_link(FILENAME_ORDERS, xtc_get_all_get_params(array ('oID', 'action')).'oID='.$_GET['oID'].'&action=send&sta=0&stc=1&site=1'); ?>"><?php echo BUTTON_ORDER_CONFIRMATION; ?></a>

3.) Bitte die Datei nochmal genau anschauen, dass da nicht noch mehr Überbleibsel aus r1144 enthalten sind, die unnötig sind.
4.) Ich würde es sehr begrüßen, wenn wir die Funktionalität wieder dahingehend erweitern, wie es DokuMan in r1144 vorgesehen hatte, dass auch die Bestellbestätigung an den Admin erneut versendet werden kann!

[Ronald Parcinski]

Beide Sachen stören nicht, deshalb ist das nicht aufgefallen. Können also entfernt werden.
Die beiden Parameter beim Buttonlink wurden schlichtweg übersehen.

Ansonsten hat die jetzige Version nichts mehr mit r1144 zu tun, das wurde alles mehrmals überarbeitet.

Der Shopadmin bekommt diese E-Mail immer, da der Shopadmin grundsätzlich immer eine Kopie bekommen sollte.

[Torsten Riemer]

In 8927:

Partly fix #683

[Torsten Riemer]

Die überflüssigen Parameter und die ungenutzte Konstante habe ich nun in r8927 entfernt.

Ich halte es dennoch für sinnvoll eine Möglichkeit zu schaffen, dass nur der Admin nochmal eine Mail bekommt und nicht zusätzlich der Kunde auch noch!

[Torsten Riemer]

Ich sehe hier doch keinen Grund, wieso nur der Admin sich selber nochmal eine Bestätigung schicken können sollte. Wenn jemandem ein schlüssiges Szenario einfällt, so möge er dieses Ticket gerne wieder öffnen.

Das Ticket ist damit aus meiner Sicht in r8927 hinreichend gefixt.