Genereller Hinweis zum Setzen von Locks über die API.
1. Locks sollten immer im try / finally block stehen
2. Es sollten "immer" nicht rekursive Locks gesetzt werden.
boolean wasLocked = medium.isLocked();
try {
if ( ! wasLocked) {
medium.setLock(true, false); // lock non-recursive
}
medium.release(false); // non recursive release
} catch(LockException lockFailed) {
context.logError("lock failed -> medium already locked by " + lockFailed.getUserLoginName());
} finally {
try {
if ( ! wasLocked) {
medium.setLock(false, false); // unlock non-recursive
}
} catch(Exception unlockFailed){
// proper error handling
}
}
zu 1.)
Der try/finally Block stellt sicher, dass das Objekt (in diesem Fall das Medium) auch im Fall eines Fehlers wieder entsperrt wird. Ansonsten bleibt das Objekt auf dem Server für die Dauer der Session mit der der Lock gesetzt wurde, erhalten. Das kann bei einem Skript auf dem Server mitunter für die gesamte Laufzeit des FirstSpirit Servers sein.
zu 2.)
Da es sich in diesem Fall um ein Medium handelt, würde ein rekursiver Lock den gleichen Effekt haben wie der nicht rekursive Lock. Man sollte sich allerdings angewöhnen immer "nicht-rekursive" Locks zu verwenden, außer man benötigt explizit einen "rekurisven" Lock (z.B. bei Page)
. Der Fehler fliegt, wenn der Workflow nach dem weiterschalten den Bearbeitungsmodus beenden will]
