par sly » 03 févr. 2016, 00:41
Dominique a écrit :Ben voilà le loup: il y a du code de recherche dans la base à factoriser pour TOUTES les pages.
Je vois le problème, comme la petite étoile qui indique un besoin de modération pour lequel j'ai écris :
FIXME : Dans une logique de rangement parfait, ça ne devrait pas être ici, mais dans chaque contrôleur qui a besoin (ou pas) de modifier le bandeau avec l'étoile, mais la factorisation a eu raison de moi ;-)
Si quelqu'un veut le bouger, il a mon feu vert -- sly
Dans la même logique de "flemme" ce que tu as fais est finalement très acceptable. C'est plus dur de se rappeler dans quel fichier on a ce code, mais ça réduit grandement les répétitions.
Dominique a écrit :moins le forum et de vieux trucs tel la gestion
Voui, là c'est pénible ce menu est vide quand on est dans le forum.
Pour /gestion bon, dès qu'un courageux s'y colle, zou ça sera réglé, pour le forum (et la gestion le temps que), bon ben, y'a qu'a mettre ton générateur de zone dans une fonction et l'appeler juste avant l'inclusion du bandeau.
Aller hop, je ne sais toujours pas où il faudrait ranger ça, donc comme l'étoile dans includes/autoconnexion.php, en attendant une prochaine tempête de réorganisation des trucs qui ne devraient pas être là où ils sont.
[quote="Dominique"]Ben voilà le loup: il y a du code de recherche dans la base à factoriser pour TOUTES les pages.
[/quote]
Je vois le problème, comme la petite étoile qui indique un besoin de modération pour lequel j'ai écris :
[quote]
FIXME : Dans une logique de rangement parfait, ça ne devrait pas être ici, mais dans chaque contrôleur qui a besoin (ou pas) de modifier le bandeau avec l'étoile, mais la factorisation a eu raison de moi ;-)
Si quelqu'un veut le bouger, il a mon feu vert -- sly
[/quote]
Dans la même logique de "flemme" ce que tu as fais est finalement très acceptable. C'est plus dur de se rappeler dans quel fichier on a ce code, mais ça réduit grandement les répétitions.
[quote="Dominique"]moins le forum et de vieux trucs tel la gestion[/quote]
Voui, là c'est pénible ce menu est vide quand on est dans le forum.
Pour /gestion bon, dès qu'un courageux s'y colle, zou ça sera réglé, pour le forum (et la gestion le temps que), bon ben, y'a qu'a mettre ton générateur de zone dans une fonction et l'appeler juste avant l'inclusion du bandeau.
Aller hop, je ne sais toujours pas où il faudrait ranger ça, donc comme l'étoile dans includes/autoconnexion.php, en attendant une prochaine tempête de réorganisation des trucs qui ne devraient pas être là où ils sont.