[PATCH] Fix related to fdo#62224.

gokul s (via Code Review) gerrit at gerrit.libreoffice.org
Wed Mar 27 05:42:08 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/3092

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/92/3092/1

Fix related to fdo#62224.

On clicking Cancel Button in Format cells Widget in Impress,
Their is change in "Spacing of contents" values.
Values should not be changed.

Change-Id: If9395eda7f82f8e25148f906094588480a6a1029
---
M svx/source/table/tablecontroller.cxx
1 file changed, 4 insertions(+), 1 deletion(-)



diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx
index 1cb3035..4b01ba8 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -73,6 +73,8 @@
 using namespace ::com::sun::star::text;
 using namespace ::com::sun::star::style;
 
+#define RTL_OK 1
+
 namespace sdr { namespace table {
 
 // --------------------------------------------------------------------
@@ -731,7 +733,8 @@
 
         SvxAbstractDialogFactory* pFact = SvxAbstractDialogFactory::Create();
         std::auto_ptr< SfxAbstractTabDialog > pDlg( pFact ? pFact->CreateSvxFormatCellsDialog( NULL, &aNewAttr, pTableObj->GetModel(), pTableObj) : 0 );
-        if( pDlg.get() && pDlg->Execute() )
+        // On Clicking Cancel button in Format Cells should not change Values.
+        if( pDlg.get() && ( pDlg->Execute() == RTL_OK ) )
         {
             SfxItemSet aNewSet( *(pDlg->GetOutputItemSet ()) );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If9395eda7f82f8e25148f906094588480a6a1029
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: gokul s <gswaminathan at kacst.edu.sa>



More information about the LibreOffice mailing list