Hello, Pour assurer la rétro compatibilité du schéma Bdd, tu vas changer des données clé ? -- Greg Le 30 juillet 2012 09:41, Dsls <dsls(a)morefnu.org&gt; a écrit :

Oh yeah, baby!

Bon j'ai fait un pull request juste quand il ne fallait pas ! c.à.d. juste quand tu commitais... Tout ça pour dire que je fais un premier jet sur un nouveau système de post_status plus générique (qui pourra marcher avec comment_status et autres) et qui reste compatible avec les anciens. Mais je me pose des questions sur la suite, car si on regarde comment ça marche aujourd'hui il y a des relations en dur entre les status et les permissions... Faut-il alors autoriser chaque script qui ajoute un status la possibilité de lui lier des permissions ? J'ai fait des essais mais pas très concluants/propres... Bref à voir. Le 31/07/2012 15:20, Dsls a écrit :

Je viens de regarder le code, j'aime bien le concept d'une classe dédiée. Il va en revanche falloir voir s'il faut vraiment gérer des classes héritées à chaque nouveau statut. Quant à la compatibilité, quitte à changer le type de la colonne, autant aussi changer la valeur des status par des choses plus parlantes... de toutes façons, je ne connais pas beaucoup de plugins à l'heure actuelle qui jouent sur les statuts des commentaires et des billets. Il ne faut pas non plus perdre de vue qu'un plugin implémentant un nouveau statut est désinstallable, et qu'il ne faut pas que tout soit cassé si ce plugin est désinstallé. Pour moi, les permissions doivent être gérées à part. Un jeu de callbacks me semble suffisant pour gérer ça. Le fait d'avoir une classe qui gère les statuts permet en plus de regrouper le tout en un seul endroit, plutôt que de s'éparpiller. -- Bruno

Le 01/08/2012 09:38, Dsls a écrit : Ça faisait trop d'argument pour une simple fonction et au début je voulais implémenter la gestion de leurs droits dedans... J'avais gardé les faux numérique pour la compatibilité et je les avais doublé avec des class-status aillant de vrai nom. Les deux donnant le même résultat. Dans ce cas la tu tombes sur le status "unknow" (gérable par un admin par exemple), rien ne bug/disparait jusqu'a ce que tu réinstalles le plugin. J'ai encore un peu de mal avec ça, en gros par exemple faut doubler la liste des status dispo avec un callback à chaque affichage d'un combo ? c.à.d. à chaque fois qu'il y a un auth->check('publish...') on le remplace par un callback ? C'est un peu lourd/bancale, nan ?