Projet

Général

Profil

Actions

Anomalie #1101

fermé

galette_local.css KO

Ajouté par Jérémy HIEULLE il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
CSS
Version cible:
-
Début:
30/10/2017
Echéance:
% réalisé:

0%

Temps estimé:
Version utilisée:

Description

Intro

Bonjour,

Comme indiqué dans le titre, le ficher galette_local.css n'est plus appelé en 0.9 (rc1 à rc3)

Tests

Plus précisément, c'est la ligne 10 du fichier template/default/common_header.tpl qui ne fonctionne pas.
Je ne suis pas un spécialiste mais si je ne me trompe pas : l'emplacement du fichier en php n'est pas bon. ci-dessous le code présent actuellement :

 {assign var="localstylesheet" value="`$galette_base_path``$template_subdir`galette_local.css"}
        {if file_exists($localstylesheet)}
            <link rel="stylesheet" type="text/css" href="{base_url}/{$localstylesheet}" />
        {/if}

ici localstylesheet est égal à : ./../themes/default/galette_local.css
alors que nous souhaitons effectuer le test soit sur le chemin absolu, soit ./themes/default/galette_local.css
(en tout cas, c'est ce que j'ai constaté lors de mes tests)

Résolution possible

J'ai modifié le code ainsi en essayant au mieux de garder la logique présente dans le code.. mais il y a peut-être mieux à faire...
(je garde le même chemin que les lignes précédentes pour l'édition de la feuille de style dans le code html mais j'utilise _CURRENT_THEME_PATH pour le test de l'existence du fichier (qui doit être défini dans smarty si j'ai compris le système).

include/dependencies.php (ajout à la ligne 110 ou une autre d'ailleurs..)

$smarty->assign('_CURRENT_THEME_PATH', _CURRENT_THEME_PATH);

template/default/common_header.tpl (remplacement ligne 10 à 14)

assign var="localstylesheet" value="`$_CURRENT_THEME_PATH`galette_local.css"}
        {if file_exists($localstylesheet)}
                it works!
            <link rel="stylesheet" type="text/css" href="{base_url}/{$template_subdir}galette_local.css" />
        {/if}

Je suis pressé de voir votre version afin de voir à quel point je m'en éloigne :).

PS : Je n'ai pas testé mais une modification similaire doit être effectuée pour galette_print_local.css

Mis à jour par Jérémy HIEULLE il y a plus de 6 ans

J'ai oublié d'enlever la ligne it works! sur mon dernier quote :(

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Catégorie mis à CSS
  • Statut changé de Nouveau à In Progress
  • Assigné à mis à Johan Cwiklinski

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus de 6 ans

  • Statut changé de Résolu à Fermé
  • % réalisé changé de 100 à 0

Jérémy HIEULLE a écrit :

Je suis pressé de voir votre version afin de voir à quel point je m'en éloigne :).

Le correctif implémenté est exactement ce que tu as proposé, merci ;)

Actions

Formats disponibles : Atom PDF