Ticket #1222 (closed defect: fixed)
Meilleur support de public_url et themes_url
| Reported by: | bruno | Owned by: | xave |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | module:core | Version: | 2.2 |
| Severity: | normal | Keywords: | |
| Cc: |
Description
Il semblerait que certains blogs soient configurés avec des public_url et themes_url contenant des url complètes ( http:// inclus).
A regarder de plus près le code, ça marchotte presque partout, sauf à quelques endroits. Exemples : admin/media_item.php, ligne 396 admin/blog_theme.php, ligne 158, ...
2 solutions :
- autoriser le http:// dans les *_url, et adapter le code en conséquence
- les filtrer à la base et ne pas supporter les url complètes
Change History
comment:2 follow-up: ↓ 3 Changed 14 years ago by xave
Permettre de gérer des urls complètes permettrait de mettre les ressources fixes sur un serveur configuré différemment. SAuf que ... les thèmes, c'est aussi du php qu'on doit interpréter, non ?
comment:3 in reply to: ↑ 2 Changed 14 years ago by bruno
Replying to xave:
Permettre de gérer des urls complètes permettrait de mettre les ressources fixes sur un serveur configuré différemment. SAuf que ... les thèmes, c'est aussi du php qu'on doit interpréter, non ?
themes_url est uniquement utilisé pour afficher les éléments "publics" du thème (images, css, ...). Les fichiers php et templates sont récupérés à partir du themes_url.
Si on prend l'exemple d'un multiblog avec sous-domaines, les répertoires de thèmes communs ne sont pas forcément accessibles à un sous blog, d'où l'éventuel intérêt d'autoriser le http dans themes_url.
Ex : une ferme avec une arborescence :
- all-blogs/ contenant toute l'installation dc, pas de sous-domaine pointant dessus
- all-blogs/admin avec un sous-domaine admin.monblog.com pointant dessus
- all-blogs/themes avec un sous-domaine themes.monblog.com pointant dessus
- blog1/ : url blog1.monblog.com
- blog2/ : url blog2.monblog.com
pour blog1 et blog2, on peut définir alors themes_url= http://themes.monblog.com et themes_path=../../all-blogs/themes Ca doit passer. Dans la version actuelle ça passe presque, à l'exception de 2-3 fichiers qui s’emmêlent les pinceaux.
comment:4 Changed 14 years ago by Dsls <dsls@…>
- Status changed from new to closed
- Resolution set to fixed
(In [91268cf48758]) Better support for http valued public_url and themes_url. Closes #1222
En fait, à trop d'endroits on "reconstruit" les public_url et themes_url.
Avoir une méthode unifiée quelque part qui le fait une fois pour toutes serait bien.
Je propose d'ajouter les méthodes getThemesURL et getPublicURL à la classe dcBlog.