developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#643 closed bug (fixed)

DAO : évènements automatiques

Reported by: thomas Owned by: thomas
Priority: normal Milestone: Jelix 1.0.5
Component: jelix:dao Version: 1.0.4
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

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.

  1. 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.

  1. 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 (1)

643-jDao-automatic-event.diff (5.3 KB) - added by thomas 12 years ago.

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by thomas

comment:1 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.5
  • Owner set to thomas

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 :-)

comment:2 Changed 12 years ago by thomas

Ligne 381 dans jDaoGenerator.class.php :

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

comment:3 Changed 12 years ago 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.

comment:4 Changed 12 years ago by laurentj

je vais ajouter des tests unitaires, et je commiterai.

comment:5 Changed 12 years ago by laurentj

  • Resolution set to fixed
  • Status changed from new to closed

Done.

Note: See TracTickets for help on using tickets.