[Issue 126305] WebDAV lock on 4.1.1 does not work

[bu...@apache.org]

https://bz.apache.org/ooo/show_bug.cgi?id=126305

Giuseppe Castagno (aka beppec56) <gi...@acca-esse.eu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #84738|review?                     |
              Flags|                            |
  Attachment #84738|0                           |1
        is obsolete|                            |
  Attachment #84784|                            |review?
              Flags|                            |

--- Comment #3 from Giuseppe Castagno (aka beppec56) <gi...@acca-esse.eu> ---
Created attachment 84784
  --> https://bz.apache.org/ooo/attachment.cgi?id=84784&action=edit
Proposed patch to fix the issue, version #2

The patch is rebased on trunk, r1683442.

It completely replaces the former one.

I added the placeholder '//->i126305' where changes where made (sfx2,
ucbhelper, comphelper, offapi, ucb).

The patch was built successfully both on Linux (Ubuntu 14.04 64bit) and Windows
(Windows 7 64bit).

Changes from the previous patch version:

- fixed a regression while doing copy/paste among AOO application;

- added a correct refresh of the document while turning it to r/w from r/o
(e.g. when the other user releases the lock on the WebDAV resource). This same
refresh is operating even when you try to switch from r/o to r/w and the
document is reported as still locked, so the user can see the current document
state;

- corrected a wrong behaviour existent in AOO 4.1.1, where the edit button on
toolbar stop functioning after you try it several time and the file is still
locked by another user;

- shortened the refresh time of the lock because some WebDAV servers even
though they are HTTP/1.1 after some time they return an error that is mapped by
serf to 'connection closed by peer'.
Time was reduced from 3 minutes down to 2 minutes to solve the problem.

- removed some unused white spaces while I was at it...

In my own opinion, this patch should be evaluated to be added to AOO 4.1.2
version.

-- 
You are receiving this mail because:
You are the assignee for the issue.