Re: cvs generator update

[Jean-Baptiste Quenot <jb...@caraldi.com>]

* Philippe LAPLANCHE:

> > > Updates in the source:
> > >
> > > Correction:   The  generator  now   looks  for  the  sitemap
> > > parameter "process-headers" (with the s  at the end) as told
> > > in the documentation
> > >
> > > New  feature:   The  generator  now accepts  a  new  sitemap
> > > parameter "max-records" which allows  to limit the number of
> > > records to read. The default is 0 ( = read all records)
> >
> > Thanks for your contribution.  Have  you been able to enter an
> > entry in  our Jira bug  tracking system?   If not, I  would be
> > glad to submit it myself.
>
> Non je n'ai pas utilis� Jira. Je regarderai � l'occasion mais en
> attendant je veux bien que tu  y mettes le .java en pi�ce jointe
> plut�t que  celui que j'ai  mis sur la mailing  list. Je n'avais
> pas  mis  �  jour  la m�thode  recycle,  elle  remet  maintenant
> this.maxrecords � 0.
>
> Depuis mon post  sur la mailing list cocoon  dev, j'ai d�couvert
> un  autre probl�me  en utilisant  le g�n�rateur  : les  "sitemap
> parameters" du composant sont lus une fois pour toutes.
>
> Je m'explique  par un exemple  car je  d�bute avec Cocoon  et je
> n'ai  pas  d'expert avec  moi  qui  pourrait mieux  formuler  le
> probleme :
>
> A un  premier endroit  d'un pipeline,  dans un  premier matcher,
> j'utilise le cvs generator avec  des param�tres sitemap tels que
> separator=;  A un  autre  endroit j'utilise  le g�n�rateur  avec
> separator=,  Et bien  malheureusement, au  deuxi�me endroit,  le
> separateur utilis� est encore un ; et non un ,
>
> Bon en  r�alit� c'est plut�t  avec le nouveau  sitemap parameter
> que  j'ai  cr��  (c-�-d  max-records)  que  cela  pose  vraiment
> probl�me (dans un map:match, je  fais un preview du fichier csv,
> dans  un suivant  j'importe tout  le  fichier dans  une base  de
> donn�es) mais le probl�me reste le m�me quel que soit le sitemap
> parameter.
>
> Pour  contourner le  probl�me je  ne vois  pas d'autre  solution
> que  de d�clarer  deuxi�me  g�n�rateur dans  la d�claration  des
> composants.

La discussion continue sur users-fr.

J'ai donc  essay� la  modification sur  CSVGenerator, le  patch me
para�t bien.

Par contre en effet je vois  un probl�me dans la gestion du cache:
la m�thode getKey() qui identifie un  object du cache ne prend pas
en compte les param�tres separator, max-records et escape.  Ce qui
explique que  le m�me  r�sultat soit  toujours retourn�  quels que
soient les param�tres.

Retester  avec  le patch  ci-joint.   Si  cela convient,  je  peux
committer dans Cocoon.

Merci,
-- 
Jean-Baptiste Quenot
http://caraldi.com/jbq/