Skip to content

FIX: recalcul des prix non déclenché sur certains événements de prix d'achat/vente - #13

Merged
mapiolca merged 2 commits into
mainfrom
2026-03-25-troubleshoot-price-update-functionality
Mar 25, 2026
Merged

FIX: recalcul des prix non déclenché sur certains événements de prix d'achat/vente#13
mapiolca merged 2 commits into
mainfrom
2026-03-25-troubleshoot-price-update-functionality

Conversation

@mapiolca

Copy link
Copy Markdown
Owner

Motivation

  • Le trigger DynamicsPrices ne lançait pas toujours le recalcul des prix parce que l'identifiant extrait du contexte ($object->id) pouvait être l'id d'une ligne de prix et non l'id du produit, empêchant la recherche du fk_product et donc le déclenchement du recalcul.

Description

  • Remplacement du fallback simpliste par une résolution d'id produit robuste en ajoutant la méthode privée resolveProductIdFromTriggerAction() dans core/triggers/interface_99_modDynamicsPrices_DynamicsPricesTriggers.class.php.
  • La résolution utilise d'abord fk_product si présent, puis : id uniquement pour les actions produit, lookup SQL dans la table product_price pour les actions PRODUCT_PRICE_*, et lookup SQL dans product_fournisseur_price pour les actions d'achat (*_BUYPRICE_*).
  • runTrigger() appelle désormais resolveProductIdFromTriggerAction() avant d'exécuter la fonction de recalcul (update_customer_prices_from_cost_price ou update_customer_prices_from_suppliers) et propage le recalcul aux kits parents via dynamicsprices_get_parent_kits().
  • Fichier modifié : core/triggers/interface_99_modDynamicsPrices_DynamicsPricesTriggers.class.php (ajout de ~63 insertions, suppression de ~13 lignes superflues).

Testing

  • Vérification de syntaxe PHP : php -l core/triggers/interface_99_modDynamicsPrices_DynamicsPricesTriggers.class.php — réussite (pas d'erreurs de syntaxe).
  • Vérification du diff et commit local : git diff et git commit — patch ajouté et commité avec succès.
  • Aucun test unitaire automatisé spécifique au module n'a été exécuté dans ce rollout. Les contrôles effectués étaient des vérifications statiques et de commit, toutes réussies.

Codex Task

@mapiolca
mapiolca merged commit a7f24d9 into main Mar 25, 2026
1 check passed
@mapiolca
mapiolca deleted the 2026-03-25-troubleshoot-price-update-functionality branch March 25, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant