summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike Kaganski <mike.kaganski@collabora.com>2021-12-16 16:51:12 +0100
committerXisco Fauli <xiscofauli@libreoffice.org>2021-12-17 23:07:02 +0100
commit37c617af996bf5f9c4be08a1245940d00668a64b (patch)
tree1c2852b78bcd58106814cbdaab89332d2537c6c4
parenttdf#144565 sw_redlinehide: fix mailmerge when flys anchored at last node (diff)
downloadcore-37c617af996bf5f9c4be08a1245940d00668a64b.tar.gz
core-37c617af996bf5f9c4be08a1245940d00668a64b.zip
tdf#146219: don't try to detect if a recent file is a directory
This avoids accessing (possibly unavailable) remote URLs, when we already know that MRU items are files. Change-Id: I97e07f007d4129b93b4939311c67cf95de6ff836 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/126943 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com> Signed-off-by: Xisco Fauli <xiscofauli@libreoffice.org> Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127004 (cherry picked from commit c247023dfd10bef22b9ff5a85cc44febfc90e951) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/126974
-rw-r--r--framework/source/uielement/recentfilesmenucontroller.cxx6
1 files changed, 4 insertions, 2 deletions
diff --git a/framework/source/uielement/recentfilesmenucontroller.cxx b/framework/source/uielement/recentfilesmenucontroller.cxx
index b25c84256cc2..6bf9c1f99858 100644
--- a/framework/source/uielement/recentfilesmenucontroller.cxx
+++ b/framework/source/uielement/recentfilesmenucontroller.cxx
@@ -204,8 +204,10 @@ void RecentFilesMenuController::fillPopupMenu( Reference< css::awt::XPopupMenu >
pVCLPopupMenu->InsertItem( sal_uInt16( i+1 ), aMenuShortCut.makeStringAndClear() );
if ( bIsIconsAllowed ) {
- Image aThumbnail = SvFileInformationManager::GetImage(aURL, false, {});
- pVCLPopupMenu->SetItemImage(sal_uInt16 ( i+1 ), aThumbnail);
+ // tdf#146219: don't use SvFileInformationManager::GetImageId,
+ // which needs to access the URL to detect if it's a directory
+ BitmapEx aThumbnail(SvFileInformationManager::GetFileImageId(aURL));
+ pVCLPopupMenu->SetItemImage(sal_uInt16 ( i+1 ), Image(aThumbnail));
}
pVCLPopupMenu->SetTipHelpText( sal_uInt16( i+1 ), aTipHelpText );