[PATCH libreoffice-4-0] copy values from the original item set

David Tardon (via Code Review) gerrit at gerrit.libreoffice.org
Mon Jun 3 06:23:45 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/4145

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/45/4145/1

copy values from the original item set

... because SfxAbstractDialog::GetOutputItemSet() only contains the
items that have been _changed_ in the dialog.

Change-Id: Iaf20bb42b65e420fe4e7aff979378eff7b5e5df2
(cherry picked from commit 9cd71f996e3625e8174c46bf85e336c1ee90df7f)

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



diff --git a/svx/source/table/tablecontroller.cxx b/svx/source/table/tablecontroller.cxx
index f29946d..b25e5f1 100644
--- a/svx/source/table/tablecontroller.cxx
+++ b/svx/source/table/tablecontroller.cxx
@@ -732,7 +732,8 @@
         std::auto_ptr< SfxAbstractTabDialog > pDlg( pFact ? pFact->CreateSvxFormatCellsDialog( NULL, &aNewAttr, pTableObj->GetModel(), pTableObj) : 0 );
         if( pDlg.get() && pDlg->Execute() )
         {
-            SfxItemSet aNewSet( *(pDlg->GetOutputItemSet ()) );
+            SfxItemSet aNewSet( aNewAttr );
+            aNewSet.Put( *(pDlg->GetOutputItemSet ()) );
 
             SvxBoxItem aNewBoxItem( static_cast< const SvxBoxItem& >( aNewSet.Get( SDRATTR_TABLE_BORDER ) ) );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9100dbe05c418a9dd1b819534a08076c9a4aed1c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: David Tardon <dtardon at redhat.com>



More information about the LibreOffice mailing list