Development #65328
Tarification - avoir un critère par défaut
0%
Description
Plutôt que de devoir définir un critère "domicile != 'commune' and domicile != 'comcom'", pour gérer tous les cas possibles, avoir un critère par défaut qui serait choisi si aucun autre critère ne matche.
Ce critère n'a du coup pas de condition (c'est le else).
Pouvoir choisir de l'activer ou non sur une catégorie ?
Pouvoir choisir le label ? ou on affiche "Sinon" ?
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0001-pricing-default-criteria-configuration-65328.patch 0001-pricing-default-criteria-configuration-65328.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
basé sur #65326
Mis à jour par Thomas Noël il y a presque 2 ans
Lauréline Guerin a écrit :
basé sur #65328
(ça doit être un autre, c'est le présent ticket)
Sinon :- il reste un "print" dans lingo/pricing/forms.py
- dans ce «clean» j'aurai bien provoqué une self.add_error si condition n'est pas vide mais que la case default a été cochée (au lieu de le vider discrètement) -- sauf que je viens de lire le javascript dans manager_criteria_form.html alors oublie moi :)
- mais par ailleurs, dans ce clean, serait-il possible de détecter qu'un critère "default" existe déjà quand on cherche à en ajouter un ?
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0001-pricing-default-criteria-configuration-65328.patch 0001-pricing-default-criteria-configuration-65328.patch ajouté
(ça doit être un autre, c'est le présent ticket)
Il fallait lire #65326 bien sûr :)
il reste un "print" dans lingo/pricing/forms.py
Supprimé
mais par ailleurs, dans ce clean, serait-il possible de détecter qu'un critère "default" existe déjà quand on cherche à en ajouter un ?
Je me suis posée la question, puis j'ai décidé que non, on verrait plus tard si nécessaire, sûrement pour de bonnes raisons mais je ne me souviens plus lesquelles.
Tu y tiens vraiment ? :)
Mis à jour par Thomas Noël il y a presque 2 ans
Lauréline Guerin a écrit :
Tu y tiens vraiment ? :)
Pas tant que ça je te rassure ! Mais alors, le possible « len(default_criterias) > 1 » qui déclenche un CriteriaConditionNotFound ça m'ennuie un poil : ça va nous retomber dans le support un jour, et on passera du temps à se rappeler le pourquoi :) On n'ajouterait pas une MultipleDefaultCriteriaCondition (ou tout autre nom plus heureux) ?
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution proposée à En cours
On n'ajouterait pas une MultipleDefaultCriteriaCondition (ou tout autre nom plus heureux) ?
Yes bonne idée; j'avais dans l'idée d'afficher quelque part dans l'interface un warning lorsqu'un modèle de grille ou une grille tarifaire est mal configurée (ticket à venir); ça pourra servir à ce moment-là
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0001-pricing-default-criteria-configuration-65328.patch 0001-pricing-default-criteria-configuration-65328.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Ca me semble bien tout bon.
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c01c17ecf8f66779911e354e921c59ef55447f99
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Jun 3 10:48:56 2022 +0200
pricing: default criteria configuration (#65328)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
pricing: default criteria configuration (#65328)