diff --git a/src/CoreBundle/Entity/ResourceFile.php b/src/CoreBundle/Entity/ResourceFile.php index e787231d83d..3297c2dddab 100644 --- a/src/CoreBundle/Entity/ResourceFile.php +++ b/src/CoreBundle/Entity/ResourceFile.php @@ -293,7 +293,7 @@ public function setMimeType(?string $mimeType): self return $this; } - public function getOriginalName(): string + public function getOriginalName(): ?string { return $this->originalName; } diff --git a/src/CoreBundle/Resources/views/Admin/files_info.html.twig b/src/CoreBundle/Resources/views/Admin/files_info.html.twig index 1ad492fc83d..e8654d2a932 100644 --- a/src/CoreBundle/Resources/views/Admin/files_info.html.twig +++ b/src/CoreBundle/Resources/views/Admin/files_info.html.twig @@ -24,36 +24,40 @@ {% for file in files %} + {% set title = file.title|default('N/A'|trans) %} + {% set originalName = file.originalName ?? file.title ?? null %} + {% set courseTitle = + (file.resourceNode and file.resourceNode.resourceLinks|length > 0) + ? (file.resourceNode.resourceLinks|first.course.title|default('N/A'|trans)) + : ('N/A'|trans) + %} + {% set username = + (file.resourceNode and file.resourceNode.resourceLinks|length > 0) + ? (file.resourceNode.resourceLinks|first.user.username|default('N/A'|trans)) + : (file.resourceNode and file.resourceNode.creator + ? (file.resourceNode.creator.username|default('N/A'|trans)) + : ('N/A'|trans)) + %} + {% set fileUrl = fileUrls[file.id]|default('') %} + {% set filePath = filePaths[file.id]|default('') %} + {% set resourceNodeId = file.resourceNode ? file.resourceNode.id : 'N/A' %} + - {{ file.title }} - {{ file.originalName }} - - {% if file.resourceNode and file.resourceNode.resourceLinks|length > 0 %} - {{ file.resourceNode.resourceLinks|first.course.title ?? 'N/A' }} - {% else %} - {{ 'N/A'|trans }} - {% endif %} - - - {% if file.resourceNode and file.resourceNode.resourceLinks|length > 0 %} - {{ file.resourceNode.resourceLinks|first.user.username ?? 'N/A' }} - {% elseif file.resourceNode and file.resourceNode.creator %} - {{ file.resourceNode.creator.username ?? 'N/A' }} - {% else %} - {{ 'N/A'|trans }} - {% endif %} - + {{ title }} + {{ originalName|default('N/A'|trans) }} + {{ courseTitle }} + {{ username }} {{ 'View'|trans }} @@ -121,16 +125,16 @@ var resourceNodeId = button.getAttribute('data-resource-node-id'); var resourceFileId = button.getAttribute('data-resource-file-id'); - document.getElementById('file-title').textContent = title; - document.getElementById('file-mime-type').textContent = mimeType; - document.getElementById('file-original-name').textContent = originalName; - document.getElementById('file-size').textContent = size + ' bytes'; - document.getElementById('file-course').textContent = course; - document.getElementById('file-user').textContent = user; - document.getElementById('file-path').textContent = filePath; - document.getElementById('file-url').href = fileUrl; - document.getElementById('resource-node-id').textContent = resourceNodeId; - document.getElementById('resource-file-id').textContent = resourceFileId; + document.getElementById('file-title').textContent = title || ''; + document.getElementById('file-mime-type').textContent = mimeType || ''; + document.getElementById('file-original-name').textContent = originalName || ''; + document.getElementById('file-size').textContent = (size || 0) + ' bytes'; + document.getElementById('file-course').textContent = course || ''; + document.getElementById('file-user').textContent = user || ''; + document.getElementById('file-path').textContent = filePath || ''; + document.getElementById('file-url').href = fileUrl || '#'; + document.getElementById('resource-node-id').textContent = resourceNodeId || ''; + document.getElementById('resource-file-id').textContent = resourceFileId || ''; modal.style.display = "block"; }; @@ -156,7 +160,7 @@ copyPathButton.classList.add('mdi-content-copy'); }, 2000); }, function(err) { - alert('Failed to copy: ', err); + alert('Failed to copy: ' + err); }); }; });