Bug 125414 - FILESAVE XLSX Ellipses of the “Mark invalid data” function shouldn't be exported when saving XLSX
Summary: FILESAVE XLSX Ellipses of the “Mark invalid data” function shouldn't be expor...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
3.5.0 release
Hardware: All All
: medium normal
Assignee: Szabolcs Toth
URL:
Whiteboard: target:7.1.0
Keywords: filter:xlsx
Depends on:
Blocks: Cell-Validity XLSX-Shapes
  Show dependency treegraph
 
Reported: 2019-05-21 11:08 UTC by NISZ LibreOffice Team
Modified: 2020-09-14 12:03 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
The original file created with LibreOffice Calc. (8.04 KB, application/vnd.oasis.opendocument.spreadsheet)
2019-05-21 11:13 UTC, NISZ LibreOffice Team
Details
The file exported as XLSX. (12.80 KB, application/vnd.openxmlformats-officedocument.spreadsheetml.sheet)
2019-05-21 11:13 UTC, NISZ LibreOffice Team
Details
A screenshot showcasing the issue. (125.90 KB, image/png)
2019-05-21 11:14 UTC, NISZ LibreOffice Team
Details

Note You need to log in before you can comment on or make changes to this bug.
Description NISZ LibreOffice Team 2019-05-21 11:08:31 UTC
Description:
The Mark invalid data function (Tools > Detective > Mark invalid data) can be used to mark invalid data in a validated cell with a red ellipse. This function also exists in Microsoft Excel, but Excel turns this function off automatically when saving the spreadsheet as XLSX. Calc on the other hand saves it as a regular, non-functioning ellipse shape in XLSX. It would probably make more sense to drop it entirely when saving the spreadsheet as XLSX, similarly how Excel turns this function off when saving.

Steps to Reproduce:
    1. Create a new spreadsheet.
    2. Fill the A1:A3 cells with the following numbers: 1, 5, 15 and the B1:B3 cells with the following numbers: 12, 8, 16.
    3. Select the A1:B3 cells and click on Data > Validity.
    4. In the “Validity” window set the following criteria: “Allow: Whole numbers”, “Data: less than”, “Maximum: 10” and click on OK.
    5. Turn on the “Mark invalid data” function by clicking on Tools > Detective > Mark invalid data – the A3, B1 and B3 cells should have red ellipses around them.
    6. Save the file as XLSX and reload it by clicking on File > Reload.

Actual Results:
The ellipse around the B3 cell is converted to a regular, non-functioning ellipse shape in XLSX, while the ellipses that were around the A3 and B1 cells are lost – presumably because parts of those shapes were outside the sheets boundary. If the same XLSX is opened in Microsoft Excel 2013 all three red ellipses are visible.

Expected Results:
Ellipses shouldn't be exported when saving the spreadsheet as XLSX.


Reproducible: Always


User Profile Reset: No



Additional Info:
LibreOffice details:
Version: 6.3.0.0.alpha1+ (x64)
Build ID: c3245cfd54f391111afb424867ab483096d9de5e
CPU threads: 4; OS: Windows 6.3; UI render: GL; VCL: win; 
TinderBox: Win-x86_64@42, Branch:master, Time: 2019-05-19_22:59:06
Locale: hu-HU (hu_HU); UI-Language: en-US
Calc: threaded

Already happens in: 
LibreOffice 3.5.0rc3 
Build ID: 7e68ba2-a744ebf-1f241b7-c506db1-7d53735
Comment 1 NISZ LibreOffice Team 2019-05-21 11:13:24 UTC
Created attachment 151554 [details]
The original file created with LibreOffice Calc.
Comment 2 NISZ LibreOffice Team 2019-05-21 11:13:48 UTC
Created attachment 151555 [details]
The file exported as XLSX.
Comment 3 NISZ LibreOffice Team 2019-05-21 11:14:31 UTC
Created attachment 151556 [details]
A screenshot showcasing the issue.
Comment 4 Xisco Faulí 2019-05-21 12:36:25 UTC
Reproduced in

Version: 6.3.0.0.alpha1+
Build ID: 38ac0586448d4f07811b139f62f62686b029feba
CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US
Calc: threaded
Comment 5 Commit Notification 2020-09-14 11:58:11 UTC
Szabolcs Toth committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/14b40ec7be7ca8315848034591e3c3a246d5a8dd

tdf#133688 tdf#125414 XLSX: don't export tracer arrows

It will be available in 7.1.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.