AUP dans form.php ET invoice.php

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66412

Bonjour

dans /administrator/templates/TPL /html/com_hikashop/order/form.php j'ai toujours les alphauserpoint avec le montant de la reduction
par contre dans /administrator/templates/TPL/html/com_hikashop/order/invoice.php leur apparition est incoherente, des fois oui des fois non sans trouver pourquoi

et ca depuis la 1.5.9 et 1.6

une idee pourquoi les 2 fichiers ne sont pas traités de la meme maniere ?

merci

Please Log in or Create an account to join the conversation.

  • Posts: 81598
  • Thank you received: 13082
  • MODERATOR
11 years 7 months ago #66607

La seule différence, c'est le code de limitation d'affichage des additionals en fonction de l'option de config order_additional_hide :
$exclude_additionnal = explode(',', $this->config->get('order_additional_hide', ''));
foreach($this->order->additional as $additional) {
if(in_array($additional->order_product_name, $exclude_additionnal)) continue;

C'est présent sur l'invoice mais pas sur le form.php.
Cette option a été ajouté par jérome pour vous.
Je suppose que c'est à cause d'elle que par moment la réduction n'est pas affichée sur l'invoice vu que c'est justement le but de cette option.

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66608

et la solution pour tout avoir pareil partout ?
vu que le code de Jerome est pour tout le monde maintenant ça serait bien d'avoir tout pareil

merci

Last edit: 11 years 7 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 81598
  • Thank you received: 13082
  • MODERATOR
11 years 7 months ago #66612

il suffit de remplacer:
foreach($this->order->additional as $additional) {

par:

$exclude_additionnal = explode(',', $this->config->get('order_additional_hide', ''));
foreach($this->order->additional as $additional) {
if(in_array($additional->order_product_name, $exclude_additionnal)) continue;

dans form.php

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66613

ok j'essaye

ca sera dans une future version en natif ?

merci

Please Log in or Create an account to join the conversation.

  • Posts: 81598
  • Thank you received: 13082
  • MODERATOR
11 years 7 months ago #66614

oui

The following user(s) said Thank You: erickb

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66615

genial

merci

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66705

Je reviens là dessus c'est dans invoice.php que ça ne marche pas, le form.php marche tres bien

form.php

if(!empty($this->order->additional)) {
foreach($this->order->additional as $additional) { ?>
<tr>

invoice.php
if(!empty($this->order->additional)) {
$exclude_additionnal = explode(',', $this->config->get('order_additional_hide', ''));
foreach($this->order->additional as $additional) {
if(in_array($additional->order_product_name, $exclude_additionnal)) continue;
?>
<tr>

et rien à faire je ne récupère pas les AUP dans invoice.php alors qu'elles apparaissent dans form.php meme en mettant dans invoice.php le code de form.php

Last edit: 11 years 7 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 81598
  • Thank you received: 13082
  • MODERATOR
11 years 7 months ago #66973

Je ne vois pas.
J'ai vérifié le code mais les additionals sont chargés de la même façon (via un loadFullOrder) pour le form et l'invoice.
La seule différence est celle dont je vous ai parlé sur mon premier message avec l'option order_additional_hide.

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
11 years 7 months ago #66975

ceque je ne comprends pas c'est que tu dis

il suffit de remplacer: foreach($this->order->additional as $additional) {

par:

$exclude_additionnal = explode(',', $this->config->get('order_additional_hide', ''));
foreach($this->order->additional as $additional) {
if(in_array($additional->order_product_name, $exclude_additionnal)) continue;

dans form.php


c'est dans invoice que ça ne marche pas , donc je ne comprend pas pourquoi remplacer du code dans form

Please Log in or Create an account to join the conversation.

  • Posts: 81598
  • Thank you received: 13082
  • MODERATOR
11 years 7 months ago #66984

Le but de ce code, dans invoice, c'est de cacher les additionals en fonction du nom.
Quelque chose qui a été demandé à jérome lors du custom AUP je crois.

Ce code n'était pas présent dans form.php. Du coup, cela affiche tout le temps tout les additionals peu importe ce qu'il y a dans l'option order_additional_hide.
Donc forcément, vous voyez toujours les additionals dans le form.

Si c'est ce que vous voulez pour invoice.php aussi, alors il suffit de supprimer le check dans invoice.php et du coup les additionals seront toujours affichés dans invoice.php

Please Log in or Create an account to join the conversation.

  • Posts: 26017
  • Thank you received: 4004
  • MODERATOR
10 years 8 months ago #118682

Bonjour,

Pour information, l'option "order_additional_hide" a été bien ajouté dans HikaShop et gère les différentes vues (comme indiqué par Nicolas).
Il s'agit d'une option dite "cachée" d'HikaShop ou il faut passer via la base de données pour la configurer.

Et comme indiqué précédement, il faut entrer en valeurs les noms des champs à cacher, séparés par des virgules.
Pour connaitre facilement le nom de la clef, vous pouvez utiliser le code HTML afin de la "déduire". Par exemple

<tr id="hikashop_checkout_cart_additional_userpoints_points_line">
Nous indique que la clé est "userpoints_points", la partie "hikashop_checkout_cart_additional_" étant spécifique pour indiquer qu'il s'agit d'un additional du cart du checkout d'HikaShop et le "_line" indiquant qu'il s'agit de la ligne entière (le tr).

Cordialement,


Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.
Last edit: 10 years 8 months ago by Jerome.

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
10 years 8 months ago #118683

effectivement j'avais oublié cette option désolé

mais j'ai bien





et points utilise s"affichent

Attachments:
Last edit: 10 years 8 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 26017
  • Thank you received: 4004
  • MODERATOR
10 years 8 months ago #118685

Merci d'utiliser cette valeur: USERPOINTS_USE_POINTS

Cordialement,


Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
10 years 8 months ago #118686

ca marche mais il etait pas evident de deduire que c'etait USERPOINTS_USE_POINTS

merci

comme on l'avait évoqué a un moment ce serait bien d'avoir tous ces champs cachés en plus dans l'admin d'hikashop a la suite sans explication dans un onglet special tout simple , un peu comme un about config de Firefox

Last edit: 10 years 8 months ago by erickb.

Please Log in or Create an account to join the conversation.

  • Posts: 26017
  • Thank you received: 4004
  • MODERATOR
10 years 8 months ago #118695

J'ai remarqué ensuite que le patch utilise en fait le "name" du additiional et non la clé.
Le name est utilisé pour la traduction il se retrouve donc assez facilement (surtout lorsqu'il n'est pas traduit).

La clé aurait été plus simple et surtout plus sure, puisque la clé est réellement unique alors que le nom peut etre utilisé par plusieur éléments.

Pour ce qui est des options cachés, nous y pensons et cela viendra en temps et en heure. Nous devons déjà améliorer l'interface en mode "simple" avant de proposer un mode "expert".

Cordialement,


Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.

Please Log in or Create an account to join the conversation.

  • Posts: 2607
  • Thank you received: 65
10 years 8 months ago #118699

en tout cas ça marche c'est le principal merci bien
sinon c'est vrai que ce serait mieux la cle que le name

oui je me doute qu'il y a plus urgent que cet onglet dans l'admin mais l'idée de ces champs caches qui n'alourdissent pas l'admin est bonne je trouve

c'est juste qu'un an plus tard j'avais complètement oublie cette option que j'avais moi même demande (ainsi que le no_css_header)

Last edit: 10 years 8 months ago by erickb.

Please Log in or Create an account to join the conversation.

Time to create page: 0.076 seconds
Powered by Kunena Forum