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

Closed 10 years ago

#1064 closed bug (fixed)

Testapp does not create jlx_cache and jsessions tables

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

Description

Using mysql (not tested with pgsql) and following all instructions of INSTALL.txt, running all PHP test of testapp leads to 12 exceptions and 2 fails because jlx_cache and jsession tables do not exist.

Attachments (1)

ticket_1064.diff (1.6 KB) - added by bricet 10 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 10 years ago by bricet

  • Summary changed from Testapp does not create jlx_cache and jsession tables to Testapp does not create jlx_cache and jsessions tables

Typo mistake : I was talking about jsessions (with a trailing "s")

Changed 10 years ago by bricet

comment:2 Changed 10 years ago by bricet

  • review set to review?

Call of sql files form "jelix" module to create those 2 tables if they do not exist. Dirty relative path, but can't find a better way to access them from here ...

comment:3 Changed 10 years ago by laurentj

  • Milestone set to Jelix 1.2

comment:4 Changed 10 years ago by laurentj

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

Ok for me. I improved the method execSQLScript to accept a module name, so we don't have to give "ugly" path :-). Improved also the support of $profile parameter.

http://bitbucket.org/jelix/jelix-trunk/changeset/25a6e0653d05/

Thanks for the patch.

Note: See TracTickets for help on using tickets.