Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

forum avec formulaire pour un groupe #257

Closed
previnfo opened this issue Oct 24, 2017 · 6 comments
Closed

forum avec formulaire pour un groupe #257

previnfo opened this issue Oct 24, 2017 · 6 comments

Comments

@previnfo
Copy link
Contributor

Dans le cadre d'un futur Projet d'amélioration :
Un groupe doit pouvoir bénéficier d'un forum avec formulaire en développant plus judicieusement la table SQL pour une meilleure répartition des variables (types de forum). En effet, actuellement "$forum_pass" concentre en lui seul, les variables type (texte étendu, groupe, caché, privé, public ...) ce qui a pour effet de brider cette option comme d'autre.
@+

@jipexu
Copy link
Member

jipexu commented Oct 24, 2017

oui ! cette partie du code est à parfaire ... quand le fait on ? ... car cela fait rajouter au moins 1 champ dans la bd .... et bien entendu modifier le code dans plusieurs fichiers ...

@previnfo
Copy link
Contributor Author

previnfo commented Oct 25, 2017

Oui, en fin de semaine cette petite modif, je vais essayer de la faire (histoire de me remettre dans le bouillon) \ (sous la bienveillance de ton contrôle)... C'est le genre de bricolage qui, jadis, était à ma portée !!! Bon maintenant, c'est probablement une autre aventure (bien moins évident)

Donc, en effet, une colonne (champ) dans la table que l'on pourrai définir comme $forum_etend a rajouter dans phpbbforum.php dans les lignes 296, 524, 576, 607.
Un peu plus complexe (pour moi) ce sera l’instruction de l'apport conditionnel (if) en ligne 554.

Puis de simples rajout du nom du champ:
-pour admin.php dans les lignes 599 & 603
-pour viewtopic.php dans la ligne 516
-pour viewtopic.phpH dans la ligne 461

  • pour viewforum dans la ligne 389

Les metamots ForumL & ForumP à mettre à jour et bien Evidemment updape de la SQL.
Y a peut être encore certain trucs ?? Afaire à suivre ;-)

@jipexu
Copy link
Member

jipexu commented Oct 25, 2017

il y potentiellement 18 fichiers à modifier ! plus la bd courante ... ça fait du boulot ! ...

@previnfo
Copy link
Contributor Author

previnfo commented Oct 25, 2017

It's true, !!
C'est très probable, avec un raisonnement très simpliste je me suis juste, dans un premier temps, préoccupé du champ complémentaire à celui de "$forum_pass.." ;-)))
Ouch !! En regardant plus sereinement /profondément la construction de ce code avec "$forum_type" cela fait bien un minimum de 20 fichiers à restructurer !!!

@jipexu jipexu added this to Triage in beta 2 May 24, 2018
@stale
Copy link

stale bot commented Sep 28, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Sep 28, 2018
@stale stale bot closed this as completed Oct 6, 2018
beta 2 automation moved this from Triage to done Oct 6, 2018
@jipexu jipexu removed this from done in beta 2 Oct 9, 2018
@jipexu jipexu reopened this Oct 27, 2018
@stale stale bot removed the wontfix label Oct 27, 2018
@stale
Copy link

stale bot commented Dec 26, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Dec 26, 2018
@stale stale bot closed this as completed Jan 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Amélioration Code
  
Awaiting triage
Development

No branches or pull requests

2 participants