J'ai l'impression qu'il y a un petit souci avec le plugin Massicot
dans sa version actuelle : aucune vérification d'autorisation n'est
effectuée, ce qui permet de recadrer des documents que l'on n'aurait
normalement pas le droit de modifier (logos d'autres articles que les
siens, logos d'autres utilisateur·trices, etc.).
J'ai bricolé le patch ci-joint pour rétablir un contrôle un peu plus
strict des autorisations, s'appuyant sur `autoriser('modifier', ...)`
pour savoir si un·e utilisateur·trice est autorisé·e à recadrer tel
ou tel logo ou document.
Je ne sais pas trop quelle est la meilleure manière de procéder, mais
si vous préférez, je peux aussi faire un pull request sur git.spip.net
(mais à ce moment-là, il me faudrait un compte sur ce site, s'il vous
plaît).
je vois que tu fork le dépôt, c'est un fonctionnement qui n'est pas encore expliqué très clairement (our bad) : sur Git comme sur l'ancienne zone SVN, tout le monde a les mêmes droits sur tous les dépôts communautaires. Il n'y a donc à peu près jamais à faire des forks dans son compte perso pour faire des modifs, puis des PR.
Il est bien mieux, à la fois plus simple pour soi *mais aussi pour les autres*, de faire une branche de dév directement dans le projet de départ (puisque tu as les droits) : cela permet à tout le monde de tester tes modifs uniquement en changeant de branche, sans devoir cloner un autre dépôt.
Donc tu fais :
git branch dev/super_modif
ou
git branche deb/issue_123 (s'il y a une demande déjà créé pour ça)
Et ensuite tout le monde pourra tester tes modifs en faisant
git checkout dev/super_modif
directement sur la même installation du plugin
Les PR peuvent être faites tout aussi facilement depuis une branche interne au même projet.
je vois que tu fork le dépôt, c'est un fonctionnement qui n'est pas encore expliqué très clairement (our bad) : sur Git comme sur l'ancienne zone SVN, tout le monde a les mêmes droits sur tous les dépôts communautaires. Il n'y a donc à peu près jamais à faire des forks dans son compte perso pour faire des modifs, puis des PR.
Il est bien mieux, à la fois plus simple pour soi *mais aussi pour les autres*, de faire une branche de dév directement dans le projet de départ (puisque tu as les droits) : cela permet à tout le monde de tester tes modifs uniquement en changeant de branche, sans devoir cloner un autre dépôt.
[...]
Les PR peuvent être faites tout aussi facilement depuis une branche interne au même projet.
Oups oui, pardon, au temps pour moi. Je n'avais pas pensé à vérifier
sur d'autres dépôts quel était l'usage en vigueur. Pas de problème,
je n'avais pas encore validé la PR. Je fais de ce pas une branche dev
directement sur le dépôt principal.
Camille aurait-il oublié de t'ajouter à l'équipe "Contrib" après la création du compte ?
(
Au passage, ya vraiment des incohérences du "nombre de membres" entre les différentes orgas "de contrib", qui pourtant devraient toutes avoir le même nombre exactement.
spip-contrib-extensions 2 équipes 460 membres
spip-contrib-graphismes 2 équipes 467 membres
spip-contrib-outils 2 équipes 468 membres
spip-contrib-squelettes 2 équipes 467 membres
spip-contrib-themes 2 équipes 467 membres
Et pareil pour le core : spip 33 membres VS galaxie 35 membres
)
Au passage, ya vraiment des incohérences du « nombre de membres » entre les différentes orgas « de contrib », qui pourtant devraient toutes avoir le même nombre exactement.
spip-contrib-extensions 2 équipes 460 membres
spip-contrib-graphismes 2 équipes 467 membres
spip-contrib-outils 2 équipes 468 membres
spip-contrib-squelettes 2 équipes 467 membres
spip-contrib-themes 2 équipes 467 membres
Oui c’est récurrent, Camille refait une synchro régulièrement.
Et pareil pour le core : spip 33 membres VS galaxie 35 membres
)
Non, la c’est logique.
Il y a des personnes hors du Core qui participent à la Galaxie.
Je vous lis et participe peu j'avoue mais pour une fois j'ai un petit
truc à dire.
Tout d'abord il est est de bon ton le 1er janvier de vous souhaiter une
excellente nouvelle année. Ça c'est fait.
Bravo pour le passage sous gitea, j'espère vraiment que cela facilitera
le quotidien des développeurs.
Je voulais juste vous partager une toute petite coquille pour laquelle
je n'ai pas trouvé le repo correspondant : https://git.spip.net/ / tout en bas dans le pied de page / Confimration
d'inscription <https://git.spip.net/MailConfirmationInscription.html>
un r un poil mal placé
