diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2025-07-06 13:08:24 +0500 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2025-07-06 14:04:56 +0200 |
commit | 9b69bc0e7db564a2f61e7a2c6fbd7ff5ce1bbd17 (patch) | |
tree | c6eb0b030b89d6d6d7db74ad199b17ec9e32ca1c /connectivity/source/drivers/macab | |
parent | 970360adcc3a6445de0759530086bb6336b18a5a (diff) |
In commit c6b95527d2e00ed3a4cf3bad3a328ca5b29a588a (INTEGRATION: CWS
chart2mst3 (1.15.4); FILE MERGED, 2007-05-22), it was implemented to
check that the target frame name is not empty and is equal to "_self".
The "not empty" check was redundant as implemented; and was dropped
in commit 33fe5b57eae6a3319b23afbe5be88f0548761f56 (Drop a redundant
check, 2025-07-04). But it looks like the original implementation
intended to do something different, namely check if the target frame
name is *either* empty, *or* "_self": see documentation for _self in
offapi/com/sun/star/frame/XFrame.idl (and implementation in various
places, like framework/source/dispatch/dispatchprovider.cxx).
This is fixed here. CommandDispatchContainer::getDispatchesForURLs
is fixed as well.
Change-Id: I9ed6981edeca16661f07e41b87b114a8e16d69ff
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/187442
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Jenkins
Diffstat (limited to 'connectivity/source/drivers/macab')
0 files changed, 0 insertions, 0 deletions