developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.
Opened 10 years ago
Closed 10 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 10 years ago by Julien
- Owner set to Julien
- Status changed from new to assigned
comment:2 follow-up: ↓ 3 Changed 10 years ago by laurentj
- review changed from review? to review-
comment:3 in reply to: ↑ 2 Changed 10 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 10 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
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.