diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-05-15 10:52:48 +0100 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-05-15 13:52:54 +0200 |
| commit | 10cdeed12ef834f5df3b6577c1d9efcc811d6938 (patch) | |
| tree | f153e8334dc27827dfd7de6939f583294daad3ff /cui | |
| parent | 773b8e3cc1340e6f5bfc364f0705344b744edd8d (diff) | |
tdf#133036 notebookbar customize uses a special combobox id
in all other cases the m_xSaveInListBox id contains a SaveInData pointer
but not the notebookbar case for some reason, we're already checking
for this special id in one case, extend to check in the other too
Change-Id: I460afa165ee2f6baa6deb1a15b0257df274a4c94
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/94288
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'cui')
| -rw-r--r-- | cui/source/customize/cfg.cxx | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/cui/source/customize/cfg.cxx b/cui/source/customize/cfg.cxx index 9ddac7620b8d..67466b3bf6d2 100644 --- a/cui/source/customize/cfg.cxx +++ b/cui/source/customize/cfg.cxx @@ -1304,17 +1304,21 @@ bool SvxConfigPage::FillItemSet( SfxItemSet* ) for (int i = 0, nCount = m_xSaveInListBox->get_count(); i < nCount; ++i) { - SaveInData* pData = - reinterpret_cast<SaveInData*>(m_xSaveInListBox->get_id(i).toInt64()); - if(m_xSaveInListBox->get_id(i) != notebookbarTabScope) + OUString sId = m_xSaveInListBox->get_id(i); + if (sId != notebookbarTabScope) + { + SaveInData* pData = reinterpret_cast<SaveInData*>(sId.toInt64()); result = pData->Apply(); + } } return result; } IMPL_LINK_NOARG(SvxConfigPage, SelectSaveInLocation, weld::ComboBox&, void) { - pCurrentSaveInData = reinterpret_cast<SaveInData*>(m_xSaveInListBox->get_active_id().toInt64()); + OUString sId = m_xSaveInListBox->get_active_id(); + if (sId != notebookbarTabScope) + pCurrentSaveInData = reinterpret_cast<SaveInData*>(sId.toInt64()); Init(); } |