[résolu] avertissement pour classes/product.php

[chantane]

Bonjour,

Dans classes/product.php, ligne 1952, il manque des parenthèses.
avant :

        $row['attribute_price'] = isset($row['id_product_attribute']) AND $row['id_product_attribute']? floatval(Product::getProductAttributePrice($row['id_product_attribute'])) : 0;

le résultat est un booléen

après :

        $row['attribute_price'] = (isset($row['id_product_attribute']) AND $row['id_product_attribute']) ? floatval(Product::getProductAttributePrice($row['id_product_attribute'])) : 0;

le résultat est numérique, ce qui est attendu.

IL serait profitable d'utiliser des variables locales car lors de la recherche des informations tel que dans la fonction getProductProperties, le même appel de fonction est effectué plusieurs fois sur quelques lignes, ce qui nuit aux performances et obscurcit le code.

[Atch]

Bonjour Reinette,

Si bug il y a, l'idéal est de le poster dans le bug tracker, une équipe les gère quotidiennement.

http://www.prestashop.com/bug_tracker/report/bug/

Merci pour cette info

V++

Atch

[chantane]

Lorsque je regarde le code de prestashop, j'ai parfois l'envie de tout réécrire.

par exemple dans classes/product.php, dans la fonction getProductProperties,
on trouve sur quelques lignes, trois appels à Product::getPriceStatic, et dans cette dernière fonction une série de tests sur les arguments de l'appel.
C'est plutôt une usine à gaz, car il suffisait d'un seul appel pour recalculer tous les prix, avec ou sans taxe, avec ou sans réduction. Au lieu de cela le programmeur, payé sans doute à la ligne, a compliqué le code et sa compréhension et sa maintenance. De plus, Il ignore totalement l'utilisation de variables de stockage de résultat temporaire et refait plusieurs fois les mêmes tests ou calculs.

Le résultat est la répétition d'appels de fonctions inutiles, un code lourd et illisible et le plus important une baisse importante de la performance.

S'il veulent auditer le code, je suis là et il y a du travail!!!

[Fabrice]

à mon avis, ils devraient t'embaucher chez PS ;-)

(et je suis sérieux...)

[chantane]

remarque effectuée sur le BT.

mise à jour effectuée dans le SVN par la team prestashop.