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

Last modified 12 years ago

#582 closed enhancement (fixed)

jForms : dao datasource. it should be possible to specify an optional profile

Reported by: bibo Owned by:
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:forms Version: trunk
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

title says it all. <datasource dao... profile="myprofile"/>

Attachments (2)

jelix-trunk-#582.patch (5.4 KB) - added by bibo 12 years ago.
add profile to dao datasource
jelix-trunk-#582.2.patch (5.3 KB) - added by bibo 12 years ago.
deuxieme version qui corrige un petit bug

Download all attachments as: .zip

Change History (6)

Changed 12 years ago by bibo

add profile to dao datasource

comment:1 Changed 12 years ago by bibo

  • review set to review?

Changed 12 years ago by bibo

deuxieme version qui corrige un petit bug

comment:2 Changed 12 years ago by laurentj

  • Documentation needed set
  • Milestone set to Jelix 1.1 beta 1
  • review changed from review? to review+
  • Severity changed from normal to trivial
  • You forgot to define the $profile property in the jFormsDaoDatasource class.
  • On the constructor, I think it is better to put the $profile parameter after the $key parameter, and with as a default value.

r+ if you fix this little recommendation.

comment:3 Changed 12 years ago by bibo

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

comment:4 Changed 12 years ago by bibo

  • Documentation needed unset
Note: See TracTickets for help on using tickets.