diff options
| author | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2024-09-30 10:15:54 +0000 |
|---|---|---|
| committer | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2024-09-30 10:15:54 +0000 |
| commit | 8e9222919fa9cc06dcfbbeb10e12667c6bd95b26 (patch) | |
| tree | 74354ce945c00d09a56f90c61188d1d8c180276b /lib/filesystem/LibraryFile.php | |
| parent | b4b0e9687ec5e9dd5eb6eeb629e6486c8efb8744 (diff) | |
prevent many warnings regarding php8 on main, fixes #4631
Closes #4631
Merge request studip/studip!3448
Diffstat (limited to 'lib/filesystem/LibraryFile.php')
| -rw-r--r-- | lib/filesystem/LibraryFile.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/filesystem/LibraryFile.php b/lib/filesystem/LibraryFile.php index c9524e3..8de46f8 100644 --- a/lib/filesystem/LibraryFile.php +++ b/lib/filesystem/LibraryFile.php @@ -46,9 +46,9 @@ class LibraryFile extends StandardFile $file->size = '0'; $file->mime_type = ''; $file->metadata = $document->toJson(); - $file->user_id = $user_id ? $user_id : $GLOBALS['user']->id; + $file->user_id = $user_id ?: $GLOBALS['user']->id; $file->filetype = get_called_class(); - if ($document->csl_data['URL'] || $document->opac_link) { + if (!empty($document->csl_data['URL']) || $document->opac_link) { $file->metadata['url'] = $document->opac_link ?: $document->csl_data['URL']; $file->metadata['access_type'] = 'redirect'; } @@ -56,10 +56,10 @@ class LibraryFile extends StandardFile $file_ref = new FileRef(); $file_ref->file_id = $file->id; - $file_ref->folder_id = $folder_id ? $folder_id : ''; + $file_ref->folder_id = $folder_id ?: ''; $file_ref->name = $file->name; $file_ref->downloads = 0; - $file_ref->description = $document->csl_data['description'] ? $document->csl_data['description'] : ''; + $file_ref->description = trim($document->csl_data['description'] ?? ''); $file_ref->content_terms_of_use_id = ContentTermsOfUse::findDefault()->id; $file_ref->user_id = $file->user_id; $file_ref->store(); |