Ticket #643 (closed bug: fixed)

Opened 2 months ago

Last modified 2 months ago

DAO : évènements automatiques

Reported by: thomas Assigned to: thomas
Priority: normal Milestone: Jelix 1.0.5
Component: jelix:dao Version: 1.0.4
Severity: normal Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

Corrections de plusieurs petits bugs:

1. Selecteur de dao

Les évènements reçoivent tous un paramètre dao qui contient le sélecteur du dao. Lors de la notification des évènements ce paramètre est récupéré grâce à la propriété _daoselector alors que cette propriété se nomme en réalité _daoSelector.

2. Nom des évènements des méthodes personnalisées

La classe jDaoGenerator génère les évènements daoSpecificUpdate* ou daoSpecificInsert* alors que la documentation précise que les méthodes personnalisées génèrent uniquement les méthodes de type update ou delete.

3. Liste des paramètres pour les méthodes personnalisées

Les évènements des méthodes personnalisées reçoivent un paramètre contenant la liste des paramètres récupérés avec la fonction func_get_args(). Cette fonction est utilisée directement en paramètre de la fonction jEvent::notify et provoque une erreur. Cette fonction dépend de la portée courante pour déterminer les détails des paramètres, ils ne peuvent être utilisés en tant que paramètre d'une fonction. Pour passer cette valeur, il faut d'abord assigner les résultats à une variable.

Attachments

643-jDao-automatic-event.diff (5.3 kB) - added by thomas on 07/07/08 17:11:09.

Change History

07/07/08 17:11:09 changed by thomas

  • attachment 643-jDao-automatic-event.diff added.

07/07/08 17:21:33 changed by laurentj

  • owner set to thomas.
  • milestone set to Jelix 1.0.5.

Tu as changé func_get_args() en $args, mais il est défini où ce $args ? ;-) Pour le reste, ok. On aurait du faire des tests unitaires là dessus :-)

07/07/08 19:48:28 changed by thomas

Ligne 381 dans jDaoGenerator.class.php :

 $src[] = '   $args = func_get_args();';

07/08/08 11:41:08 changed by laurentj

  • review changed from review? to review+.
  • version changed from trunk to 1.0.4.

Ok, j'avais pas vu. Merci pour le patch :-)

Note pour le commiter: à commiter dans le trunk et la branche 1.0.x.

07/08/08 12:31:51 changed by laurentj

je vais ajouter des tests unitaires, et je commiterai.

07/08/08 14:31:10 changed by laurentj

  • status changed from new to closed.
  • resolution set to fixed.

Done.

Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed