Fehler in admin/products_attributes.php

[web28]

Gerade entdeckt:

Beim Löschen eines Artikelmerkmals (Optionsname) werden die zugehörigen Optionswerte nicht mitgelöscht, obwohl das eigentlich so vorgesehen ist!

Originalcode admin/products_attributes.php ca. Zeile 73:

Code: PHP  [Auswählen]

case 'delete_option':

    $del_options = xtc_db_query("select products_options_values_id from " . TABLE_PRODUCTS_OPTIONS_VALUES_TO_PRODUCTS_OPTIONS . " where products_options_id = '" . $_GET['option_id'] . "'");
    while($del_options_values = xtc_db_fetch_array($del_options)){
          xtc_db_query("delete from " . TABLE_PRODUCTS_OPTIONS_VALUES . " where products_options_values_id = '" . $_GET['option_id'] . "'");
         }

Bugfix:

Code: PHP  [Auswählen]

case 'delete_option':
                $del_options = xtc_db_query("select products_options_values_id from " . TABLE_PRODUCTS_OPTIONS_VALUES_TO_PRODUCTS_OPTIONS . " where products_options_id = '" . $_GET['option_id'] . "'");
                while($del_options_values = xtc_db_fetch_array($del_options)){
                        xtc_db_query("delete from " . TABLE_PRODUCTS_OPTIONS_VALUES . " where products_options_values_id = '" . $del_options_values['products_options_values_id'] . "'");
        }

Gruss Web28



Linkback: https://www.modified-shop.org/forum/index.php?topic=1863.0

[Tomcraft]

Fix kann ich nachvollziehen und funktioniert.

Danke für den Hinweis!

Grüße

Torsten

[DokuMan]

Soll das mit in den Trunk?

[web28]

Soll das mit in den Trunk?

Davon gehe ich doch mal stark aus, ist ja schließlich ein Bugfix.  

Gruss Web28

[Tomcraft]

Jepp, konnte nur gestern nach dem Update von Mac OS X von 10.5.x nach 10.6 nicht mehr mein SCPlugin bedienen. Das verwehrt seinen Dienst mit 10.6! *gg*

Danke für's Einspielen in den Trunk.

Grüße

Torsten

[DokuMan]

Wenn du deinen netten Podcast u.a. mit "Snow Leopard" hören willst:
http://www.z-pod.de/archives/322-Z!-Episode-143-Presto-Presto.html#extended

Bei 10.6 scheint mehr nicht mehr so einfach zu funktionieren.

[Tomcraft]

Argh... sag mir doch sowas nicht! Jetzt ist es installiert und MUSS laufen!