tdf#117276 sc: autofilter was unexpected reset with OK pressed

Change-Id: I53331434bb4d5e2f13e9f2ab5b768e9880aecca8
Reviewed-on: https://gerrit.libreoffice.org/68765
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

tdf#126306: ignore Top10, Empty and NonEmpty in fix for tdf#117276

Regression from 4e7e3670c31329d60f5cc782abc4568c2aba33a7

Change-Id: I034f4f087580aa7c54b92909736af712047251a6
Reviewed-on: https://gerrit.libreoffice.org/75304
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
Reviewed-on: https://gerrit.libreoffice.org/76350
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

4 files changed, 218 insertions, 1 deletions

diff --git a/sc/qa/uitest/autofilter/tdf117276_autofilter_reset.py b/sc/qa/uitest/autofilter/tdf117276_autofilter_reset.py
new file mode 100644
index 000000000000..6d01a4b1fd61
--- /dev/null
+++ b/sc/qa/uitest/autofilter/tdf117276_autofilter_reset.py
@@ -0,0 +1,191 @@
+# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-offset: 4 -*-
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+from uitest.framework import UITestCase
+from uitest.uihelper.calc import enter_text_to_cell
+from libreoffice.calc.document import get_sheet_from_doc
+from uitest.debug import sleep
+from libreoffice.uno.propertyvalue import mkPropertyValues
+from libreoffice.calc.document import get_row
+from uitest.path import get_srcdir_url
+
+
+def get_url_for_data_file(file_name):
+    return get_srcdir_url() + "/sc/qa/uitest/autofilter/data/" + file_name
+
+def is_row_hidden(doc, index):
+    row = get_row(doc, index)
+    val = row.getPropertyValue("IsVisible")
+    return not val
+
+#Bug 117276 - Autofilter settings being reset in some cases
+
+class tdf117276_autofilter_reset(UITestCase):
+    def get_values_count_in_AutoFilter(self, xGridWindow, columnIndex, buttonName = "cancel"):
+        # open filter pop-up window
+        self.assertIsNotNone(xGridWindow)
+        xGridWindow.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", "COL": columnIndex, "ROW": "0"}))
+        xFloatWindow = self.xUITest.getFloatWindow()
+        self.assertIsNotNone(xFloatWindow)
+
+        # get check list
+        xCheckListMenu = xFloatWindow.getChild("check_list_menu")
+        self.assertIsNotNone(xCheckListMenu)
+
+        xTreeList = xCheckListMenu.getChild("check_list_box")
+        self.assertIsNotNone(xTreeList)
+
+        valuesCount = len(xTreeList.getChildren())
+
+        # close pop-up window
+        xButton = xFloatWindow.getChild(buttonName)
+        self.assertIsNotNone(xButton)
+        xButton.executeAction("CLICK", tuple())
+
+        return valuesCount
+
+    def openAutoFilterAndCloseIt(self, xGridWindow, columnIndex, buttonName):
+        self.get_values_count_in_AutoFilter(xGridWindow, columnIndex, buttonName)
+
+    def check_state(self, document, xGridWindow):
+        self.assertFalse(is_row_hidden(document, 0))  # column headers
+        self.assertTrue(is_row_hidden(document, 1))
+        self.assertTrue(is_row_hidden(document, 2))
+        self.assertTrue(is_row_hidden(document, 3))
+        self.assertTrue(is_row_hidden(document, 4))
+        self.assertFalse(is_row_hidden(document, 5))
+
+        self.assertEqual(1, self.get_values_count_in_AutoFilter(xGridWindow, "0"))
+        self.assertEqual(2, self.get_values_count_in_AutoFilter(xGridWindow, "1"))
+
+    def test_run(self):
+        self.ui_test.create_doc_in_start_center("calc")
+        document = self.ui_test.get_component()
+        calcDoc = self.xUITest.getTopFocusWindow()
+        xGridWindow = calcDoc.getChild("grid_window")
+
+#        self.ui_test.execute_dialog_through_command(".uno:Insert")  # insert sheet
+#        xDialog = self.xUITest.getTopFocusWindow()
+#        xOKButton = xDialog.getChild("ok")
+#        xOKButton.executeAction("CLICK", tuple())
+
+        # 1. prepare document
+        #    |    A       | B                      |
+        #  -----------------------------------------
+        #  1 | HEADER-A   | HEADER-B               |
+        #  -----------------------------------------
+        #  2 | Unique a2  | common value for B2:B4 |
+        #  3 | Unique a3  | common value for B2:B4 |
+        #  4 | Unique a4  | common value for B2:B4 |
+        #  5 | Unique a5  | Unique b5              |
+        #  6 | Unique a6  | Unique b6              |
+        #
+        # row-1
+        enter_text_to_cell(xGridWindow, "A1", "HEADER-A")
+        enter_text_to_cell(xGridWindow, "B1", "HEADER-B")
+
+        # row-1
+        enter_text_to_cell(xGridWindow, "A2", "Unique a2")
+        enter_text_to_cell(xGridWindow, "B2", "common value for B2:B4")
+
+        # row-2
+        enter_text_to_cell(xGridWindow, "A3", "Unique a3")
+        enter_text_to_cell(xGridWindow, "B3", "common value for B2:B4")
+
+        # row-3
+        enter_text_to_cell(xGridWindow, "A4", "Unique a4")
+        enter_text_to_cell(xGridWindow, "B4", "common value for B2:B4")
+
+        # row-4
+        enter_text_to_cell(xGridWindow, "A5", "Unique a5")
+        enter_text_to_cell(xGridWindow, "B5", "Unique b5")
+
+        # row-5
+        enter_text_to_cell(xGridWindow, "A6", "Unique a6")
+        enter_text_to_cell(xGridWindow, "B6", "Unique b6")
+
+        xGridWindow.executeAction("SELECT", mkPropertyValues({"RANGE": "A1:B6"}))
+
+        # 2. switch on auto-filter
+        self.ui_test.execute_dialog_through_command(".uno:DataFilterAutoFilter")
+        xDialog = self.xUITest.getTopFocusWindow()
+        xYesBtn = xDialog.getChild("yes")
+        self.ui_test.close_dialog_through_button(xYesBtn)
+
+        # autofilter still exist
+        self.assertEqual(document.getPropertyValue("UnnamedDatabaseRanges").getByTable(0).AutoFilter, True)
+
+        # 3. open filter of column A and deselect first 3 entires (Unique a2, Unique a3, Unique a4)
+        xGridWindow.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", "COL": "0", "ROW": "0"}))
+        xFloatWindow = self.xUITest.getFloatWindow()
+        xCheckListMenu = xFloatWindow.getChild("check_list_menu")
+        xTreeList = xCheckListMenu.getChild("check_list_box")
+
+        xEntry1 = xTreeList.getChild("0")
+        xEntry1.executeAction("CLICK", tuple())   # Unique a2
+        xEntry2 = xTreeList.getChild("1")
+        xEntry2.executeAction("CLICK", tuple())   # Unique a3
+        xEntry3 = xTreeList.getChild("2")
+        xEntry3.executeAction("CLICK", tuple())   # Unique a4
+
+        xOkButton = xFloatWindow.getChild("ok")
+        xOkButton.executeAction("CLICK", tuple())
+
+        # check filtering
+        #    |    A       | B                      |
+        #  -----------------------------------------
+        #  1 | HEADER-A   | HEADER-B               |
+        #  -----------------------------------------
+        #  5 | Unique a5  | Unique b5              |
+        #  6 | Unique a6  | Unique b6              |
+        self.assertFalse(is_row_hidden(document, 0))  # column headers
+        self.assertTrue(is_row_hidden(document, 1))
+        self.assertTrue(is_row_hidden(document, 2))
+        self.assertTrue(is_row_hidden(document, 3))
+        self.assertFalse(is_row_hidden(document, 4))
+        self.assertFalse(is_row_hidden(document, 5))
+
+        self.assertEqual(5, self.get_values_count_in_AutoFilter(xGridWindow, "0"))
+        self.assertEqual(2, self.get_values_count_in_AutoFilter(xGridWindow, "1"))
+
+        # 4. open filter of column B and deselect "Unique b5"
+        xGridWindow.executeAction("LAUNCH", mkPropertyValues({"AUTOFILTER": "", "COL": "1", "ROW": "0"}))
+        xFloatWindow = self.xUITest.getFloatWindow()
+        xCheckListMenu = xFloatWindow.getChild("check_list_menu")
+        xTreeList = xCheckListMenu.getChild("check_list_box")
+
+        xEntry = xTreeList.getChild("0")
+        xEntry.executeAction("CLICK", tuple())   # Unique b5
+
+        xOkButton = xFloatWindow.getChild("ok")
+        xOkButton.executeAction("CLICK", tuple())
+
+        # check filtering
+        #    |    A       | B                      |
+        #  -----------------------------------------
+        #  1 | HEADER-A   | HEADER-B               |
+        #  -----------------------------------------
+        #  6 | Unique a6  | Unique b6              |
+        self.check_state(document, xGridWindow)
+
+        # 4. open filters of column A, B and close it using different buttons
+        self.openAutoFilterAndCloseIt(xGridWindow, "1", "ok")
+        self.check_state(document, xGridWindow)
+
+        self.openAutoFilterAndCloseIt(xGridWindow, "1", "cancel")
+        self.check_state(document, xGridWindow)
+
+        self.openAutoFilterAndCloseIt(xGridWindow, "0", "cancel")
+        self.check_state(document, xGridWindow)
+
+        self.openAutoFilterAndCloseIt(xGridWindow, "0", "ok")
+        self.check_state(document, xGridWindow)
+
+        # finish
+        self.ui_test.close_doc()
+
+
+# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/source/ui/inc/checklistmenu.hxx b/sc/source/ui/inc/checklistmenu.hxx
index 5f58e8015a6c..66ba96cc7564 100644
--- a/sc/source/ui/inc/checklistmenu.hxx
+++ b/sc/source/ui/inc/checklistmenu.hxx
@@ -306,6 +306,13 @@ public:
         {
             return aName < rhs.aName;
         }
+
+        bool operator == (const ResultEntry& rhs) const
+        {
+            return aName == rhs.aName &&
+                   bValid == rhs.bValid &&
+                   bDate == rhs.bDate;
+        }
     };
     typedef std::set<ResultEntry> ResultType;
 
diff --git a/sc/source/ui/inc/gridwin.hxx b/sc/source/ui/inc/gridwin.hxx
index 1775114c27c8..914d7faf03ce 100644
--- a/sc/source/ui/inc/gridwin.hxx
+++ b/sc/source/ui/inc/gridwin.hxx
@@ -24,6 +24,7 @@
 #include "viewutil.hxx"
 #include "viewdata.hxx"
 #include "cbutton.hxx"
+#include "checklistmenu.hxx"
 #include <o3tl/deleter.hxx>
 #include <svx/sdr/overlay/overlayobject.hxx>
 #include <com/sun/star/sheet/DataPilotFieldOrientation.hpp>
@@ -43,7 +44,6 @@ namespace sc {
 class FmFormView;
 struct ScTableInfo;
 class ScDPObject;
-class ScCheckListMenuWindow;
 class ScDPFieldButton;
 class ScOutputData;
 class ScFilterListBox;
@@ -146,6 +146,8 @@ class ScGridWindow : public vcl::Window, public DropTargetHelper, public DragSou
     VclPtr<ScCheckListMenuWindow>    mpDPFieldPopup;
     std::unique_ptr<ScDPFieldButton> mpFilterButton;
 
+    ScCheckListMenuWindow::ResultType aSaveAutoFilterResult;
+
     sal_uInt16              nCursorHideCount;
 
     sal_uInt16              nButtonDown;
diff --git a/sc/source/ui/view/gridwin.cxx b/sc/source/ui/view/gridwin.cxx
index badb7901c538..ba3653d73420 100755
--- a/sc/source/ui/view/gridwin.cxx
+++ b/sc/source/ui/view/gridwin.cxx
@@ -704,6 +704,9 @@ void ScGridWindow::LaunchAutoFilterMenu(SCCOL nCol, SCROW nRow)
     aConfig.mbRTL = pViewData->GetDocument()->IsLayoutRTL(pViewData->GetTabNo());
     mpAutoFilterPopup->setConfig(aConfig);
     mpAutoFilterPopup->launch(aCellRect);
+
+    // remember filter rules before modification
+    mpAutoFilterPopup->getResult(aSaveAutoFilterResult);
 }
 
 void ScGridWindow::RefreshAutoFilterButton(const ScAddress& rPos)
@@ -777,6 +780,20 @@ void ScGridWindow::UpdateAutoFilterFromMenu(AutoFilterMode eMode)
         pViewData->GetDispatcher().Execute(SID_FILTER, SfxCallMode::SLOT|SfxCallMode::RECORD);
         return;
     }
+    if (eMode != AutoFilterMode::Top10
+            && eMode != AutoFilterMode::Empty
+            && eMode != AutoFilterMode::NonEmpty)
+    {
+        // do not recreate auto-filter rules if there is no any changes from the user
+        ScCheckListMenuWindow::ResultType aResult;
+        mpAutoFilterPopup->getResult(aResult);
+
+        if (aResult == aSaveAutoFilterResult)
+        {
+            SAL_INFO("sc.ui", "nothing to do when autofilter entries are the same");
+            return;
+        }
+    }
 
     ScQueryParam aParam;
     pDBData->GetQueryParam(aParam);