> Hello,
>
> following places use the enumerateLocks(...) method:
>
> 1) LockImpl.unlock
> 2) LockImpl.renew
> 3) LockImpl.kill
> 4) MacroImpl.deleteObject
> 5) WebdavServlet.displayDirectoryBrowsing
> 6) PropFind.parsePropertiesOfObject
> 7) LockMethod.executeRequest
>
>
> Suggestion:
>
> 1) inherited == false
> 2) false
> 3) false
> 4) false
> 5) ? false ?
> 6) ? false ?
> 7) ? call LockImpl.renew ?
>
> Could someone please verify 5-7 (possibly also 1-4). Did I miss something?
+1 for 5.
I think 6 has to show the inherited locks too, but check in the spec to make
sure of that.
7 = false too. (in the case LOCK_REFRESH, we already call lock.renew)
This bug has always been there, but never showed up because inherited locks
aren't very well supported / used by many people.
Remy