[Lazarus] Patch for Grids.pas - Adds soVisibleOnly to SaveOptions

Alexsander Rosa alexsander.rosa at gmail.com
Tue Oct 16 19:51:33 CEST 2012


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 <jesusrmx at yahoo.com.mx>

>
> >- 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus-ide.org/pipermail/lazarus/attachments/20121016/e5078d78/attachment-0003.html>


More information about the Lazarus mailing list