[PATCH] Change in core[libreoffice-4-0]: Template Manager: don't deselect item after setting default ...

[Bosdonnat Cedric (via Code Review)]

Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1611

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/11/1611/1

Template Manager: don't deselect item after setting default template

Change-Id: I860fb2c91d6843a1d5345ab2cf9270473f73f5bc
---
M sfx2/source/doc/templatedlg.cxx
1 file changed, 8 insertions(+), 10 deletions(-)



diff --git a/sfx2/source/doc/templatedlg.cxx b/sfx2/source/doc/templatedlg.cxx
index 282372d..bff2dc1 100644
--- a/sfx2/source/doc/templatedlg.cxx
+++ b/sfx2/source/doc/templatedlg.cxx
@@ -1069,19 +1069,17 @@
 
 void SfxTemplateManagerDlg::OnTemplateAsDefault ()
 {
-    assert(!maSelTemplates.empty());
-
-    const TemplateViewItem *pItem = static_cast<const TemplateViewItem*>(*(maSelTemplates.begin()));
-
-    OUString aServiceName;
-    if (lcl_getServiceName(pItem->getPath(),aServiceName))
+    if (!maSelTemplates.empty())
     {
-        SfxObjectFactory::SetStandardTemplate(aServiceName,pItem->getPath());
+        const TemplateViewItem *pItem = static_cast<const TemplateViewItem*>(*(maSelTemplates.begin()));
 
-        createDefaultTemplateMenu();
+        OUString aServiceName;
+        if (lcl_getServiceName(pItem->getPath(),aServiceName))
+        {
+            SfxObjectFactory::SetStandardTemplate(aServiceName,pItem->getPath());
 
-        // clear selection and display view/action toolbars
-        maView->deselectOverlayItem(pItem->mnId);
+            createDefaultTemplateMenu();
+        }
     }
 }
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I860fb2c91d6843a1d5345ab2cf9270473f73f5bc
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Bosdonnat Cedric <cedric.bosdonnat at free.fr>