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

jDbPDOResultSet.fetchAll.patch (2.9 KB) - added by Vincentv 9 years ago.

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by Vincentv

comment:1 Changed 9 years ago by laurentj

  • review changed from review? to review-

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.

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.