Re: RE : RE : RE : [sdx-developers] sdxtest attend la critique ;-)

[Pierrick Brihaye]

Bonjour,

Rasik Pandey a écrit:

> AbstractIndexableDocument.java  ligne 133
> AbstractIndexableDocument.java  ligne 626
> LuceneDocumentBase.java                 ligne 1503
> LuceneDocumentBase.java                 ligne 1927

Difficile pour moi d'utiliser des numéros de ligne sur le vieil état CVS 
dont je dispose. Peu importe...

Je suis d'accord. On ne fait pas forcément appel à repository ; ça peut 
être storeRepo obtenu grâce à getRepositoryForStorage.

Mais dans ce cas, pourquoi appeler une méthode avec un argument qui sera 
peut-être modifié et ce, d'autant plus que l'on sait déjà comment il le 
sera ?

Ensuite, getRepositoryForStorage va chercher le storeRepo 
d'AbstractDocument et ce storeRepo est positionné via 
setRepositoryForStorage. OK.

Et là, ceci explique peut-être ma mauvaise interprétation, le seul appel 
à cette méthode se trouve dans Abstract*Indexable*Document. Je l'avais 
vainement cherché dans, par exemple, binaryDocument.

L'appel se fait dans startElement qui est appelée lors de l'indexation 
du document (indexable).

Wow !

On n'aurait pas pu faire plus simple ?

De plus, est-ce que le couple id/repository n'est pas *indissociable* 
dans SDX ce qui devrait donner lieu, IMHO à un constructeur prenant les 
2 arguments  ?

A terme, comment envisager un truc du genre :

<sdx:uploadDocument id="xxx" repository="yyy" url="zzz" 
mimetype="image/jpeg">
  <sdx:relation origin="aaa" target="bbb" type="attached">
  <sdx:behaviour type="replace"/>
</sdx:uploadDocument>

A bientôt,

--
Pierrick Brihaye, informaticien
Service régional de l'Inventaire
DRAC Bretagne
mailto:address@hidden