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