[Libreoffice] [PATCH] Base fdo#40079 "file / save (as)" inoperant
Lionel Elie Mamane
lionel at mamane.lu
Sat Aug 20 09:32:03 PDT 2011
On Thu, Aug 18, 2011 at 12:52:55PM +0100, Noel Power wrote:
>On 17/08/11 19:02, Lionel Elie Mamane wrote:
>> On Wed, Aug 17, 2011 at 09:48:04AM +0100, Noel Power wrote:
>>> On 16/08/11 17:32, Lionel Elie Mamane wrote:
>> I notice that calc and writer have the good sense of displaying an
>> error message when an exception is raised during save. That's
>> better than Base, that just silently aborts the save with no error
>> However, calc seems to get very confused and disables (greys out) the
>> save, close file, quit application, etc features :-|
> then it might be an idea to back out that commit,
I'd rather we fix *that* separate bug cleanly; after all, other
exceptions may by thrown during a save. That commit is only in master
(not libreoffice-3-4) anyway, so we have breathing space.
> it appears the exception here isn't really handled, I think the
> exception is caught ( and ignored ) previously precisely because
> nothing really can be done ( or at least the calling code lacks the
> brains to handle the exception ).
Writer seems to handle it correctly... Also, the exception *is* caught
(file sfx2/source/doc/objserv.cxx function
SfxObjectShell::ExecFile_Impl line 680 in master), as the nice pop-up
error message shows. Probably calc "just" mishandles an error return
value of that function.
That code throws away the message of the exception, though :-(
It will probably be about two weeks before I can look into it.
On Thu, Aug 18, 2011 at 02:06:54PM +0100, Noel Power wrote:
> On 17/08/11 19:02, Lionel Elie Mamane wrote:
>>>> Which leads me to another bug: If I remove the librarie's only
>>>> dialog and save, I restart LO, I reopen the file again, the
>>>> dialog is back.
>> Fixed (by me) in master.
> maybe a better test would be
> + if ( ! ( xNameAccess->hasElements() || bInplaceStorage ) )
> instead of
> + if ( ! ( xNameAccess->hasElements() || isModified() ) )
I'm not familiar with that code, if you say so, then make that
More information about the LibreOffice