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

#1087 closed bug (fixed)

jInstallChecker's error reporting may lead to an exception

Reported by: bricet Owned by:
Priority: normal Milestone: Jelix 1.2 beta
Component: jelix:installer Version: trunk
Severity: normal Keywords: jInstallChecker errors
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

On jInstallChecker's run(), various checks are made. If there is an error (e.g. : var/log is not writable by Jelix), we get a dirty error message instead of a fine welcome Jelix page with CSS presentation and so on.

This is due to a double-call to jInstallerMessageProvider::get(). We also note that some errors are counted twice.

Attachments (1)

ticket_1087.diff​ (1.0 KB) - added by bricet 10 years ago.

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by bricet

comment:1 Changed 10 years ago by laurentj

  • Component changed from jelix to jelix:installer
  • Milestone set to Jelix 1.2 beta
  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from new to closed

ok for me. thank for this patch.

​http://bitbucket.org/jelix/jelix-trunk/changeset/9b24b08a2859/

Note: See TracTickets for help on using tickets.