Apache OpenOffice (AOO) Bugzilla – Issue 64800
FolderPicker cannot open with display of a given folder
Last modified: 2017-05-20 09:32:01 UTC
Tested on Windows XP SP2, OpenOffice Options : "Use OpenOffice.org dialogs" NOT checked. In the XFolderPicker interface, method setDisplayDirectory sets the root directory that the FolderPicker should display. This was the case with 1.1.x (currently tested with 1.1.4) This is not the case with 2.0.x (currently tested with 2.0.2) Next attachment contains a macro to show this.
Created attachment 36072 [details] Basic macro calling FolderPicker
sw->tra: this used to be your area methinks
reassign
.
*** Issue 67643 has been marked as a duplicate of this issue. ***
Adjusted target to CWS.
Issue #79211 is in the same area. Maybe, maybe, it has some relationship with this one?
*** Issue 80972 has been marked as a duplicate of this issue. ***
No time left to fix this for 2.4.
Retargeted.
Works for me with system paths rather than URLs. OOo 2.4 (Sun), Linux.
Here's a patch stolen from issue 97062: Index: MtaFop.cxx =================================================================== --- MtaFop.cxx (revision 263749) +++ MtaFop.cxx (working copy) @@ -470,12 +470,6 @@ NULL ); } } - - if (pIMalloc.is()) - pIMalloc->Free(lpItemIdList); - - lpItemIdList = NULL; - return lpItemIdList; }
*** Issue 97062 has been marked as a duplicate of this issue. ***
Carsten, can you please consider this patch as part of your file picker rework on Windows?
cd: Accepted.
@bmarcelly: Thanks for your patch I used it as base for my own changes. According to the documentation IShellFolder::ParseDisplayName PIDLIST_RELATIVE *ppidl, only as a out parameter. Therefore allocating memory before calling ParseDisplayName makes no sense and creates a memory leak. The original code looks very strange and is definitely wrong. cd: Fixed.
cd->tm: Please verify.
Checked and verified in cws filepicker01 -> OK !