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

Closed 11 years ago

#999 closed bug (fixed)

trunk: installapp fix

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

Description

Hello,

I think I found some bugs in the install system in the trunk version.

Here's a patch, please check if I did right. Thanks.

Attachments (2)

999-installapp-fix.diff (2.0 KB) - added by Julien 11 years ago.
999-installapp-fix-2.diff (2.1 KB) - added by Julien 11 years ago.

Download all attachments as: .zip

Change History (8)

Changed 11 years ago by Julien

comment:1 Changed 11 years ago by Julien

  • Status changed from new to assigned

comment:2 Changed 11 years ago by laurentj

-  jxs_init_jelix_env(); 

I'm not sure you should remove it. Why did you remove it ?

comment:3 Changed 11 years ago by Julien

there's an error while initing environement. Canno't remeber which one exactly, will try again this afternoon.

As I said before, I didn't follow the installer's development, so I may have missed something.

comment:4 Changed 11 years ago by Julien

Ok, the problem was jxs_init_jelix_env() checking installer.ini.php, which didn't exist as the app was not installed.

Here's a clean patch.

Changed 11 years ago by Julien

comment:5 Changed 11 years ago by laurentj

  • review changed from review? to review+

ok

comment:6 Changed 11 years ago by Julien

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.