View Issue Details

IDProjectCategoryView StatusLast Update
0000012CAMAP/InterAMAP44CAMAPpublic2023-11-07 16:20
ReporterPatrice Assigned ToAmaury  
PrioritynormalSeveritymajorReproducibilityalways
Status closedResolutionfixed 
PlatformGandiVPSOSDebianOS Version11
Product VersionPreprod 
Target Versionv1 (Prod InterAMAP44)Fixed in Versionv1 (Prod InterAMAP44) 
Summary0000012: Gestion de la provision enregistrée dans un contrat AMAP variable
Description

Pour reproduire l'erreur:

Cela concerne les contrats AMAP variable avec un "minimum de commande sur la durée du contrat" ( ici dans l'exemple 10€ )

1 - une souscription a été saisi par l'amapien avec une première commande supérieure au "minimum de commande sur la durée du contrat" au prorata des distributions restantes ( dans l'exemple 8€ )

2 - l'amapien remet un chèque de provision supérieur à sa première commande en vu de futures commandes ( dans exemple 50€ )

3 - l'amapien, saisi une commande pour une autre date de distribution dont le total est de 20€ donc bien inférieur à sa provision de 50€ , la commande est bloquée avec ce message:

Pour aider à trouver l'erreur : si dans la gestion des paiements le chèque de provision n'est pas enregistré il n'y a pas d'erreur.

TagsRemonté à Alilo

Activities

Amaury

2023-04-12 23:41

administrator   ~0000008

Last edited: 2023-11-07 16:20

camap-hx/src/service/SubscriptionService.hx : commenter

          if ( subscriptionPayments != 0 && subscriptionPayments > catalogMinOrdersTotal) {
              return subscriptionPayments;
          }
426     /**
 427         Compute catalog minimum orders total.
 428         Can be used with catalog arg if subscription is not created yet.
 429     **/
 430     public static function getCatalogMinOrdersTotal( catalog:db.Catalog, ?subscription:db.Subscription ) : Float {
 431
 432         if ( catalog.catalogMinOrdersTotal == 0 ) {
 433             return null;
 434         }
 435
 436         var subscriptionDistribsNb = 0;
 437         if ( subscription != null ) {
 438             subscriptionDistribsNb = getSubscriptionDistribsNb( subscription, null, true );
 439         } else {
 440             subscriptionDistribsNb = db.Distribution.manager.count( $catalog == catalog && $date >= SubscriptionService.getNewSubscriptionStartDate( catalog ) );
 441         }
 442
 443         var catalogAllDistribsNb = db.Distribution.manager.count( $catalog == catalog );
 444         if ( catalogAllDistribsNb == 0 ) return null;
 445         var ratio = subscriptionDistribsNb / catalogAllDistribsNb;
 446
 447         // safer to do a "floor" than a "round"
 448         var catalogMinOrdersTotal = Math.floor(ratio * catalog.catalogMinOrdersTotal);
 449
 450 /**
 451 Issue #0000012
 452 **/
 453
 454
 455         //si paiements, le minimum à commander correspond à la provision déja payée
 456         if ( subscription != null ) {
 457             var subscriptionPayments = subscription.getPaymentsTotal();
 458 //          if ( subscriptionPayments != 0 && subscriptionPayments > catalogMinOrdersTotal) {
 459 //              return subscriptionPayments;
 460 //          }
 461         }
 462
 463         return catalogMinOrdersTotal;
 464     }

Amaury

2023-04-29 13:46

administrator   ~0000010

Corrigé par François dans les sources github

Issue History

Date Modified Username Field Change
2023-04-12 23:34 Amaury New Issue
2023-04-12 23:34 Amaury Status new => assigned
2023-04-12 23:34 Amaury Assigned To => Patrice
2023-04-12 23:34 Amaury Reporter Amaury => Patrice
2023-04-12 23:41 Amaury Note Added: 0000008
2023-04-12 23:42 Amaury Note Edited: 0000008
2023-04-12 23:56 Amaury Tag Attached: Remonté à Alilo
2023-04-12 23:56 Amaury Assigned To Patrice => Amaury
2023-04-13 09:25 Amaury Status assigned => resolved
2023-04-13 09:25 Amaury Resolution open => fixed
2023-04-13 09:25 Amaury Fixed in Version => Preprod
2023-04-29 13:46 Amaury Status resolved => closed
2023-04-29 13:46 Amaury Note Added: 0000010
2023-05-24 21:51 Amaury Target Version => v1 (Prod InterAMAP44)
2023-06-18 23:48 Amaury Fixed in Version Preprod => v1 (Prod InterAMAP44)
2023-07-15 16:49 Amaury Category CAMAP Amap44 => CAMAP
2023-11-07 16:19 Amaury Description Updated
2023-11-07 16:20 Amaury Note Edited: 0000008