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

Owner

Christophe Battarel

Progress

Workflow

Patch submission

Status

Software has been fully integrated

Submission on June 15 2010 by Christophe Battarel
Finalization on June 25 2010 by Bernard Paques

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

Discussion (4)
Attachments (1)

Christophe Battarel	on June 25 2010 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 -
Bernard Paques	on June 25 2010 Le logiciel a été intégré
Bernard Paques	on June 25 2010 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 ?
Christophe Battarel	on June 15 2010 La page a été créée Christophe Battarel - Société altairis -