developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 13 years ago
Closed 13 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:
- 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.
- 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.
- 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)
Change History (6)
Changed 13 years ago by thomas
comment:1 Changed 13 years ago by laurentj
- Milestone set to Jelix 1.0.5
- Owner set to thomas
comment:2 Changed 13 years ago by thomas
Ligne 381 dans jDaoGenerator.class.php :
$src[] = ' $args = func_get_args();';
comment:3 Changed 13 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 13 years ago by laurentj
je vais ajouter des tests unitaires, et je commiterai.
comment:5 Changed 13 years ago by laurentj
- Resolution set to fixed
- Status changed from new to closed
Done.
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 :-)