After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 330129 - Export locking sheet to Excel
Export locking sheet to Excel
Status: RESOLVED FIXED
Product: Gnumeric
Classification: Applications
Component: import/export MS Excel (tm)
git master
Other All
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2006-02-06 14:57 UTC by julien
Modified: 2006-03-14 02:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Excel support (both import/export) for protection of worksheets and workbook. (3.18 KB, patch)
2006-03-11 22:12 UTC, Klokan Petr Pridal
none Details | Review
New version of previous patch - Workbook Window Protection is turned off (3.14 KB, patch)
2006-03-13 12:23 UTC, Klokan Petr Pridal
none Details | Review

Description julien 2006-02-06 14:57:09 UTC
Please describe the problem:
When your sheet in gnumeric is locked, and then you saved as a Ms Excel file 
format, when you open this file in Ms Excel 97, your sheet is doesn't locked.

Steps to reproduce:
1. In gnumeric you lock the sheet, you save
2. then you save as Ms Excel 97/2000
3. You open the new file in Ms Excel 97, and you see that the sheet is not 
locked


Actual results:
The sheet in Ms Excel 97 is not locked

Expected results:
The sheet in Ms Excel 97 is locked

Does this happen every time?
yes

Other information:
Comment 1 Morten Welinder 2006-02-06 19:36:46 UTC
If MS Excel does not see the lock, we must be _ex_porting it incorrectly.
Comment 2 Klokan Petr Pridal 2006-03-11 22:12:14 UTC
Created attachment 61105 [details] [review]
Excel support (both import/export) for protection of worksheets and workbook.

Excel support for protection of worksheets and workbook. Password protection is not yet implemented. Look at Bug 334257 too.
Comment 3 Klokan Petr Pridal 2006-03-13 12:23:54 UTC
Created attachment 61171 [details] [review]
New version of previous patch - Workbook Window Protection is turned off

WINDOWPROTECT (which protects the window setting like size of window, etc.) is now not turned on by workbook protection. I think this is expected behaviour. Just the content of workbook e.g. PROTECT is turned on during export to Excel file. No change on importing was made.
Comment 4 Morten Welinder 2006-03-13 19:23:06 UTC
The main difference between this patch and its xml cousin is that I am
more confident I understand the xml one, :-)

Anyway, shouldn't excel_read_workbook_PROTECT check for (q->length >= 2)
if it is going to read two bytes?  Or is it really supposed to read just
one byte?

Are we *sure* that ewb->gnum_wb_view is not NULL in the code
+	ms_biff_put_2byte (ewb->bp, BIFF_WINDOWPROTECT, 0);
+	ms_biff_put_2byte (ewb->bp, BIFF_PROTECT, ewb->gnum_wb_view->is_protected ? 1 : 0);

?  And what if there is more than one view?  I notice that code a few lines
later actually loops through views.
Comment 5 Morten Welinder 2006-03-14 02:29:19 UTC
Fixed in cvs, thanks.