search

AlternC / alternc-mailman

Mailman module for AlternC
2 stars 10 forks source link

100 customizable mailman url broken v1 #20

Closed sebas891 closed 6 years ago

sebas891 commented 6 years ago

Lors de la création de la liste dans l'interface web, ça passe ici: https://github.com/AlternC/alternc-mailman/blob/bee46f6cd22744c2db08f104c89d595d54be3941/bureau/class/m_mailman.php#L248

ça prend la variable globale ici : $L_FQDN https://github.com/AlternC/alternc-mailman/blob/bee46f6cd22744c2db08f104c89d595d54be3941/bureau/class/m_mailman.php#L249

Le INSERT dans la table mailman se trouve ici, avec comme paramêtre $L_FQDN dans le champs URL https://github.com/AlternC/alternc-mailman/blob/bee46f6cd22744c2db08f104c89d595d54be3941/bureau/class/m_mailman.php#L307

Donc, on a déjà de rempli le champs url avant la création de la liste dans mailman. Peut-être qu'on devrait viser directement dans là aller mettre la bonne valeur dans URL, utiliser $MAILMAN_URL à la place de $L_FQDN

Quand le script update_mailman.sh roule c'est ici que ça boucle dans les listes a créer. https://github.com/AlternC/alternc-mailman/blob/78176a87ddc8ff2b907f82c5f73878c5d2d670ed/src/update_mailman.sh#L43

Ça fait une configuration du URL seulement dans mailman et non pas dans la db d'alternc, dans la table mailman...

Mais bon, donc, au lieu de laisser le status de la création à "OK", je le mets à "SETURL" comme ça la boucle plus bas va configurer le URL correctement.

Il reste que dans la boucle plus bas, https://github.com/AlternC/alternc-mailman/blob/78176a87ddc8ff2b907f82c5f73878c5d2d670ed/src/update_mailman.sh#L169

Il manque toujours un INSERT pour mettre à jour la db pour le url. C'est ça que j'ai ajouté.

Le code dans le "create list" devrait être modifié parce qu'il duplique en partie la configuration de l'URL... (ça serait un autre PR)

sebas891 commented 6 years ago

puf! j'ai 11 commit, c'était pas l'idée ici... je ferme encore. Désolé du bruit.