About fix for tdf#82744: WebDAV LOCK is released when document is saved
Hi all, while implementing a fix for: https://bugs.documentfoundation.org/show_bug.cgi?id=82744 using ideas from a similar patch I proposed in AOO: https://bz.apache.org/ooo/show_bug.cgi?id=126305 I came across a comment on top of method SfxMedium::ShowLockedDocumentDialog. The comment states that lock/unlock should be put in the relevant ucb component. Unfortunately I started implementing the fix managing the WebDAV lock/unlock from the framework instead. Implementing the same thing directly in ucb/webdav appears difficult, because I didn't find there anything that has a lifetime usable for lock/unlock. So, given the two choices: 1) do I continue to implement the fix in framework, hopefully finishing it in few days, or 2) do I start studying what to rewrite in webdav ucb component in order to support the lock/unlock directly there, in this case I have no idea what to to ATM; it can take some time to work out a suitable solution. What would be the best course of action? -- Kind Regards, Giuseppe Castagno aka beppec56 Acca Esse http://www.acca-esse.eu giuseppe.castagno at acca-esse.eu ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: About fix for tdf#82744: WebDAV LOCK is released when document is saved
BTW, what does HAVE_FEATURE_MULTIUSER_ENVIRONMENT compile time flag (inside the commit recalled above) mean? The HAVE_FEATURE_* macros are (or should be) documented where defined, in config_host/config_features.h.in. In practice, HAVE_FEATURE_MULTIUSER_ENVIRONMENT currently means anything except Android and iOS. --tml ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: About fix for tdf#82744: WebDAV LOCK is released when document is saved
On 07/14/2015 12:34 PM, Giuseppe Castagno wrote: I came across a comment on top of method SfxMedium::ShowLockedDocumentDialog. The comment states that lock/unlock should be put in the relevant ucb component. You mean the FIXME: ... comment added with http://cgit.freedesktop.org/libreoffice/core/commit/?id=063b32804c58f663e42444f676af1a1e3a1cc2a8 Don't use lock files when sandboxed on OS X? I wouldn't worry too much about it. IIUC, it is concerned with LO's home-grown locking scheme (involving additional .~lock.*# files), which is only marginally related to the question whether to send WebDAV UNLOCK when saving vs. when closing a document. Unfortunately I started implementing the fix managing the WebDAV lock/unlock from the framework instead. You mean, you somehow manage with modifications to the framework code to cause the sending of WebDAV UNLOCK to happen upon closing a document, instead upon saving it? Sounds good then, IMO. ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: About fix for tdf#82744: WebDAV LOCK is released when document is saved
Hi Stephan, On 07/14/2015 02:12 PM, Stephan Bergmann wrote: On 07/14/2015 12:34 PM, Giuseppe Castagno wrote: I came across a comment on top of method SfxMedium::ShowLockedDocumentDialog. The comment states that lock/unlock should be put in the relevant ucb component. You mean the FIXME: ... comment added with http://cgit.freedesktop.org/libreoffice/core/commit/?id=063b32804c58f663e42444f676af1a1e3a1cc2a8 Don't use lock files when sandboxed on OS X? I wouldn't worry too much about it. IIUC, it is concerned with LO's home-grown locking scheme (involving additional .~lock.*# files), which is only marginally related to the question whether to send WebDAV UNLOCK when saving vs. when closing a document. exactly that, 3rd hunk of the commit you pointed me to. Unfortunately I started implementing the fix managing the WebDAV lock/unlock from the framework instead. You mean, you somehow manage with modifications to the framework code to cause the sending of WebDAV UNLOCK to happen upon closing a document, instead upon saving it? Sounds good then, IMO. the framework changes I'working on are meant to manage correctly the lock/unlock in WebDAV. ASA I have a working commit, I'll push to gerrit. BTW, what does HAVE_FEATURE_MULTIUSER_ENVIRONMENT compile time flag (inside the commit recalled above) mean? -- Kind Regards, Giuseppe Castagno aka beppec56 Acca Esse http://www.acca-esse.eu giuseppe.castagno at acca-esse.eu ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
Re: About fix for tdf#82744: WebDAV LOCK is released when document is saved
Hi Tor, On 07/14/2015 03:13 PM, Tor Lillqvist wrote: BTW, what does HAVE_FEATURE_MULTIUSER_ENVIRONMENT compile time flag (inside the commit recalled above) mean? The HAVE_FEATURE_* macros are (or should be) documented where defined, in config_host/config_features.h.in http://config_features.h.in. In practice, HAVE_FEATURE_MULTIUSER_ENVIRONMENT currently means anything except Android and iOS. thanks, exactly the clarification I needed. -- Kind Regards, Giuseppe Castagno aka beppec56 Acca Esse http://www.acca-esse.eu giuseppe.castagno at acca-esse.eu ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice