Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#29 closed amelioration (fixed)

Contenu du sujet de discussion en paramètre

Reported by: fabien Owned by: youp3
Priority: mineur Milestone: 2.2
Component: e2t Version:
Keywords: Cc:

Description

Passer le contenu du sujet de discussion dans les settings

Change History (31)

comment:1 Changed 16 years ago by fabien

  • Type changed from probleme to amelioration

comment:2 Changed 16 years ago by youp3

Peux-tu être plus précis Fabien car je ne vois pas en quoi consiste cette demande.

comment:3 Changed 16 years ago by fabien

On en avait déjà parlé sur l'ancien tracker.
Ça consiste à passer le template topic_text dans les paramètres de l'E2T.

comment:4 Changed 16 years ago by fabien

  • Milestone changed from 2.3 to 2.2

comment:5 Changed 16 years ago by youp3

Fabien,

quand penses-tu pouvoir traiter ce ticket ? Parce que si cela demande trop de temps, on le repasse sur la milestone 2.3 et on peut sortir la 2.2 RC1

comment:6 Changed 16 years ago by youp3

Au fait, c'est quoi ce "sujet de discussion" ? Parce que je peux éventuellement m'en charger, mais je ne vois pas ce que c'est.

comment:7 Changed 16 years ago by fabien

C'est le contenu des AutoTopics?. Il est actuellement dans un template au lieur d'être dans les settings. Si tu veux t'en occuper, je suis pas contre :)

comment:8 Changed 16 years ago by youp3

Je viens de regarder les templates. Il s'agit bien de topic_text ? Je ne comprends décidément pas où tu veux aller en mettant ceci dans les settings. Qu'est-ce que cela va apporter de déplacer ceci des templates dans les settings ? Plus facile (intuitif à personnalisé) ? En l'état actuel, je suis incapable de traiter ce ticket.

comment:9 Changed 16 years ago by fabien

Oui, c'est bien topic_text. Pour moi, c'est pas logique de mettre ça dans les templates car on va pas le changer selon de skin. Une fois que le sujet est créé c'est tout. Je pense aussi aux personnes qui ont plusieurs skins, s'ils modifient le format du sujet, ils devront le faire sur tous les skins (ou alors ils auront des sujets de discussion différent selon le skin qu'utilise le 'posteur')

comment:10 Changed 16 years ago by youp3

  • Owner changed from somebody to youp3
  • Status changed from new to assigned

Bon aller, je me lance sur ce ticket!

comment:11 Changed 16 years ago by youp3

J'ai besoin de votre avis. Il faut que je crée une fonction qui devra construire le contenu du sujet pour pouvoir être ensuite passé en paramètre aux APIs de topic. Sauf que j'hésite où créer cette fonction. J'ai pensé dans le fichier API mais finalement je ne pense pas que cela soit judicieux. Peut-être dans le fichier class_e2t_functions.php ?

comment:12 Changed 16 years ago by fabien

Moi, je le mettrais dans class_e2t_functions.php :)

comment:13 Changed 16 years ago by fabien

Je comprends pas ton paramètre ID 266 (Les champs perso). Pourquoi tu prends pas l'affichage défini lors de l'ajout d'un champ ?

comment:14 Changed 16 years ago by anonymous

J'ai pas mal fait de tests avant d'en arriver à utiliser ce paramètre. Il est vrai que chaque champ personnalisé dispose de sa propre définition d'affichage. Je trouvais que c'était plus simple d'avoir quelque chose de générique notamment pour ajouter le <br /> supplémentaire quand les champs perso sont renseignés. Parce que pour la mise en page, c'est quand même pas pratique d'en définir une partie dans les paramètres et une autre dans la gestion des champs perso.

Maintenant si vous trouvez qu'il vaut mieux supprimer cela, pourquoi pas.

comment:15 Changed 16 years ago by youp3

Ok, j'ai trouvé pour les problèmes de retour à la ligne. Et suite à de nouveaux tests, je vais faire sauter le paramètre ID 266

comment:16 Changed 16 years ago by prolag

Euh Pourquoi tu charge la lib bbcode ? car elle est deja chargé dans dl_func...

comment:17 Changed 16 years ago by youp3

C'est une bonne question, il faut que je regarde :)

comment:18 Changed 16 years ago by youp3

Il y a surement quelque chose à faire mais là je patauge et n'y arrive pas. Je te laisse regarder ça.

comment:19 Changed 16 years ago by fabien

Apparemment, il suffit de ne pas la charger ainsi que e2t_functions :p

comment:20 Changed 16 years ago by youp3

Ne pas la charger dans quel fichier ? Pour moi dans class_e2t_tool.php il faut laisser comme c'est actuellement. Maintenant il reste le cas du fichier FileCP.php

comment:21 Changed 16 years ago by Prolag

J'ai fait un essai chez moi et on dirait que ca marche :) Je vais faire un commit avec les differents modifications ;)

comment:22 Changed 16 years ago by fabien

oui, j'ai essayé chez moi aussi mais très rapidemment :)

comment:23 Changed 16 years ago by youp3

Prolag : avec ta révision [209], on peut considérer que ce ticket est terminé (il faut que je fasse le ménage dans le code tout de même avant).

comment:24 Changed 16 years ago by Prolag

Normalement c'est bon Il faudrait bien verifier que le parser marche bien (pas de bug bizarre..)

comment:25 Changed 16 years ago by fabien

Reste au moins la template a supprimer :p

comment:26 Changed 16 years ago by youp3

Bonne remarque concernant cela ! Pour une nouvelle installation, c'est simple puisqu'il suffit de l'enlever du fichier des templates. Par contre pour une mise à jour, cela se passe comment pour supprimer le vieux template ?

comment:27 Changed 16 years ago by fabien

Pour supprimer un template pendant la mise à jour, regarde le fichier sql.php du dossier de mise à jour (tout à la fin) ;)

comment:28 Changed 16 years ago by youp3

  • Resolution set to fixed
  • Status changed from assigned to closed

Ok Fabien, j'ai donc ajouté la requête supplémentaire. Le ménage est fait. Ticket terminé par la livraison [213]

comment:29 Changed 16 years ago by fabien

Le template est toujours là dans le .xml :p

comment:30 Changed 16 years ago by fabien

Je l'ai fais dans la révision 215 :)

comment:31 Changed 16 years ago by youp3

Ok, merci. J'avais effectivement oublié le fichier xml

Note: See TracTickets for help on using tickets.