The Wayback Machine - https://web.archive.org/web/20200903004822/https://github.com/SheetJS/sheetjs/issues/1967
Skip to content

/ sheetjs

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Excel crashes when printing XLS from https://sheetjs.com/demo/table.html #1967

Open
lwinsor opened this issue May 25, 2020 · 4 comments
Open

Excel crashes when printing XLS from https://sheetjs.com/demo/table.html #1967

lwinsor opened this issue May 25, 2020 · 4 comments
Labels
good first issue

Comments

@lwinsor
Copy link

@lwinsor lwinsor commented May 25, 2020

Steps to produce the problem:

  1. From https://sheetjs.com/demo/table.html click [Export to XLS!]
  2. For comparison, export XLSX and XLSB too.
  3. Open the file in Excel 365. In Excel, click [Enable Editing].
  4. Ctrl-P to print. ** CRASH **
    OR
  5. From https://sheetjs.com/demo/table.html click [Export to XLS!]
  6. Open the file in Excel 2010. Excel title bar shows 'compatibility mode'.
  7. Ctrl-P to print. ** CRASH **

The same process does NOT crash if I choose XLSX or XLSB.

The problem details from Excel show fault module mso20win32client.dll for Excel 365 or MSVCR90.dll for Excel 2010.

Many thanks!

Bug seen on:
Windows 7, Chrome, Excel 365
Windows 7, Chrome Excel 2010 32-bit
@lwinsor
Copy link
Author

@lwinsor lwinsor commented May 25, 2020

Also seen on Windows 10.
@zzfzfz
Copy link

@zzfzfz zzfzfz commented Aug 31, 2020

the same bug.
Excel crashes when printing XLS
Is there any solution？@lwinsor @SheetJSDev
@SheetJSDev
Copy link
Contributor

@SheetJSDev SheetJSDev commented Aug 31, 2020

One-line fix (feel free to send a PR): https://github.com/SheetJS/sheetjs/blob/master/bits/78_writebiff.js#L248

-	if(b8 && _WB.Views) write_biff_rec(ba, "Window2", write_Window2(_WB.Views[0]));
+	if(b8) write_biff_rec(ba, "Window2", write_Window2((_WB.Views||[])[0]));
@zzfzfz
Copy link

@zzfzfz zzfzfz commented Aug 31, 2020

it work for me.Thanks @SheetJSDev
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue
3 participants
@SheetJSDev @lwinsor @zzfzfz
You can’t perform that action at this time.