Need help regarding BUG #60700
Michael Meeks
michael.meeks at suse.com
Sat Mar 16 14:01:04 PDT 2013
Hi Alex,
On Sat, 2013-03-16 at 13:16 -0700, Alex Ivan wrote:
> Michael Meeks-2 wrote
> >> I've been looking into bug #60700
> >> <https://bugs.freedesktop.org/show_bug.cgi?id=60700> and have
>
> The first place that i found where these directories are created is in
> framework/source/uiconfiguration/uiconfigurationmanager.cxx:
> void UIConfigurationManager::impl_Initialize()
Yes indeed - odd that we create the storage elements there :-) no idea
why we do that; prolly worth checking that out:
> ...
> xElementTypeStorage = m_xDocConfigStorage->openStorageElement(
> rtl::OUString::createFromAscii( UIELEMENTTYPENAMES[i] ), nModes );
> ...
>
> Without this bit, an empty .odf does not end up containing those
> directories.
Nice ;-) I wonder though whether we shouldn't un-conditionally load it
as read-only, and re-open it in the 'store' method as writeable if
indeed the document is write-able (?) Of course, we'd need to audit the
uses of the xStorage member.
> On the one hand, there are other similar calls in this and other files, so
> it may have just been a slip up when writing these. On the other hand, i
> didn't have any good examples to test this out.
Right - of course, finding something that actually uses this so we can
test the (already broken) feature would be useful :-)
> and, since no one responded to this for some time, i did not investigate
> further. I will do so now that there is at least one example and update
> you if if figure it out.
Thanks ! of course, if you run out of time, just dumping the research /
code pointers into the easy-hack for the next person might make it even
easier for them.
> Thank you for your help,
And thanks for your interest / research Alex :-)
All the best,
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list