diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-02-24 09:28:19 +0200 |
|---|---|---|
| committer | Andras Timar <andras.timar@collabora.com> | 2017-02-24 12:24:46 +0100 |
| commit | f32cfbc05eec644e2f37aac38410ad3f8c944ef5 (patch) | |
| tree | 6362461f7475b4316a916d8251895d1a06541928 /forms | |
| parent | 16400f7f1ab81720b0c3a742b4e699ce7d7792e6 (diff) | |
tdf#103830 Form field names not update in python macro
regression from commit 5f15cc01b31ccaed0c6482a36556dece084ce302
"new loplugin: use more efficient find() methods"
Regression found by Aron Budea.
Change-Id: I0e84376dfd754738de97882979058f86c2476bd6
Reviewed-on: https://gerrit.libreoffice.org/34600
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Julien Nabet <serval2412@yahoo.fr>
(cherry picked from commit 3a404ea870f1eed86f9765447ce0a364d39ae8f8)
Reviewed-on: https://gerrit.libreoffice.org/34601
(cherry picked from commit 81e5ba515c66bc23520f3d9c06c54c63d927e467)
Diffstat (limited to 'forms')
| -rw-r--r-- | forms/source/misc/InterfaceContainer.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/forms/source/misc/InterfaceContainer.cxx b/forms/source/misc/InterfaceContainer.cxx index e8973e7d6000..cf64c9742617 100644 --- a/forms/source/misc/InterfaceContainer.cxx +++ b/forms/source/misc/InterfaceContainer.cxx @@ -661,9 +661,9 @@ throw (css::uno::RuntimeException, std::exception) { { ::osl::MutexGuard aGuard( m_rMutex ); OInterfaceMap::iterator i = m_aMap.find(::comphelper::getString(evt.OldValue)); - if (i != m_aMap.end() && (*i).second != evt.Source) + if (i != m_aMap.end() && i->second == evt.Source) { - css::uno::Reference<css::uno::XInterface> xCorrectType((*i).second); + css::uno::Reference<css::uno::XInterface> xCorrectType(i->second); m_aMap.erase(i); m_aMap.insert(::std::pair<const OUString, css::uno::Reference<css::uno::XInterface> >(::comphelper::getString(evt.NewValue),xCorrectType)); } |