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.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 9 years ago
Closed 9 years ago
#1419 closed bug (fixed)
Impossible de forcer un fetch_mode
| Reported by: | Vincentv | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Jelix 1.3 |
| Component: | jelix:db | Version: | 1.2.4 |
| Severity: | normal | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Documentation needed: | no | |
| Hosting Provider: | Php version: |
Description
Bugs: impossible de forcer un fetch_mode, les paramètres sont de facto inutile, le défaut de fetchAll est PDO::FETCH_OBJ alors que la logique et la fonction de PHP a un défaut à FETCH_ASSOC, fetch_all n'appelle pas les callbacks pour modifier les résultats, ...
Correction: Complétion de la fonction
Correction optionnelle: remetttre le mode correct par défaut à fetchAll (mais celà casse la compatibilité bien sur, à voir pour 1.3 j'imagine)
Remarque: Testé uniquement avec Oracle et c'est une contribution de A. Nonyme :)
Attachments (1)
Change History (3)
Changed 9 years ago by Vincentv
comment:1 Changed 9 years ago by laurentj
- review changed from review? to review-
comment:2 Changed 9 years ago by laurentj
- Resolution set to fixed
- review review- deleted
- Status changed from new to closed
I made requested changes on the patch and pushed it. https://github.com/jelix/jelix/commit/895a80c2af59319d78bdbd8748a9b631b22f8ac2
Note: See
TracTickets for help on using
tickets.
oui la valeur par défaut est FETCH_OBJ, parce que dans jelix, on veut tout en objet. il faut indiquer FETCH_OBJ à parent::fetchAll(); quand il n'y a pas de fetch_style. je ne veux pas de cassage de fonctionnalité.
tu as aussi un problème de variable $rec dans ta boucle sur les modifiers. D'ailleurs, je trouve cette boucle assez moyen : ça fait un parcours de plus sur les résultats. on augmente la complexité algo.