Merge pull request #391 from nextcloud/fix/unlock-app-lock

Allow force unlock of automated client locks

.gitignore

@@ -3,6 +3,7 @@
 \.idea/
 
 /build/
+/css/
 /js/
 /vendor/
 /node_modules/

lib/Db/LocksRequest.php

@@ -31,7 +31,8 @@ public function save(FileLock $lock) {
 			->setValue('token', $qb->createNamedParameter($lock->getToken()))
 			->setValue('creation', $qb->createNamedParameter($lock->getCreatedAt()))
 			->setValue('type', $qb->createNamedParameter($lock->getType()))
-			->setValue('ttl', $qb->createNamedParameter($lock->getTimeout()));
+			->setValue('ttl', $qb->createNamedParameter($lock->getTimeout()))
+			->setValue('owner', $qb->createNamedParameter($lock->getDisplayName()));
 
 		try {
 			$qb->execute();

lib/Listeners/LoadAdditionalScripts.php

@@ -24,5 +24,6 @@ public function handle(Event $event): void {
 
 		Util::addInitScript(Application::APP_ID, 'files_lock-init');
 		Util::addScript(Application::APP_ID, 'files_lock-main');
+		Util::addStyle(Application::APP_ID, 'files_lock-main');
 	}
 }

lib/Service/LockService.php

@@ -24,6 +24,7 @@
 use OCP\Files\Lock\OwnerLockedException;
 use OCP\Files\NotFoundException;
 use OCP\IL10N;
+use OCP\IRequest;
 use OCP\IUserManager;
 use OCP\IUserSession;
 
@@ -43,6 +44,7 @@ class LockService {
 	private IAppManager $appManager;
 	private IEventDispatcher $eventDispatcher;
 	private IUserSession $userSession;
+	private IRequest $request;
 
 
 	private array $locks = [];
@@ -61,6 +63,7 @@ public function __construct(
 		IAppManager $appManager,
 		IEventDispatcher $eventDispatcher,
 		IUserSession $userSession,
+		IRequest $request,
 	) {
 		$this->l10n = $l10n;
 		$this->userManager = $userManager;
@@ -70,6 +73,7 @@ public function __construct(
 		$this->appManager = $appManager;
 		$this->eventDispatcher = $eventDispatcher;
 		$this->userSession = $userSession;
+		$this->request = $request;
 
 		$this->setup('app', 'files_lock');
 	}
@@ -162,9 +166,9 @@ public function lock(LockContext $lockScope): FileLock {
 			$this->generateToken($lock);
 			$lock->setCreation(time());
 			$this->notice('locking file', false, ['fileLock' => $lock]);
+			$this->injectMetadata($lock);
 			$this->locksRequest->save($lock);
 			$this->propagateEtag($lockScope);
-			$this->injectMetadata($lock);
 			return $lock;
 		}
 	}
@@ -317,7 +321,11 @@ public function injectMetadata(FileLock $lock): FileLock {
 			$displayName = $this->getAppName($lock->getOwner()) ?? null;
 		}
 		if ($lock->getType() === ILock::TYPE_TOKEN) {
-			$displayName = $this->userManager->getDisplayName($lock->getOwner()) ?? $lock->getDisplayName();
+			$clientHint = $this->getClientHint();
+			$displayName = $lock->getDisplayName() ?: (
+				$this->userManager->getDisplayName($lock->getOwner()) . ' ' .
+				($clientHint ? ('(' . $clientHint . ')') : '')
+			);
 		}
 
 		if ($displayName) {
@@ -326,6 +334,22 @@ public function injectMetadata(FileLock $lock): FileLock {
 		return $lock;
 	}
 
+	private function getClientHint(): ?string {
+		if ($this->request->isUserAgent([IRequest::USER_AGENT_CLIENT_DESKTOP])) {
+			return $this->l10n->t('Desktop client');
+		}
+
+		if ($this->request->isUserAgent([IRequest::USER_AGENT_CLIENT_IOS])) {
+			return $this->l10n->t('iOS client');
+		}
+
+		if ($this->request->isUserAgent([IRequest::USER_AGENT_CLIENT_ANDROID])) {
+			return $this->l10n->t('Android client');
+		}
+
+		return null;
+	}
+
 	/**
 	 * @param FileLock $lock
 	 */