À partir de (from_price) [RÉSOLU]

[Olivier75009]

Bonjour,
J'ai remarqué que la classe Combination retournait $min_attribute sans faire la distinction d'un changement de prix (si toutes les valeurs price sont à 0 ou qu'elles sont les mêmes, il y a quand même un retour). Du fait, des produits déclinés ayants les mêmes prix (couleurs...) ont "à partir de" avant le prix.

[eolia]

Ok,
Pour le correctif (qui sera effectif dans la 1.6.2.21)
Modifier la classe Product.php lignes 5410 et suivantes par:

        $row['from_price'] = false;
        if(Configuration::get('PS_ATTRIBUTE_CATEGORY_FROM') && $id_product_attribute) {
            $info_attribute = Combination::getMinProductAttributePrice($row['id_product'], true);
            if($info_attribute['min'] != $info_attribute['max']) {
                $row['from_price'] = Tools::ps_round(
                    Product::getPriceStatic(
                        (int)$row['id_product'],
                        true,
                        (int)$info_attribute['id_product_attribute'],
                        6
                    ),
                    (int)Configuration::get('PS_PRICE_DISPLAY_PRECISION')
                );
            }
        }

Et la classe Combination.php, fonction getMinProductAttributePrice() par:

    public static function getMinProductAttributePrice($id_product, $with_price = true)
    {
        $min_attribute = Db::getInstance()->getRow('
            SELECT pa.`id_product_attribute`, pa.`price`,  MIN(pa.`price`) as `min`, MAX(pa.`price`) as `max`
            FROM `'._DB_PREFIX_.'product_attribute` pa
            '.Shop::addSqlAssociation('product_attribute', 'pa').'
            WHERE pa.`id_product` = '.(int)$id_product.'
            order by pa.`price` ASC
        ');

        if($min_attribute) {
            if($with_price) {
                return $min_attribute;
            }
            else {
                return (int)$min_attribute['id_product_attribute'];
            }
        }

        return 0;
    }

[Olivier75009]

Merci ! Ça fonctionne ! (enfin, l'affichage si prix différents). Par contre ça affiche le prix max, pas le min.

[Olivier75009]

Précision : dans les déclinaisons, il se peut (ici c'est le cas) que l'incidence sur le prix soit négative (soustraction). Se peut-il que le calcul soit fait sur la valeur absolue ?

[eolia]

Alors on va plutôt se baser sur une autre table.
Essayez ceci:

    public static function getMinProductAttributePrice($id_product, $with_price = true)
    {
        $min_attribute = Db::getInstance()->getRow('
            SELECT ai.`id_attribute`, ai.`price`, 
                MIN(ai.`price`) as `min`, MAX(ai.`price`) as `max`
            FROM `'._DB_PREFIX_.'attribute_impact` ai
            '.Shop::addSqlAssociation('attribute', 'ai').'
            WHERE ai.`id_product` = '.(int)$id_product.'
            order by ai.`price` ASC
        ');

        if($min_attribute) {
            if($with_price) {
                return $min_attribute;
            }
            else {
                return (int)$min_attribute['id_product_attribute'];
            }
        }

        return 0;
    }

[Olivier75009]

J'ai le bon prix, mais je n'ai plus la mention "À partir de" (la table attribute_impact est vide)

[Olivier75009]

les 8.40 d'impact négatif sur le prix sont bien dans la table product_attribute initialement invoquée dans la classe Combination

[eolia]

Ok, donc c'était bien la première table qui était la bonne (Attribute_impact n'est utilisée que dans des cas particuliers)
MySQL ne pouvant gérer correctement la combinaison de max/min avec le order by, voici la fonction à utiliser:

    public static function getMinProductAttributePrice($id_product, $with_price = true)
    {
        $attributes = Db::getInstance()->executeS('
            SELECT pa.`id_product_attribute`, pa.`price`
            FROM `'._DB_PREFIX_.'product_attribute` pa
            '.Shop::addSqlAssociation('product_attribute', 'pa').'
            WHERE pa.`id_product` = '.(int)$id_product.'
            ORDER BY pa.`price` ASC
        ');
        
        if($attributes) {
            $min_attribute = reset($attributes);
            $max = end($attributes);
            if($with_price) {
                $min_attribute['min'] = $min_attribute['price'];
                $min_attribute['max'] = $max['price'];
                return $min_attribute;
            }
            else {
                return (int)$min_attribute['id_product_attribute'];
            }
        }

        return 0;
    }

[Olivier75009]

On y est presque, le prix est toutefois affiché TTC sur ces produits-là, alors qu'il est bien HT sur les autres (lorsqu'affiché HT pour ce groupe client).

[eolia]

        $row['from_price'] = false;
        if(Configuration::get('PS_ATTRIBUTE_CATEGORY_FROM') && $id_product_attribute) {
            $info_attribute = Combination::getMinProductAttributePrice($row['id_product'], true);
            if($info_attribute['min'] != $info_attribute['max']) {
                $row['from_price'] = Tools::ps_round(
                    Product::getPriceStatic(
                        (int)$row['id_product'],
                        (self::$_taxCalculationMethod == PS_TAX_EXC ? false : true),
                        (int)$info_attribute['id_product_attribute'],
                        6
                    ),
                    (int)Configuration::get('PS_PRICE_DISPLAY_PRECISION')
                );
            }
        }

[Olivier75009]

Merci, tout est bon