FusionForge de l'ADULLACT: openADS : [#9316] Correction des méthodes de mise à jour d'une valeur

Date :
25/02/2020 18:19

Priorité :
1

État :
Closed

Proposé par :
Sofien TIMEZOUAGHT (softime)

Confié à :
Sofien TIMEZOUAGHT (softime)

Version cible :
4.13.0

Version concernée :
4.12.0

Résumé :
Correction des méthodes de mise à jour d'une valeur

Description détaillée
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()".
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()".

Commentaires (2)
Pièces jointes
Modifications (2)

Message
Date : 20/01/2021 15:22 Expéditeur : 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 : 20/01/2021 15:22 Expéditeur : 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

Pas de documents joints

Champ	Ancienne valeur	Date	Par
status_id	Open	20/01/2021 15:22	Sofien TIMEZOUAGHT
close_date	Aucun(e)	20/01/2021 15:22	Sofien TIMEZOUAGHT

Powered By FusionForge Collaborative Development Environment

Charte d'utilisation / Nous contacter / Mentions légales

Haut de page