search

AntonyCorbett / JWLMerge

Utility to merge jwlibrary backup files
MIT License
194 stars 33 forks source link

Export #61

Closed Chiriat closed 2 years ago

Chiriat commented 2 years ago

Hi bro Anthony, i am trying to export bible notes to excel and i always get this error message.

Thanks for your help

System.ArgumentOutOfRangeException: Cells can hold a maximum of 32,767 characters. (Parameter 'cellValue') at ClosedXML.Excel.XLCell.SetInternalCellValueString(String cellValue, Boolean validate, Boolean parseToCachedValue) at ClosedXML.Excel.XLCell.SetValue[T](T value, Boolean setTableHeader) at ClosedXML.Excel.XLCell.SetValue[T](T value) at JWLMerge.ExcelServices.ExcelService.SetCellStringValue(IXLWorksheet worksheet, Int32 row, Int32 col, String value) at JWLMerge.ExcelServices.ExcelService.AppendToBibleNotesFile(String excelFilePath, IReadOnlyCollection`1 notes, Int32 startRow, String backupFilePath) at JWLMerge.BackupFileServices.BackupFileService.ExportBibleNotesToExcel(BackupFile backupFile, String bibleNotesExportFilePath, IExcelService excelService) at JWLMerge.ViewModel.MainViewModel.<>c__DisplayClass84_0.b0() at System.Threading.Tasks.Task.InnerInvoke() at System.Threading.Tasks.Task.<>c.<.cctor>b272_0(Object obj) at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state) --- End of stack trace from previous location --- at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state) at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread) --- End of stack trace from previous location --- at JWLMerge.ViewModel.MainViewModel.ExportBibleNotesAsync(String filePath)

AntonyCorbett commented 2 years ago

@Chiriat Hi. Yes, I saw your crash in the App Center dashboard. I have fixed this by truncating any notes that are too long and then displaying a notification. At least the export continues. The code is already in the repo, but I have not published a new release yet. Look out for v2.0.0.10+ which I should get done at the weekend. Regards, Antony

AntonyCorbett commented 2 years ago

@Chiriat v2.0.0.10 is now available. Since long notes are now truncated, the export should work but of course the excel file will contain truncated notes!

Chiriat commented 2 years ago

thanks a lot