In that case, I would prefer to add another parameter to *SaveToCSVFile*:
boolean "VisibleColumnsOnly" with default false.
The procedure *LoadFromCSVFile* requires no change, as the CSV does not
have visibility information.

2012/10/16 Jesus Reyes <[email protected]>

>
> >- adds soVisibleOnly to SaveOptions
> >- with soVisibleOnly saves only visible columns
> -> without (default) it works as before
>
>
> I think using some SaveOptions options only for XML and some only for CSV
> might cause confusion.
>
> The problem with SaveOptions is that it's supposed to apply only to
> Save/LoadFromFile (I wish to have named SaveToCSVFile as ExportToCSVFile
> instead) visibility on CustomColumns should be preserved on saving/loading
> to/from a XML file (that by the way is missing), but it's ok to export only
> visble columns to  CSV file. So what about adding yet another parameter to
> CSV functions?,
>
> Even, if more features continue to be added to CSV support, it may be
> better to move the whole CSV support out of grids unit and create a
> component or something.
>
> What you think.
>
> Jesus Reyes A.
>
>

-- 
Atenciosamente,
Alexsander da Rosa
--
_______________________________________________
Lazarus mailing list
[email protected]
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to