Support for old locks

panel/src/components/Views/PreviewView.vue

@@ -91,10 +91,16 @@
 				</header>
 				<iframe v-if="hasChanges" ref="changes" :src="src.changes"></iframe>
 				<k-empty v-else>
-					{{ $t("lock.unsaved.empty") }}
-					<k-button icon="edit" variant="filled" :link="back">
-						{{ $t("edit") }}
-					</k-button>
+					<template v-if="lock.isLegacy">
+						This content is locked by our old lock system. <br />
+						Changes cannot be previewed.
+					</template>
+					<template v-else>
+						{{ $t("lock.unsaved.empty") }}
+						<k-button icon="edit" variant="filled" :link="back">
+							{{ $t("edit") }}
+						</k-button>
+					</template>
 				</k-empty>
 			</section>
 		</main>
@@ -223,6 +229,8 @@ export default {
 	flex-grow: 1;
 	justify-content: center;
 	flex-direction: column;
+	text-align: center;
+	padding-inline: var(--spacing-3);
 	gap: var(--spacing-6);
 	--button-color-text: var(--color-text);
 }

src/Api/Controller/Changes.php

@@ -3,7 +3,9 @@
 namespace Kirby\Api\Controller;
 
 use Kirby\Cms\ModelWithContent;
+use Kirby\Content\Lock;
 use Kirby\Content\VersionId;
+use Kirby\Filesystem\F;
 use Kirby\Form\Form;
 
 /**
@@ -18,13 +20,23 @@
  */
 class Changes
 {
+	/**
+	 * Cleans up legacy lock files
+	 */
+	protected static function cleanup(ModelWithContent $model): void
+	{
+		F::remove(Lock::legacyFile($model));
+	}
+
 	/**
 	 * Discards unsaved changes by deleting the changes version
 	 */
 	public static function discard(ModelWithContent $model): array
 	{
 		$model->version(VersionId::changes())->delete('current');
 
+		static::cleanup($model);
+
 		return [
 			'status' => 'ok'
 		];
@@ -41,6 +53,8 @@ public static function publish(ModelWithContent $model, array $input): array
 			input: $input
 		);
 
+		static::cleanup($model);
+
 		// get the changes version
 		$changes = $model->version(VersionId::changes());
 
@@ -77,6 +91,8 @@ public static function save(ModelWithContent $model, array $input): array
 		$changes = $model->version(VersionId::changes());
 		$latest  = $model->version(VersionId::latest());
 
+		static::cleanup($model);
+
 		// combine the new field changes with the
 		// last published state
 		$changes->save(

src/Content/Lock.php

@@ -5,7 +5,9 @@
 use Kirby\Cms\App;
 use Kirby\Cms\Language;
 use Kirby\Cms\Languages;
+use Kirby\Cms\ModelWithContent;
 use Kirby\Cms\User;
+use Kirby\Data\Data;
 use Kirby\Toolkit\Str;
 
 /**
@@ -27,6 +29,7 @@ class Lock
 	public function __construct(
 		protected User|null $user = null,
 		protected int|null $modified = null,
+		protected bool $legacy = false
 	) {
 	}
 
@@ -39,6 +42,11 @@ public static function for(
 		Version $version,
 		Language|string $language = 'default'
 	): static {
+
+		if ($legacy = static::legacy($version->model())) {
+			return $legacy;
+		}
+
 		// wildcard to search for a lock in any language
 		// the first locked one will be preferred
 		if ($language === '*') {
@@ -87,13 +95,21 @@ public function isActive(): bool
 	}
 
 	/**
-	 * Check if content locking is enabled at all
+	 * Checks if content locking is enabled at all
 	 */
 	public static function isEnabled(): bool
 	{
 		return App::instance()->option('content.locking', true) !== false;
 	}
 
+	/**
+	 * Checks if the lock is coming from an old .lock file
+	 */
+	public function isLegacy(): bool
+	{
+		return $this->legacy;
+	}
+
 	/**
 	 * Checks if the lock is actually locked
 	 */
@@ -124,6 +140,49 @@ public function isLocked(): bool
 		return true;
 	}
 
+	/**
+	 * Looks for old .lock files and tries to create a
+	 * usable lock instance from them
+	 */
+	public static function legacy(ModelWithContent $model): static|null
+	{
+		$kirby = $model->kirby();
+		$file  = static::legacyFile($model);
+		$id    = '/' . $model->id();
+
+		// no legacy lock file? no lock.
+		if (file_exists($file) === false) {
+			return null;
+		}
+
+		$data = Data::read($file, 'yml', fail: false)[$id] ?? [];
+
+		// no valid lock entry? no lock.
+		if (isset($data['lock']) === false) {
+			return null;
+		}
+
+		// has the lock been unlocked? no lock.
+		if (isset($data['unlock']) === true) {
+			return null;
+		}
+
+		return new static(
+			user: $kirby->user($data['lock']['user']),
+			modified: $data['lock']['time'],
+			legacy: true
+		);
+	}
+
+	/**
+	 * Returns the absolute path to a legacy lock file
+	 */
+	public static function legacyFile(ModelWithContent $model): string
+	{
+		$root = $model::CLASS_ALIAS === 'file' ? dirname($model->root()) : $model->root();
+		return $root . '/.lock';
+	}
+
 	/**
 	 * Returns the timestamp when the locked content has
 	 * been updated. You can pass a format to get a useful,
@@ -147,6 +206,7 @@ public function modified(
 	public function toArray(): array
 	{
 		return [
+			'isLegacy' => $this->isLegacy(),
 			'isLocked' => $this->isLocked(),
 			'modified' => $this->modified('c'),
 			'user'     => [

tests/Content/LockTest.php

@@ -319,6 +319,7 @@ public function testToArray()
 		);
 
 		$this->assertSame([
+			'isLegacy' => false,
 			'isLocked' => true,
 			'modified' => date('c', $modified),
 			'user'     => [

tests/Panel/Areas/SiteTest.php

@@ -38,6 +38,7 @@ public function testPage(): void
 
 		$this->assertSame('default', $props['blueprint']);
 		$this->assertSame([
+			'isLegacy' => false,
 			'isLocked' => false,
 			'modified' => null,
 			'user'     => [
@@ -95,6 +96,7 @@ public function testPageFile(): void
 
 		$this->assertSame('image', $props['blueprint']);
 		$this->assertSame([
+			'isLegacy' => false,
 			'isLocked' => false,
 			'modified' => null,
 			'user'     => [
@@ -190,6 +192,7 @@ public function testSiteFile(): void
 
 		$this->assertSame('image', $props['blueprint']);
 		$this->assertSame([
+			'isLegacy' => false,
 			'isLocked' => false,
 			'modified' => null,
 			'user'     => [