<html>
<head>
<base href="https://bugs.documentfoundation.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - Can't edit file on samba shares ( see comment 28 )"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=115747#c34">Comment # 34</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - Can't edit file on samba shares ( see comment 28 )"
href="https://bugs.documentfoundation.org/show_bug.cgi?id=115747">bug 115747</a>
from <span class="vcard"><a class="email" href="mailto:mikekaganski@hotmail.com" title="Mike Kaganski <mikekaganski@hotmail.com>"> <span class="fn">Mike Kaganski</span></a>
</span></b>
<pre>(In reply to Thorsten Behrens (CIB) from <a href="show_bug.cgi?id=115747#c32">comment #32</a>)
<span class="quote">> (In reply to Justin L from <a href="show_bug.cgi?id=115747#c31">comment #31</a>)
> > I have no idea why so much code was removed from the exception clause, but
> > restore the part with "if ( bUseSystemLock || !IsOOoLockFileUsed() )" solves
> > both the read-only problem on loading, and the save failure.
>
> Because that hides the fact that the lock file could not be created. If in a
> deployment people are ok to only use system file locking, the config option
> UseDocumentOOoLockFile should be set to false.</span >
In my opinion, the original fix (and this comment) somehow change the proper
notion of a feature/function that helps to reach some goals (reliably detect
locking/identify locking person) to a requirement for function (=if we cannot
detect locking reliably, then fail to work). Is that what is intended? and what
exact problem was solved by that change, that justifies making this
enforcement? The original problem in <a class="bz_bug_link
bz_status_RESOLVED bz_closed"
title="RESOLVED FIXED - When drive is full and open a file, wrong message for lock (no space for the lock-file)"
href="show_bug.cgi?id=106942">tdf#106942</a> only mentioned a corner case of
"drive full", which could be detected separately and acted accordingly.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>