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

#1333 closed bug (fixed)

installWizard: jDbProfile using dsn syntax is not updated

Reported by: Julien Owned by: Julien
Priority: normal Milestone: Jelix 1.3 beta 1
Component: jelix:installer Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Change History (4)

comment:1 Changed 9 years ago by Julien

  • Owner set to Julien
  • Status changed from new to assigned

comment:2 follow-up: Changed 9 years ago by laurentj

  • review changed from review? to review-

DSN notation is deprecated, since 1.2, as it is indicated in the manual. I prefer to have generic parameters. It's easier for developers (personnaly, I always have some difficulties to remember the DSN syntax :-) ), and ease implementation in jDb/wizard. This is why I created the usepdo parameter.

comment:3 in reply to: ↑ 2 Changed 9 years ago by Julien

  • review changed from review- to review?

Replying to laurentj:

DSN notation is deprecated, since 1.2, as it is indicated in the manual.

yup, I know this, but I discovered this bug when installing testapp, because PDO profile is still with dsn notation, and thus when trying to change db's name for exemple in the wizard it fails.

I prefer to have generic parameters. It's easier for developers (personnaly, I always have some difficulties to remember the DSN syntax :-) ), and ease implementation in jDb/wizard. This is why I created the usepdo parameter.

no problem for that, but we then should change profiles syntax in testapp.

comment:4 Changed 9 years ago by laurentj

  • Resolution set to fixed
  • review review? deleted
  • Status changed from assigned to closed

I fixed the bug, by just removing the dsn option during the write of the configuration. https://bitbucket.org/jelix/jelix-trunk/changeset/7cbb0ebe554c

Note: See TracTickets for help on using tickets.