Calcul des règles paniers HT/TTC [RÉSOLU]

[Olivier75009]

pas ceux offerts (ils sont offerts uniquement si le montant a dépassé un certain montant). Mais lorsqu'il y a des fdp, simplement, le calcul de la tva ne s'appliquait pas dans le calcul

[Olivier75009]

Mais quoi qu'il en soit, sur la première image, il y a un calcul sur les frais de port. Donc ça a l'air de fonctionner pour les règles comme pour les fdp.

[Olivier75009]

Les fdp offerts sont à partir de X en fonction du transporteur dans les préférences transporteurs du BO

[Olivier75009]

Est-ce que par hasard il faudrait appliquer l'ajout de taxe fr dans les préférences du transporteur (Transport->transporteurs sur PhenixSuite, Livraison->transporteurs sur notre boutique) :

[eolia]

Ah ben si vous n'en avez pas mis aucune taxe n'est ajoutée au transport^^

[Olivier75009]

Auparavant elles s'ajoutaient pourtant. Se pourrait-il que la mise à jour ait eu à voir avec la modification de ce paramètre ?

[eolia]

Auparavant quand ? A moins d'un override Prestashop n'a jamais ajouté de taxe à un transporteur qui n'en a pas.

[Olivier75009]

A mon avis ça vient de chez nous, certains transporteurs ont bien les taxes FR, d'autres non... On s'est séparé d'un bras cassé en début d'année...

Du coup, on peut appliquer les corrections dans la foulée pour le calcul de la TVA sur les règles... ?

[Olivier75009]

Je viens de checker d'anciennes commandes (avant la migration sur PhenixSuite) et en effet la TVA sur les FDP n'a pas été calculée (de par le fait, comme on dit chez soi).
Il n'y a donc bien que la TVA sur les règles paniers à corriger.

[Olivier75009]

Sur une commande (payplug) j'ai ce message lorsque dans paiement je tente de rajouter le montant du remboursement (TVA non calculée sur la règle panier) :


C'est pour que la facture soit juste pour le client.

[Olivier75009]

Je viens de m'apercevoir qu'il n'y a qu'un groupe client dont la TVA n'est pas appliquée sur les règles paniers : les professionnels dont l'affichage des prix est configuré en HT, alors qu'en bas de la commande il est indiqué :

[eolia]

@Olivier75009
Curieux, le contrôle effectué par le code me semble pourtant juste

$new_sold = $order->total_paid_real + $amount;
                    if($new_sold > $order->total_paid) {
                        $this->errors[] = Tools::displayError('You cannot add a new payment because this order has already been paid in full');
                    }

Ou alors le contrôle devrait être peut-être être fait sur les arrondis.
Qu'avez-vous comme total_paid_real dans la table orders ?

[eolia]

@Olivier75009 Oui, et c'est ce que j'ai corrigé.

[Olivier75009]

pour cette commande : 119.480000 (il y a déjà eu un remboursement de 1,82€)

[Olivier75009]

Peut-on avoir la correction pour le calcul du groupe pro ?

[Olivier75009]

@eolia C'est le seul cas où nous n'avons pas pu éditer la commande)

[eolia]

@Olivier75009 a dit dans Calcul des règles paniers HT/TTC :

Peut-on avoir la correction pour le calcul du groupe pro ?

Si on m'en laisse le temps et que je puisse finaliser tous les tests, la version 1.6.2.21 sera libérée ce soir.

[Olivier75009]

Merci encore.

[Olivier75009]

@eolia Si j'affiche $new_sold et $total_paid dans Tools::displayError : "You cannot add a new payment because this order has already been paid in full 117.66 sur 117.660000".
En ajoutant $new.'0000' > (...) l'ajout passe.

[eolia]

Correctif:

                elseif(!Validate::isNegativePrice($amount)) {
                    $this->errors[] = Tools::displayError('The amount is invalid.');
                }
                elseif(!Validate::isGenericName(Tools::getValue('payment_method'))) {
                    $this->errors[] = Tools::displayError('The selected payment method is invalid.');
                }
                elseif(!Validate::isString(Tools::getValue('payment_transaction_id'))) {
                    $this->errors[] = Tools::displayError('The transaction ID is invalid.');
                }
                elseif(!Validate::isLoadedObject($currency)) {
                    $this->errors[] = Tools::displayError('The selected currency is invalid.');
                }
                elseif($order_has_invoice && !Validate::isLoadedObject($order_invoice)) {
                    $this->errors[] = Tools::displayError('The invoice is invalid.');
                }
                elseif(!Validate::isDate(Tools::getValue('payment_date'))) {
                    $this->errors[] = Tools::displayError('The date is invalid');
                }
                else {
                    $amount = Tools::ps_round($amount, _PS_PRICE_COMPUTE_PRECISION_);
                    $new_sold = $order->total_paid_real + $amount;
                    if($new_sold > $order->total_paid) {
                        $this->errors[] = Tools::displayError('You cannot add a new payment because this order has already been paid in full');
                    }

1ère ligne modifiée (1837) et ajout avant la ligne 1956 ($amount = Tools::ps_round($amount, PS_PRICE_COMPUTE_PRECISION);)