Recent

Author Topic: minor fpspreadsheet.pas typo  (Read 6989 times)

bobo

  • Full Member
  • ***
  • Posts: 171
minor fpspreadsheet.pas typo
« on: June 16, 2011, 08:35:58 am »
  lpUnsupportedReadFormat := 'Tryed to read a spreadsheet using an unsupported format';
  lpUnsupportedWriteFormat := 'Tryed to write a spreadsheet using an unsupported format';

Tryed should be Tried

Btw, shouldn't non-supported export formats be commented out or something so at compile time we could see that for example there is no .csv (sfCSV in TsSpreadsheetFormat) export supported?

felipemdc

  • Administrator
  • Hero Member
  • *
  • Posts: 3538
Re: minor fpspreadsheet.pas typo
« Reply #1 on: June 16, 2011, 10:01:35 am »
Thanks, I fixed the string.

Quote
Btw, shouldn't non-supported export formats be commented out or something so at compile time we could see that for example there is no .csv (sfCSV in TsSpreadsheetFormat) export supported?

Not possible because support for formats is added at runtime. But anyway, the exception should be pretty clear about what the problem was.

Having said that, CSV exporting should be trivial to implement, if you would like support for it please open a bug report.
« Last Edit: June 16, 2011, 10:10:32 am by felipemdc »

bobo

  • Full Member
  • ***
  • Posts: 171
Re: minor fpspreadsheet.pas typo
« Reply #2 on: June 16, 2011, 11:19:57 pm »
That is why I mentioned CSV , because it is not supported (runtime or otherwise). No "support for formats is added at runtime" for CSV.
Should not "sfCSV" be commented out from the definitions until there is a possibility to add it at runtime?

I just checked CSV and using the excel8 exports without a problem, so I do not know if any of the other formats are not supported yet.

felipemdc

  • Administrator
  • Hero Member
  • *
  • Posts: 3538
Re: minor fpspreadsheet.pas typo
« Reply #3 on: June 17, 2011, 12:45:15 pm »
There is a table showing the currently supported formats here:

http://wiki.lazarus.freepascal.org/FPSpreadsheet#Current_Progress

 

TinyPortal © 2005-2018