Opened 15 years ago

Last modified 15 years ago

#81 new amelioration

Ré-écriture de la partie création/edition de fichier

Reported by: fabien Owned by: somebody
Priority: critique Milestone: 3.0
Component: e2t Version:
Keywords: Cc:

Description (last modified by prolag)

Il faut revoir complètement les fichiers

sources/classes/e2t/class_e2t_post.php,

sources/classes/e2t/class_e2t_post_new.php,

sources/classes/e2t/class_e2t_post_edit.php et

sources/handlers/han_e2t_post.php.
Je pense que classes/e2t/class_e2t_post.php, classes/e2t/class_e2t_post_new.php et classes/e2t/class_e2t_post_edit.php doivent être fusionné en un seul fichier (trop de code en double ou très similaire). Quand à sources/handlers/han_e2t_post.php il devrait être supprimé, nous n'aurons jamais besoin, à priori, d'avoir un deuxième formulaire d'ajout/édition.

Change History (10)

comment:1 Changed 15 years ago by prolag

  • Description modified (diff)
  • Milestone changed from 2.3 to 3.0

J'ai fait comme Matt.. (ce que j'avais codé avant pour l'ajout et edition de fichier était encore pire... )

Donc je pensais que le code de Matt serai mieux... ;)

comment:2 Changed 15 years ago by prolag

  • Description modified (diff)

comment:3 Changed 15 years ago by prolag

  • Description modified (diff)

comment:4 Changed 15 years ago by fabien

Oui, les deux tickets se joignent. D'ailleurs, j'avais en tête de faire une fonction pour générer un sujet de discussion car le code est répété plein de fois dans l'E2T alors que c'est le même.

comment:5 Changed 15 years ago by fabien

Je crois qu'on va devoir mettre ça pour l'E2T 2.3 :p

Les fichiers actuels ne permettent pas d'avoir une gestion des multi-fichiers correctement.

comment:6 Changed 15 years ago by prolag

Pourquoi ca permet pas une gestion correcte ?

comment:7 Changed 15 years ago by fabien

Parce que ces classes sont prévues pour gérer qu'un seul fichier. Déjà pour mettre le fichier dans une table à part, j'ai eu du mal car je voulais mmettre mon code dans class_e2t_post.php et pas le répéter pour l'ajout et pour l'édit (d'ailleurs, finalement j'ai dû le répéter :'()

Si on veut gérer plusieurs fichier il faudrait quasiment tout ré-écrit (on pourrait aussi "bricoler" pour faire passer ça mais bof bof).

comment:8 Changed 15 years ago by prolag

Euh Tu veux pas attendre que la lib sur la gestion de fichier soit la ? (pour moi c'est le boulot de la lib de faire ca... ajout / suppression)

comment:9 Changed 15 years ago by fabien

Attendons la lib alors :)

comment:10 Changed 15 years ago by prolag

Regarde comment c'est généré dans IPB pas de Left JOIN mais une autre requete qui parcours la table attachements

Note: See TracTickets for help on using tickets.