Fehler & Frage bzgl. r9137 (Updatekompatibilität der Module)

[Torsten Riemer]

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.)

[Torsten Riemer]

Tabelle configuration.png

[Ronald Parcinski]

In 9144:

partly fixed #733

[Gerhard Waldemair]

r9137 wurde revoked und dafür ein neuen Hinweissystem eingeführt, das fehlende Einträge für Module erkennt.