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 8 years ago

Closed 7 years ago

Last modified 7 years ago

#1302 closed bug (fixed)

testapp database installation fails if tables already created

Reported by: rahal Owned by: laurentj
Priority: normal Milestone: Jelix 1.3 beta 1
Component: app:testapp Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

The sql scripts should use "CREATE IF NOT EXISTS" instead of "CREATE"

Both jelix_tests and testapp modules are concerned

https://bitbucket.org/rahal/jelix-trunk-patches/src/b21785c4e441/fix.installer.jelix_tests.patch and https://bitbucket.org/rahal/jelix-trunk-patches/src/b21785c4e441/fix.installer.testapp.patch for a patch

Change History (4)

comment:1 Changed 8 years ago by laurentj

The sql scripts should use "CREATE IF NOT EXISTS" instead of "CREATE"

Why ? a database is supposed to be empty when you install an application.

comment:2 Changed 8 years ago by rahal

Yes it is supposed to..

In my case, the database was existing ( from previous jelix installation ) and the install just failed ( for some reason, the install script wasn't supposed to be run, but did! ) .

I checked the other install scripts for the other jelix-modules and found that they were all using "CREATE IF NOT EXISTS" .. Than it seemed to me that we can use the same thing in testapp modules as we can avoid this failure.

comment:3 Changed 7 years ago by laurentj

  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from new to closed

comment:4 Changed 7 years ago by laurentj

  • Milestone set to Jelix 1.3
Note: See TracTickets for help on using tickets.