| correction skin_variant pour Yacs Martin RC7 - 17,563 bytes |
Communauté « Développement « yacs Martin RC7 est livré «
Pb avec skin_variant [Integrated]
| Workflow | Patch submission |
| Status | Software has been fully integrated
|
| Owner | Christophe Battarel |
| Progress | ![]() |
Issue description
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à.
Je joins ici la correction basée sur la 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 -


