Antwort #22 am: 18. Juli 2018, 14:26:33
Hallo und guten Tag,
sehr schön, dass jetzt wünsche zu Hook-Points offiziell empfangen werden.
Um die Backup-Update-Routine für System-Module zu erweitern wären an dieser Stelle Hook-Points durchaus sinnvoll. Nicht jedes Modul kann sich zur Konfiguration auf die Konfigurations-Konstanten beschränken und wenn ein Modul zusätzliche Tabellen enthält wäre es schon wenn diese beim Modul-Backup und -update gleich mit gesichert bzw. zurück gesetzt werden könnten.
/admin/modules.php ab Zeile 125:
} elseif ($action == 'backupconfirm') { // save values xtc_backup_configuration
($module->keys()); if (method_exists($module, 'backup')) $module->backup(); $messageStack->add_session(MODULE_BACKUP_CONFIRM
, 'success'); } elseif ($action == 'restoreconfirm') { // reset backup values xtc_restore_configuration
($module->keys()); if (method_exists($module, 'restore')) $module->restore(); $messageStack->add_session(MODULE_RESTORE_CONFIRM
, 'success'); } elseif ($action == 'custom') { [ ... ]
Des Weiteren hätte ich gerne auch noch ein auto_include in der admin/includes/modules/products_images.php, um die HTML-Tabelle um weitere Einträge (die ja sonst gut über admin/includes/extra/modules/add_db_fields integriebar sind) für Produktbilder updatesicher zu erweitern.
[ ... ]
Das möchten wir sehr gerne unterstützen, auch für Kategorie-Bilder:
/admin/includes/modules/new_category.php ab Zeile 212:
<tr>
<td class="dataTableConfig col-left"><?php echo TEXT_DELETE; ?></td>
<td class="dataTableConfig col-middle"><?php echo xtc_draw_checkbox_field('del_cat_pic', 'yes'); ?></td>
</tr>
<?php foreach (auto_include(DIR_FS_ADMIN . 'includes/extra/modules/new_category/form_image/') as $file) require ($file); ?>
/admin/includes/modules/products_images.php ab Zeile 38:
<tr>
<td class="dataTableConfig col-left"><?php echo TEXT_DELETE; ?></td>
<td class="dataTableConfig col-middle"><?php echo xtc_draw_checkbox_field('del_pic', $pInfo->products_image); ?></td>
</tr>
<?php foreach (auto_include(DIR_FS_ADMIN . 'includes/extra/modules/products_images/form_first/') as $file) require ($file); ?>
und ab Zeile 64:
<tr>
<td class="dataTableConfig col-left">
<?php echo TEXT_DELETE
; ?></td>
<td class="dataTableConfig col-middle">
<?php echo xtc_draw_checkbox_field
('del_mo_pic[]', (isset($mo_images[$i]['image_name']) ?
$mo_images[$i]['image_name'] : '')); ?></td>
</tr>
<?php foreach (auto_include
(DIR_FS_ADMIN
. 'includes/extra/modules/products_images/form_more/') as $file) require ($file); ?>Weiter gibt es in der Klasse
shoppingCart eine funktion
cleanup() die uns schon mehr als einmal heftig geärgert hat. Hier wöre es wünschenswert über die Klassenerweiterung (oder alternativ Hook-Points) die Möglichkeit zu haben in die Abarbeitung und Entscheidung wann was und welchen Umständen gelöscht wird einzugreifen.
Sicher gibt es noch viele weitere Hook-Points die sinnvoll zu setzen wären, doch das soll fürs erste reichen.