[Libreoffice-commits] core.git: Branch 'libreoffice-6-3' - sd/source

[Xisco Fauli (via logerrit)]

 sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit e2aa8de67c3bf0cc09160b256c825fb586947ce6
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Tue Feb 11 12:11:32 2020 +0100
Commit:     Michael Stahl <michael.stahl at cib.de>
CommitDate: Wed Feb 12 11:49:12 2020 +0100

    tdf#130440 only select first page if no other page is selected
    
    Found while implementing the UItest.
    See https://gerrit.libreoffice.org/c/core/+/88437
    it asserts on line 34 AssertionError: 2 != 1 as the first and
    second slides are selected
    
    Change-Id: I249dc0ac6faa55f0f15deedb848beabbc64aeed4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88439
    Tested-by: Xisco Faulí <xiscofauli at libreoffice.org>
    Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>
    (cherry picked from commit 7ed602a3b8c0ffe922b4f082cd4cdaa5a8f0d64c)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/88405
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <michael.stahl at cib.de>

diff --git a/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx b/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
index e146ee0a960c..81403c7bc1d8 100644
--- a/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
+++ b/sd/source/ui/slidesorter/controller/SlsSelectionObserver.cxx
@@ -130,7 +130,7 @@ void SelectionObserver::EndObservation()
     bool bSuccess = mrSlideSorter.GetController().GetFocusManager().SetFocusedPageToCurrentPage();
     // tdf#129346 nothing currently selected, select something, if possible
     // but (tdf#129346) only if setting focus to current page failed
-    if (!bSuccess && rSelector.GetPageCount())
+    if (!bSuccess && rSelector.GetPageCount() && rSelector.GetSelectedPageCount() == 0)
         rSelector.SelectPage(0);
 }