id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc	blockedby	blocking
1698	warning beim Kauf aus Kundenkonto, Attribute ohne IDs	info@…	somebody	"Hi,

vielleicht habt ihr schon eine Lösung vorliegen. 
Folgendes Szenario:
Ein alter Shop wurde auf 2.0.x aktualisiert.
In der damaligen Shopversion wurden noch keine IDs zu Attributen einer Bestellung gespeichert.
Nutzen Kunden nun die Funktion, Bestellungen aus dem Kundenkonto in den Warenkorb zu legen, gibt es eine PHP-Warnung:

{{{
in_array() expects parameter 2 to be array, null given in File [...] includes/classes/shopping_cart.php on Line: 969
}}}

Ursache: da die Option-IDs nicht gespeichert wurden, ist diese mit 0 vorbelegt. Ein Attributwert wird nun in einem Array gesucht, dass nicht existiert, weil $option 0 ist

{{{
if (!in_array($value,$products_attributes_array[$pID][(int)$option])) {}
}}}

Die Bestellung wird dann nicht oder nicht vollständig in den Warenkorb gelegt.

Wie können wir das bestmöglich korrigieren?

Mein Ansatz wäre dieser:
1.) Prüfen, ob es überhaupt ein Array gibt:

includes/classes/shopping_cart.php
{{{
function validate_attributes($products_id, $attributes, $flag = '') {
    static $products_attributes_array;
    if (!is_array($products_attributes_array)) {
      $products_attributes_array = array();
    }

    $check = true;
    if (is_array($attributes) && count($attributes)) {
      $pID = (int)$products_id;
      if (!isset($products_attributes_array[$pID])) {
        $products_attributes_array[$pID] = $this->create_products_attributes_array($pID);
      }
if(!array_key_exists((int)$option, $products_attributes_array[$pID])) {
  return false;
}
      foreach($attributes as $option => $value) {
        if (!in_array($value,$products_attributes_array[$pID][(int)$option])) {
          $check = false;
          break;
        }
      }
    }
    return $check;
  }
}}}

2.) Um das Einkaufserlebnis hoch zu halten in der cart_action.php versuchen die Attribut-IDs zu ermitteln, wenn diese noch mit ""0"" in der Datenbank stehen.

Gibt es Einwände?

Viele Grüße
Andreas
"	Bug/Fehler	closed	normal	modified-shop-2.0.5.0	Shop	trunk	fixed