| correction skin_variant pour Yacs Martin RC7 - 17,563 bytes, 6 downloads |
Projets « Campagnes de tests « yacs Martin RC7 est livré «
Pb avec skin_variant [Integrated]
Yacs permet d'afficher certaines pages en utilisant une variante de skin. Par exemple, on peut utiliser template_foo_bar.php au lieu de template.php
| Workflow | Patch submission |
| Status | Software has been fully integrated
|
| Owner | Christophe Battarel |
| Progress | ![]() |
Pour ce faire, on peut paramétrer une section en lui passant variant_foo_bar en option, ou passer dans l'url cette variante avec sections/view.php?id=xx&variant=foo_bar
Il y a un petit souci avec ce dernier mode de fonctionnement, car, si $context['skin_variant'] est bien mis à jour avec le $_REQUEST['variant'], cette valeur est écrasée par la suite en ligne 786-796 de global.php.
Il suffit de tester si $context['skin_variant'] n'est pas déjà renseigné à cet endroit-là.
correction skin_variant pour Yacs Martin RC7
Files
Comments
Ok, donc on est d'accord que le paramétrage d'une section peut être changé lors de l'appel de la page, en complétant l'URL ?
J'ai "linéarisé" le code que tu proposes comme suit :
// variant is already set
if(isset($context['skin_variant']))
;
// use item variant
elseif($options && preg_match('/\bvariant_(.+?)\b/i', $options, $matches))
$context['skin_variant'] = $matches[1];
Question subsidiaire : comment ça marche avec les liens canoniques, et les redirections systématiques telles que mises en place à la demande de Thierry ?
Merci pour cette intégration !
" Question subsidiaire : comment ça marche avec les liens canoniques, et les redirections systématiques telles que mises en place à la demande de Thierry ? "
et bien, ça marche très bien ! il suffit de tester si l'url contient déjà un '?', auquel cas on lui concatène '&variant=ma_variante', sinon '?variant=ma_variante'.
Christophe Battarel - Société altairis -












