Bug 100517

Summary: Comment-only cells not moving with data when sorting
Product: LibreOffice Reporter: David Remelts <remeltdb>
Component: CalcAssignee: Dennis Francis <dennisfrancis.in>
Status: RESOLVED FIXED    
Severity: normal CC: dennisfrancis.in, miguelangelrv
Priority: medium    
Version: 5.1.3.2 release   
Hardware: All   
OS: All   
Whiteboard: target:5.3.0 target:7.2.0
Crash report or crash signature: Regression By:

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.