Merge pull request #6814 from getkirby/fix/6786-file-preview-links

Fix file preview links

src/Cms/File.php

@@ -630,12 +630,6 @@ public function previewUrl(): string|null
 			case 'page':
 				$preview = $parent->blueprint()->preview();
 
-				// user has no permission to preview page,
-				// also return null for file preview
-				if ($preview === false) {
-					return null;
-				}
-
 				// the page has a custom preview setting,
 				// thus the file is only accessible through
 				// the direct media URL

tests/Cms/Files/FileTest.php

@@ -769,7 +769,7 @@ public function testPreviewUrlUnauthenticated()
 		]);
 
 		$file = $page->file('test.pdf');
-		$this->assertNull($file->previewUrl());
+		$this->assertSame('/media/pages/test/' . $file->mediaHash() . '/test.pdf', $file->previewUrl());
 	}
 
 	public function testPreviewUrlForDraft()
@@ -852,7 +852,7 @@ public function testPreviewUrlForPageWithDeniedPreviewSetting()
 		$app->impersonate('[email protected]');
 
 		$file = $app->file('test/test.pdf');
-		$this->assertNull($file->previewUrl());
+		$this->assertSame('/media/pages/test/' . $file->mediaHash() . '/test.pdf', $file->previewUrl());
 	}
 
 	public function testPreviewUrlForPageWithCustomPreviewSetting()