Le 26/02/2013 21:17, Eric Chatellier a écrit :
Ok, merci pour ce retour. Je me doutais bien que le model (load()) ne servait
pas a grand chose.
Avec un simple PropertyModel, c'est tout de suite plus propre.
Tu peux d'ailleurs le passer en paramètre de ton ReadListDocumentsPanel au
lieu du modèle de molécule. Ce serait plus cohérent : le panel sert à afficher
le modèle de ce qu'il doit afficher.
En fait je me sers quand même de la molécule pour ajouter les documents dessus
quand on passe
en édition. Je trouves le couplage avec l'entité sauvegardée en base plus fort
qu'une simple
liste (lazy) qui pourrait peut être poser d'autres problèmes.
C'est ok pour moi comme tu l'as écris. Je t'avais répondu pour un cas plus général et c'est sûr que dans ce cas précis on a besoin d'accéder aux documents de l'entité en modification.

Du coup, l'interface DocumentAttachable (sur les entités sur lequelles ont peut
ajouter
des documents) ne servirait pas non plus ?
Je n'ai pas vu la manière dont tu t'en servais avant mais l'ajout d'une interface me semble être une bonne idée car elle apporte plus de clarté au niveau du modèle. Par ailleurs, elle permet de contraindre l'ajout de document aux seules entités qui puissent en avoir (IModel<? extends DocumentAttachable> que tu as mis en signature du constructeur ManageDocumentPage).

--
Adrien Cheype
Ingénieur en Systèmes d'Information
Service « Informatique Scientifique et Appui aux Partenaires du Sud »
Direction du Système d'Information (DSI)
http://www.ird.fr/dsi/
http://www.ird.fr/informatique-scientifique/

INSTITUT DE RECHERCHE POUR LE DEVELOPPEMENT
BP A5 - 98848 Nouméa - Nouvelle Calédonie
Tél. +687 260 789