Chargement en cours
Validations dans la source 18
-
RastaPopoulos a rédigé
N'afficher le titre H1 que si demandé en param (car le bloc doit pouvoir être utilisé dans une page déjà existante) + corriger la récup de l'étape en cours qui cherchait dans {e} uniquement alors que le param peut être personnalisé. -
RastaPopoulos a rédigé
-
RastaPopoulos a rédigé
-
Maïeul a rédigé
-
RastaPopoulos a rédigé
-
RastaPopoulos a rédigé
Fix : tout comme en PHP, en squelette aussi il faut mettre _ devant l'argument pour que ça ne supprime pas les S à la fin, sinon ça n'appelle pas la bonne autorisation et du coup ça fait croire que c'est autorisé et cliquable alors que non
-
RastaPopoulos a rédigé
Deux améliorations : on précalcule toutes les autorisations des étapes directement dans le PHP puisqu'on boucle déjà sur chacune à ce moment là. Ainsi dans le squelette on se concentre sur l'affichage uniquement, aucun test à faire. De cette façon 1) le squelette de menu a alors le même cache pour tout le monde, et 2) comme on contrôle plus haut dans le PHP, on peut alors transmettre les du tunnel à tous les autoriser(). Et dans ces options on y passe le #ENV tout le temps. Ainsi les fonctions d'autoriser ont accès aux infos du contexte/page où on a inséré le tunnel : id_patate de la page en cours, etc.
-
RastaPopoulos a rédigé
Nouvelle fonctionnalité sans normalement casser l'existant : une nouvelle autorisation est *possible* autoriser_montunnel_lister_etape_tunnel() où le correspond à l'identifiant d'une étape à tester. Par défaut cette autorisation est toujours true donc l'étape existe, elle est listée. Mais si on renvoie false alors l'étape n'existe plus, elle n'est plus listée ni prise en compte dans les précédent/suivant etc. Cela permet donc de faire des étapes optionnelles très facilement (exemple facile à comprendre : ne montrer une étape Livraison d'une tunnel de commande, que si ya au moins un objet physique à envoyer dans le panier). On augmente Y, et au passage on met que ça marche en SPIP 4.
-
RastaPopoulos a rédigé
-
RastaPopoulos a rédigé
Oups coquille dans le test de l'étape courante, si c'est pas listable on doit revenir en arrière aussi !
-
RastaPopoulos a rédigé
Il manquait le passage du au deuxième tunnel_etape_suivante() qui se fait pour chaque étape pour le chemin + coquille
-
RastaPopoulos a rédigé
-
RastaPopoulos a rédigé
C'était encore plus fin que ça : le code était bon au départ, on ne doit tester si on peut accéder à une étape que si elle n'est pas masquée (celles masquées ne doivent pas être pris en compte, comme si elles n'existaient pas du tout)… SAUF s'il s'agit de celle demandé en param (la dernière à tester), dans ce cas, si elle est masquée, pas listable, c'est interdit, on doit être là par un vieux lien, mauvais URL, donc là aussi on force l'étape précédente.
-
tcharlss a rédigé
On peut placer les squelettes des étapes dans des sous-dossiers correspondant à chaque tunnel, ceux-ci seront utilisés en priorité. Utile quand on a plusieurs tunnels, cela permet de décider quels squelettes sont mutualisés et lesquels sont spécifiques à chaque tunnel.
-
tcharlss a rédigé
Et on passe en test.
-
tcharlss a rédigé
-
tcharlss a rédigé
-
JamesRezo a rédigé