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

Closed 13 years ago

#470 closed bug (fixed)

message d'avertissement manquant lors du traitement des $userConfig

Reported by: rahal Owned by: laurentj
Priority: normal Milestone: Jelix 1.0.3
Component: jelix:core Version: 1.0.1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version: 5.2.1

Description

Un oubli de ; ( ou suppression non intentionnelle :) ) avant le <?php die(); ?> dans les fichiers de config ne relève pas d'erreurs..

On se retrouve avec $userConfig qui vaut array(), et donc les parametres de config utlisateurs ne sont pas pris en compte ( et ca fait perdre énormement de temps si on ne repère pas de suite ).

A mon avis, c'est présent dans toutes les versions php, peut importe la plateforme.

Ce bout de code devrait eviter de perdre pas mal de temps ( jConfigCompiler )

if( false === ($userConfig = parse_ini_file(JELIX_APP_CONFIG_PATH.$configFile,true)))
   die("Syntax error in the Jelix config file $configFile !");
if (count($userConfig)==0)
 die("User config file is empty or ; missing before <?php die(''); ?> in   $configFile !");

Il faudra peut être faire ce test au niveau du traitement de defaultconfig.ini.php aussi , et eventuellement jIniFile.class.php ( on traite rarement un fichier ini pour recuperer rien, quoi que.. ).

Change History (3)

comment:1 Changed 13 years ago by laurentj

J'aimerai bien comprendre : c'est quoi le problème exactement ??

comment:2 Changed 13 years ago by laurentj

  • Component changed from jelix to jelix:core
  • Milestone set to Jelix 1.0.3
  • Owner set to laurentj

ok j'ai compris. Cela concerne la lecture des fichiers de config de jelix.

comment:3 Changed 13 years ago by laurentj

  • Resolution set to fixed
  • Status changed from new to closed

Ce serait dommage que ça fasse une erreur juste pour un fichier vide. J'ai plutôt enlevé les @ devant les instructions parse_ini_file.

Note: See TracTickets for help on using tickets.