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

Jesus Reyes jesusrmx at yahoo.com.mx
Tue Oct 16 19:33:54 CEST 2012


>- 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.




More information about the Lazarus mailing list