id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
733	Fehler & Frage bzgl. r9137 (Updatekompatibilität der Module)	Torsten Riemer	Ronald Parcinski	"Das Changeset r9137 führt dazu, dass bereits beim Aufruf der Seite ""Module"" -> ""Zahlungsoptionen"" die MySQL-Inserts durchgeführt werden und diese werden doppelt ausgeführt, siehe Screenshot.

Generell finde ich die Idee aber super, nur sollte um die Abfrage herum noch zusätzlich abgefragt werden, ob das Modul überhaupt installiert ist.
Also am Beispiel von ""/includes/modules/payment/cod.php"" anstatt:

{{{
    //update compatibility
    if (!defined('MODULE_PAYMENT_COD_LIMIT_ALLOWED')) {
      xtc_db_query(""insert into "".TABLE_CONFIGURATION."" ( configuration_key, configuration_value,  configuration_group_id, sort_order, date_added) values ('MODULE_PAYMENT_COD_LIMIT_ALLOWED', '600', '6', '3', now())"");
    }
}}}

vielleicht lieber:

{{{
    //update compatibility
    if (MODULE_PAYMENT_COD_STATUS == 'True' && !defined('MODULE_PAYMENT_COD_LIMIT_ALLOWED')) {
      xtc_db_query(""insert into "".TABLE_CONFIGURATION."" ( configuration_key, configuration_value,  configuration_group_id, sort_order, date_added) values ('MODULE_PAYMENT_COD_LIMIT_ALLOWED', '600', '6', '3', now())"");
    }
}}}

Zusätzlich würde ich dann noch die folgenden Module ebenfalls auf diese Art erweitern, damit die Schritte zum deinstallieren und neu installieren der Module aus der Updateanleitung entfallen:

- ""/admin/includes/modules/system/janolaw.php""
- ""/includes/modules/order_total/ot_shipping.php""
- ""/includes/modules/payment/banktransfer.php""
- ""/includes/modules/payment/paypal.php""
- ""/includes/modules/payment/paypalexpress.php""
- ""/includes/modules/payment/moneybookers_*.php""
- ""/includes/modules/payment/pn_sofortueberweisung.php"" (müsste in Shopversion 2.00 wieder eingefügt werden, damit die Datenbank-Erweiterungen für das Update von 1.05 SP1d nach 1.06 ausgeführt werden können.)"	Bug/Fehler	closed	hoch	modified-shop-2.0.0.0	Module	2.0.0.0	fixed