Skip to content

Commit

Permalink
do not break personal settings page is viewer is not there
Browse files Browse the repository at this point in the history
Signed-off-by: Simon L <[email protected]>
  • Loading branch information
szaimen committed Feb 7, 2024
1 parent ed3a1e7 commit b0c57ed
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions apps/settings/lib/Settings/Personal/ServerDevNotice.php
Original file line number Diff line number Diff line change
Expand Up @@ -78,11 +78,8 @@ public function getForm(): TemplateResponse {

$hasInitialState = false;

// viewer is default enabled and this makes a zero-cost assertion for Psalm
assert(class_exists(LoadViewer::class));

// If the Reasons to use Nextcloud.pdf file is here, let's init Viewer
if ($userFolder->nodeExists('Reasons to use Nextcloud.pdf')) {
// If the Reasons to use Nextcloud.pdf file is here, let's init Viewer, also check that Viewer is there
if (class_exists(LoadViewer::class) && $userFolder->nodeExists('Reasons to use Nextcloud.pdf')) {
$this->eventDispatcher->dispatch(LoadViewer::class, new LoadViewer());

Check failure on line 83 in apps/settings/lib/Settings/Personal/ServerDevNotice.php

View workflow job for this annotation

GitHub Actions / static-code-analysis

UndefinedClass

apps/settings/lib/Settings/Personal/ServerDevNotice.php:83:37: UndefinedClass: Class, interface or enum named OCA\Viewer\Event\LoadViewer does not exist (see https://psalm.dev/019)

Check failure on line 83 in apps/settings/lib/Settings/Personal/ServerDevNotice.php

View workflow job for this annotation

GitHub Actions / static-code-analysis

UndefinedClass

apps/settings/lib/Settings/Personal/ServerDevNotice.php:83:60: UndefinedClass: Class, interface or enum named OCA\Viewer\Event\LoadViewer does not exist (see https://psalm.dev/019)
$hasInitialState = true;
}
Expand Down

0 comments on commit b0c57ed

Please sign in to comment.