Bug 100517 - Comment-only cells not moving with data when sorting
Summary: Comment-only cells not moving with data when sorting
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
5.1.3.2 release
Hardware: All All
: medium normal
Assignee: Dennis Francis
URL:
Whiteboard: target:5.3.0 target:7.2.0
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-21 13:58 UTC by David Remelts
Modified: 2021-02-18 14:13 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description David Remelts 2016-06-21 13:58:35 UTC

    
Comment 1 David Remelts 2016-06-21 14:17:39 UTC
If a sort range contains a column with cells that have comments embedded in them and there are NO cells in that column containing numbers, text, formulas, etc, the column with the comments won't be sorted with the rest of the data.  
So, the comments will no longer be associated with the original row.

Example:

Pre sort:
Col A      Col B
Text A     Comment1
Text B     Comment2
Text C     Comment3

After sort on Col A:
Col A      Col B
Text C     Comment1
Text B     Comment2
Text A     Comment3

Note: This may be related to Bug 42705 & 45471
Comment 2 m_a_riosv 2016-06-21 21:31:02 UTC
Reproducible
Win10x64
Version: 5.1.4.2 (x64)
Build ID: f99d75f39f1c57ebdd7ffc5f42867c12031db97a
CPU Threads: 1; OS Version: Windows 6.19; UI Render: GL;
Comment 3 Commit Notification 2016-10-24 11:29:14 UTC
Dennis Francis committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=258d039798b9ca4cc95a14d62e30bb725ad2c8ec

tdf#100517 : Optionally include comment-only columns when sorting

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 4 Commit Notification 2021-02-18 14:13:18 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/4e36ff33245ef1bb7cfb1f2d86d84736d2ceeaa2

tdf#100517: sc: Add UItest

It will be available in 7.2.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.