nommages trop génériques
Les nommages dans ce plugin sont trop génériques et risquent d'entrer en conflit avec d'autres plugins, d'autres librairies ou le noyau de spip... Du coup ça interdirait à ton plugin de cohabiter avec dautres ressources... voir de fonctionner sur SPIP un jour ! Faut anticiper ça car c'est bien plus facile à gérer au début du développement...
Le pire, selon les voix qui s'élèvent sur irc, c'est la fonction bouton_ajouter
. Il faut vraiment la renommer, avoir un nom plus spécifique au projet de ce plugin.
Ensuite il y a aussi le nom et le préfixe de ce plugin : "ajouter". Et la balise #BOUTON_GOOGLE (google c'est tellement vaste qu'il y a plein de possibilités de boutons).
Donc il faudrait utiliser des noms qui indiquent plus précisément le contexte et la fonction réalisée. Ou utiliser un préfixe pour tout ce qui a trait à ce plugin.
Par exemple (juste des suggestions mais ya le choix) :
- le préfixe et le nom du plugin pourraient être "ajoutergooglecalendar".
- Et alors la fonction pourrait s'appeler
agc_bouton_ajouter()
, et les boutons #BOUTON_AGC_GOOGLE et #BOUTON_AGC_ICS...