alain91

Prestashop 1.6.1.7 Cart Class public function updateAddressId($id_address, $id_address_new) { } 1/ This method change either address-delivery and/or address-invoice and I think it could have unexpected effets on those addresses. We would have better to have two methods one for delivery and the other for invoice 2/ The method updates address-delivery of cart-product without checking that we have PS_ALLOW_MULTISHIPPING. 3/ The method updates address-delivery of customization without checking this feature is activated as setNoMultishipping does.

Prestashop 1.6.1.7 public function setNoMultishipping() { .../... $sql = 'UPDATE `'._DB_PREFIX_.'cart_product` SET `id_address_delivery` = ( SELECT `id_address_delivery` FROM `'._DB_PREFIX_.'cart` WHERE `id_cart` = '.(int)$this->id.' AND `id_shop` = '.(int)$this->id_shop.' ) WHERE `id_cart` = '.(int)$this->id.' '.(Configuration::get('PS_ALLOW_MULTISHIPPING') ? ' AND `id_shop` = '.(int)$this->id_shop : ''); } 1/ I don't understand the relationship between MULTISHIPPING and MULTISHOP in the last statment. 2/ The Update should depend of the PS_ALLOW_MULTISHIPPING: if multishipping is allowed then id-adress-delivery have to be reset with the id-address-delivery of the cart but if not the id-adress-delivery of cart-product have to be reset to 0

Je n'ai pas trouvé de bouton/icone pour me déconnecter du forum. Comment fait-on pour se déconnecter ?

Prestashop v 1.6.1.x he second hook "actionProductListModifier" may have side effects because it can modify nbProducts and cat_products and some actions like pagaination have allready be made. It seems that this second hook can be delete becasue there is not really a great difference with the first one "actionProductListOverride" and it is best to limit the hooks. then I suggest the following refactoring : /** * Assigns product list template variables */ public function assignProductList() { $hook_executed = false; Hook::exec('actionProductListOverride', array( 'nbProducts' => &$this->nbProducts, 'catProducts' => &$this->cat_products, 'hookExecuted' => &$hook_executed, )); // The hook was not executed, standard working if (!$hook_executed) { $this->context->smarty->assign('categoryNameComplement', ''); $this->nbProducts = $this->category->getProducts(null, null, null, $this->orderBy, $this->orderWay, true); $this->cat_products = $this->category->getProducts($this->context->language->id, (int)$this->p, (int)$this->n, $this->orderBy, $this->orderWay); } $this->pagination((int)$this->nbProducts); // Pagination must be call after the last "getProducts" $this->addColorsToProductList($this->cat_products); foreach ($this->cat_products as &$product) { if (isset($product['id_product_attribute']) && $product['id_product_attribute'] && isset($product['product_attribute_minimal_quantity'])) { $product['minimal_quantity'] = $product['product_attribute_minimal_quantity']; } } $this->context->smarty->assign('nb_products', $this->nbProducts); }

De quelle fonction packs parles-tu ?

J'ai créé 2 groupes de coloris. Hors dans l'affichage de la fiche produit, une palette apparait pour le premier groupe pas pour les deux. Existe-t-il une restriction d'utilisation de plusieurs groupes de coloris dans une fiche produit ? Edit: j'ai depuis trouvé qu'il y a une liste déroulante pour afficher une palette de couleur ou aucune. Mais pourquoi ne pas permettre de les afficher toutes ? proposition de New Feature.

Suite au Barcamp d'hier (fécilitations aux organisateurs), j'ai installé le module PrestaBoard et il se trouve que je n'arrive pas sous IE6 à sauvegarder les configurations d'accès à part pour toute la boutique (ni en utilisateur ni en groupe). Par ailleurs, la gestion du changement (utilisateur, groupe, tout le site) n'est pas intuitive. Sous IE6 toujours, il faut enregistrer pour faire changer la liste des accès. Par ailleurs, le bouton Enregistrer est dans un bloc différent et il n'est pas trivial qu'il agit aussi pour sauvegarder les paramètres des accès. Une petite amélioration ergonomique serait profitable à beaucoup. Sinon, j'ai pu faire des essais (en mode toute la boutique qui est la seule qui marche sous IE6) et le résultat est vraiment très intéressant. Félicitations à l'équipe et au développeur en charge de PrestaBoard.

OK j'ai mis des précisions dans mon post initial

Un patch dans admin/tags/adminModulePosition.php pour présenter une liste triée des hooks disponibles pour greffer un module. // modif alain debut private function _compare($a, $ { return strcmp($a['title'], $b['title']); } // modif alain fin public function displayForm() <-- LOOK NEAR THIS METHOD ;-) { global $currentIndex; ... existing code ... $modules = $instances; $hooks = Hook::getHooks(0); // modif alain debut usort($hooks, array($this,'_compare')); // modif alain fin ... End of code ...