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

Closed 12 years ago

#748 closed bug (fixed)

Mauvais chemin d'images à l'installation check_install.tpl

Reported by: Surfoo Owned by: bballizlife
Priority: low Milestone: Jelix 1.0.7
Component: module:jelix Version: 1.0.6
Severity: minor Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

J'installe Jelix comme le tuto (http://jelix.org/articles/fr/manuel-1.1/installation/application) et sans modifier le document root, je déplace lib/jelix-www/ à la racine que je renomme en www et modifie le fichier defaultconfig.ini.php et application.ini.php en conséquence.

Mais il y a des images qui sont en 404 : logo_jelix_moyen.png et jelix_powered.png.

C'est le template "lib/jelix/core-modules/jelix/templates/fr_FR/check_install.tpl" qui donne l'URL en dur au lieu de choisir la config :

<img src="{$j_basepath}jelix/design/images/logo_jelix_moyen.png" alt=""/>

Meme chose pour "lib/jelix-modules/jWSDL/templates/soap_doc.tpl"

Attachments (1)

748-1.patch (4.7 KB) - added by bballizlife 12 years ago.
Change {$j_basepath}jelix/design to {$j_jelixwww}design

Download all attachments as: .zip

Change History (10)

comment:1 in reply to: ↑ description Changed 12 years ago by Julien

Replying to Surfoo:

je déplace lib/jelix-www/ à la racine que je renomme en www

attention, il est dit dans la doc que lib/jelix-www/ doit être copié/déplacé/linké/renommé en "jelix", dans le document root du site (= le rép www de l'application).

on peut donc avoir (en très résumé):

  • lib
  • modules
  • var
  • www
    • jelix (copie/alias de lib/jelix-www)
    • index.php

Si l'hébergeur ne propose pas de pouvoir mettre des fichier hors document root, alors tu auras qqchose de ce genre :

  • lib
  • modules
  • var
  • jelix (copie/alias de lib/jelix-www)
  • index.php

Bref, je penses que tu as confondu lib/jelix-www et app/www

comment:2 Changed 12 years ago by Julien

donc INVALID a priori. confirmation ?

comment:3 Changed 12 years ago by laurentj

  • Component changed from jelix to module:jelix
  • Priority changed from normal to low
  • Severity changed from normal to minor

Non, pas invalide. Le template devrait tenir compte de l'option jelixWWWPath dans la config, et non avoir en dur jelix/. On devrait donc utiliser {$j_jelixwww} et non {$j_basepath}jelix/ .

comment:4 Changed 12 years ago by Julien

décidément c'est pas ma semaine (et on est que mardi, olé !)... j'ai pas lu jusqu'au bout...

j'ai (trop vite) pensé que l'erreur venait d'une confusion entre app/www et lib/jelix-www ...

Changed 12 years ago by bballizlife

Change {$j_basepath}jelix/design to {$j_jelixwww}design

comment:5 Changed 12 years ago by bballizlife

  • Owner set to bballizlife

Patch attached in order to change the path in the templates check_install.tpl (fr_FR, en_En and en_US). I created a new app to check images are displayed.

comment:6 Changed 12 years ago by laurentj

  • Milestone changed from jelix 1.1 to Jelix 1.0.7
  • review set to review+
  • Version changed from 1.1 beta 1 to 1.0.6

It's ok.

Land this patch into 1.0.x branch + trunk.

comment:7 Changed 12 years ago by Surfoo

Il y a encore le fichier lib/jelix-modules/jWSDL/templates/soap_doc.tpl (ligne 85) à corriger. Et sur la ligne 7 il y a "jelix/" en trop mais c'est mis en commentaire aussi.

comment:8 Changed 12 years ago by bballizlife

Commited in the trunk, r1181 and in the 1.0.x branch, r1182

comment:9 Changed 12 years ago by bballizlife

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