Fehler in der admin/categories.php beim Erzeugen der AdminCategoriesExtenderComponent

Hallo zusammen,

hat schon mal jemand die AdminCategoriesExtenderComponent genutzt?

Auch in der aktuellen 2.2.0.0 sieht der Code für die Erzeugung und Initialisierung der Component in der admin/categories.php so aus:

PHP:

$coo_admin_categories_extender_component = MainFactory::create_object('AdminCategoriesExtenderComponent');
$coo_admin_categories_extender_component->set_data('GET', $_GET);
$coo_admin_categories_extender_component->set_data('POST', $_POST);
if(empty($t_products_id) != false)
{
    $coo_admin_categories_extender_component->set_data('products_id', $t_products_id);
}
if(empty($t_categories_id) != false)
{
    $coo_admin_categories_extender_component->set_data('categories_id', $t_categories_id);
}
$coo_admin_categories_extender_component->proceed();


Ich habe mich in der SampleExtender.php schon gewundert, warum dann so auf die product_id zugegriffen wird:

PHP:

// products_id is always available (insert & update)
$t_products_id = $this->v_data_array['GET']['pID'];


Denn eigentlich würde ich den Wert in $this->v_data_array['products_id'] erwarten ...

Allerding bei genauerer Betrachtung:

PHP:

if ( empty($t_products_id) != false )
{
    $coo_admin_categories_extender_component->set_data('products_id', $t_products_id);
}


Bei doppelter Negierung neigt man leicht zu Fehlern:

Wenn in $t_products_id ein Wert drin steht, dann liefert die Funktion empty() den Wert FALSE, nur wenn die Variable leer ist, liefert die Funktion TRUE. Der Vergleich auf Ungleich FALSE führt somit dazu, dass niemals die set_dat() Methode aufgerufen wird ...

Somit macht der Umweg im SampleExtender wieder Sinn, weil die eigentlichen Werte nie gesetzt werden

Es wäre schön, wenn das im nächsten Update gefixt wird.

PHP:

if(empty($t_products_id) == false)
{
    $coo_admin_categories_extender_component->set_data('products_id', $t_products_id);
}
if(empty($t_categories_id) == false)
{
    $coo_admin_categories_extender_component->set_data('categories_id', $t_categories_id);
}


Ciao,
Mike