View Trackers | Bugs | Export CSV

Date:
2020-02-25 18:19
Priority:
1
State:
Closed
Submitted by:
Sofien TIMEZOUAGHT (softime)
Assigned to:
Sofien TIMEZOUAGHT (softime)
Version cible:
4.13.0
Version concernée:
4.12.0
Summary:
Correction des méthodes de mise à jour d'une valeur

Detailed description
Certaines méthodes permet de mettre à jour un nombre limité de valeur (1 ou 2 en principe) à l'aide de "autoexecute()", utilisent en données à mettre à jour le tableau "$this->valF", or ce tableau récupère toutes les valeurs du formulaire posté. Dans ce cas il n'y a aucun intérêt à utiliser "autoexecute()" directement, autant passer par la méthode du core "modifier()".
Si le besoin est de mettre à jour une seule valeur alors cette façon peut provoquer des erreurs de traitement. En effet l'utilisation de $this->valF revient à mettre à jour tous les champs avec les valeurs du formulaire posté, donc si une valeur a été modifiée en amont cette modification est écrasé par la valeur de base du formulaire.

Ces méthodes ne sont pas toujours correctement utilisée, car dans certains cas la bonne solution aurait été de modifier directement le "setValF()".
Message  ↓
Date: 2021-01-20 15:22
Sender: Sofien TIMEZOUAGHT

Bonjour,

Correction des méthodes de mise à jour d'une valeur.

Cette évolution est déjà disponible dans la version 4.13.0 d'openADS.

La modification a été intégrée à openADS :
- Merge sur la version de développement : https://adullact.net/scm/viewvc.php/openfoncier?revision=9111&view=revision

Cordialement.
--
Sofien Timezouaght
http://www.atreal.fr

Date: 2021-01-20 15:22
Sender: Sofien TIMEZOUAGHT

Bonjour,

[Description du bug corrigé / de l'évolution développée]

Cette évolution est déjà disponible dans la version de développement et sera dans la prochaine version d'openADS.

La modification a été intégrée à openADS :
- Branche de développement :
- Merge sur la version de développement :

Cordialement.
--
[Signature]
http://www.atreal.fr

No attached documents

Field Old Value Date By
status_idOpen2021-01-20 15:22Sofien TIMEZOUAGHT
close_dateNone2021-01-20 15:22Sofien TIMEZOUAGHT
FEDER Powered By FusionForge Collaborative Development Environment Charte d'utilisation / Nous contacter / Mentions légales Haut de page