I have here a document (which can't be uploaded for security reasons) containing a few hundred rows with formatted borders. Not all rows have content in it as some of them are placeholders reserving the next rows. I have now hidden the most rows and saved the document and on the next opening all rows containing content in it were still hidden while the empty rows with only a border were not.
Created attachment 127325 [details] Hide rows 1-12, save and reopen This seems to be the same issue I just reported, bug 102188. Attaching a test spreadsheet, hide rows 1-12, save and reopen. Rows 7-11 will be visible upon reopening. Reproduced with v5.2.1.2, not reproduced with v5.1.5.2 / Windows 7. => regression
*** Bug 102188 has been marked as a duplicate of this bug. ***
@Aron: This does not block bug 102154 (i.e. that does not depend on this being fixed) but they may be related.
Hi Markus, It looks like your commit author Markus Mohrhard <markus.mohrhard@googlemail.com> 2016-04-13 10:37:10 (GMT) committer Markus Mohrhard <markus.mohrhard@googlemail.com> 2016-04-14 05:33:01 (GMT) commit 44c1cfc7d8ab7afb1e01ac57d3feeb410ba43748 (patch) tree 8b635afcd1f20c619625df1c9856b64eafb94ae6 parent ea6196f0a51d1bf4cd722468406dcc8c64c7435c (diff) no need to call rowHidden and rowFiltered repeatedly, tdf#90659 Introduced this regression. Could you please take a look whenever you have some time? Regards
Adding Cc: to Markus Mohrhard
Stupid bug by me. Fix is trivial.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7e7f6b0a056b51a425934f63745ae5c9fed2e51f tdf#101843, ensure repeated rows set the hidden/filtered flag correctly 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.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7cf444454c0c27e2f6d764164ea880b87163f45a add test for tdf#101843 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.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-5-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=3b0a57d6de5931fcbd554a11cb9abdd09c34a735&h=libreoffice-5-2 tdf#101843, ensure repeated rows set the hidden/filtered flag correctly It will be available in 5.2.3. 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.