diff options
| author | Szymon Kłos <szymon.klos@collabora.com> | 2021-11-15 09:13:35 +0100 |
|---|---|---|
| committer | Szymon Kłos <szymon.klos@collabora.com> | 2022-01-24 12:17:20 +0100 |
| commit | b9f53322df937098eef4fd5b2889989a38e4ffda (patch) | |
| tree | 64f26297e3f7b8bdef791b573db5a95d53aa3442 /vcl/jsdialog/executor.cxx | |
| parent | e064a2a1e82e913c49312706cd52736fa6a4f994 (diff) | |
jsdialog: use string identifiers
Thanks to that it is easier to debug and find widgets in maps.
Sidebar and notebookbar are now in different map entries.
This fixes the issue when destroying notebookbar deleted sidebar's
widgets.
After this patch sidebar works correctly when using notebookbar mode in
lok.
Change-Id: Ie9dcb82675129bdb567b766e29779744f500cb48
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125216
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Szymon Kłos <szymon.klos@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128853
Tested-by: Jenkins
Diffstat (limited to 'vcl/jsdialog/executor.cxx')
| -rw-r--r-- | vcl/jsdialog/executor.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vcl/jsdialog/executor.cxx b/vcl/jsdialog/executor.cxx index 742e56800eca..87183f504c67 100644 --- a/vcl/jsdialog/executor.cxx +++ b/vcl/jsdialog/executor.cxx @@ -34,14 +34,14 @@ StringMap jsonToStringMap(const char* pJSON) return aArgs; } -void SendFullUpdate(sal_uInt64 nWindowId, const OString& rWidget) +void SendFullUpdate(const std::string& nWindowId, const OString& rWidget) { weld::Widget* pWidget = JSInstanceBuilder::FindWeldWidgetsMap(nWindowId, rWidget); if (auto pJSWidget = dynamic_cast<BaseJSWidget*>(pWidget)) pJSWidget->sendFullUpdate(); } -bool ExecuteAction(sal_uInt64 nWindowId, const OString& rWidget, StringMap& rData) +bool ExecuteAction(const std::string& nWindowId, const OString& rWidget, StringMap& rData) { weld::Widget* pWidget = JSInstanceBuilder::FindWeldWidgetsMap(nWindowId, rWidget); |