Le laboratoire de Darathor

Bonjour,

Ce mod m'a l'air des plus interessants
Seulement, son aspect de bêta me rebute un peu, étant donné que je souhaite l'utiliser sur un mon forum (TerresLegends) qui ne me sert pas de test, et j'aimerais éviter de faire des betises.

Je n'ai pas encore eu le temps de le tester de fonds en comble dans mon environement de développement (easyphp ), et c'est pourquoi je souhaiterais avoir quelques remontés sur des utilisateurs l'utilisant et l'avis du coddeur lui même ^^.

ManiaGames

Alors je l'ai installé cet après-midi sur les 3 forums tournant sur ce serveur etqui partagent le même script (donc celui-ci, ~lien~ et ~lien~ ) et pour pour l'instant, je n'ai constaté aucun problème, mais bon, on est pas à l'abri d'un oubli quelque part, faudrait un peu plus de temps avant d'être catégorique

Par contre, j'ai constaté aussi que niveau base de données, ça allège très peu (moins que je ne le pensais )... Sur mon gros forum, la table des textes des posts fait dans les 115Mo et je n'ai gagné qu'1Mo environ...

Autre truc, apparemment, il vaut mieux réordonner la table des textes des posts après parce que sinon la partie index prend un place folle (à cause des trous laissés par l'effacement de la colone je pense)...

Je suis donc en train de l'installer, mais un détail me tracasse :

Tu nous demande de supprimer la partie rouge dans le code suivant :



N'est-ce-pas génant de garder la case 1 vide ?

Salut

Faudrait peut etre laisser le break; au moins

Nope en fait, ce cas ne devrait plus arriver normalement, vu qu'il correspond au cas du tri par sujet de posts (cas que j'ai supprimé du menu déroullant).

Mais au cas où, je l'ai laissé (Et comme ce qui suit c'est le tri par titre de topic, c'est ce qu'il y a de plus proche)

Rappel : Dans un switch, on cherche le case correspondant puis on exécute tout jusqu'au break;, donc là, ça fait le même traitement pour les deux cas

Oui dans ce cas c'est bon
Au début je pensais a carrément supprimer le case mais bon... pas très utile c'est vrai :P

Ouaip, c'est une solution aussi en effet, mais si jamais on appelle ce cas ailleurs que là où je l'ai enlevé, le default correspond moins, donc c'est mieux comme ça je pense

J'avais oublié de modifier le fichier modcp.php pour tenir compte de la suppression du champ (donc ça ne se remarquait qu'en cas de tentative de division de sujet, ce qui est relativement rare, c'est pour ça que je ne m'en suis pas rendu compte plus tôt ).

La version TEST - B corrige cet oubli (seules les modifications sur "modcp.php" et "templates/subsilver/modcp_split.tpl" se sont rajoutées, tout le reste est inchangé)

Merci