[PATCH] Change in core[libreoffice-4-0]: don't create multiple dialogs in the loop, fdo#59314

[Markus Mohrhard (via Code Review)]

Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1778

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/78/1778/1

don't create multiple dialogs in the loop, fdo#59314

Change-Id: Ia48262cdf8b07caf6622f2d4a48f366f3f26c0fd
---
M sc/source/ui/view/cellsh1.cxx
1 file changed, 1 insertion(+), 0 deletions(-)



diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx
index 68be16dd..2227c19 100644
--- a/sc/source/ui/view/cellsh1.cxx
+++ b/sc/source/ui/view/cellsh1.cxx
@@ -1805,6 +1805,7 @@
                             // found a matching range, edit this conditional 
format
                             nKey = pCondFormat->GetKey();
                             pCondFormatDlg.reset( new ScCondFormatDlg( 
pTabViewShell->GetDialogParent(), pDoc, pCondFormat, rCondFormatRange, aPos, 
condformat::dialog::NONE ) );
+                            break;
                         }
                     }
 

-- 
To view, visit https://gerrit.libreoffice.org/1778
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia48262cdf8b07caf6622f2d4a48f366f3f26c0fd
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Markus Mohrhard <markus.mohrh...@googlemail.com>

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice