Ticket #582 (closed enhancement: fixed)

Opened 6 months ago

Last modified 5 months ago

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

Reported by: bibo Assigned to:
Priority: normal Milestone: Jelix 1.1 beta 1
Component: jelix:forms Version: trunk
Severity: trivial Keywords:
Cc: Php version:
Review: review+ Hosting Provider:
Documentation needed: 0 Blocking:

Description

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

Attachments

jelix-trunk-#582.patch (5.4 kB) - added by bibo on 05/16/08 15:27:59.
add profile to dao datasource
jelix-trunk-#582.2.patch (5.3 kB) - added by bibo on 05/16/08 18:02:41.
deuxieme version qui corrige un petit bug

Change History

05/16/08 15:27:59 changed by bibo

  • attachment jelix-trunk-#582.patch added.

add profile to dao datasource

05/16/08 15:29:36 changed by bibo

  • review set to review?.

05/16/08 18:02:41 changed by bibo

  • attachment jelix-trunk-#582.2.patch added.

deuxieme version qui corrige un petit bug

05/28/08 10:33:48 changed by laurentj

  • review changed from review? to review+.
  • docneeded set to 1.
  • severity changed from normal to trivial.
  • milestone set to Jelix 1.1 beta 1.
  • 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.

05/29/08 10:44:41 changed by bibo

  • status changed from new to closed.
  • resolution set to fixed.

06/22/08 17:03:38 changed by bibo

  • docneeded deleted.
Download in other formats: Comma-delimited Text Tab-delimited Text RSS Feed