Skip to content
Snippets Groups Projects
Commit cca44456 authored by Jan-Hendrik Willms's avatar Jan-Hendrik Willms Committed by David Siegfried
Browse files

allow updating of mvv files, fixes #696 

Closes #696

Merge request !3095
parent 41c5d920
Branches
No related tags found
No related merge requests found
Show whitespace changes
Inline Side-by-side
app/controllers/materialien/files.php
+ 33
− 28
View file @ cca44456
...@@ -362,29 +362,37 @@ class Materialien_FilesController extends MVVController ...@@ -362,29 +362,37 @@ class Materialien_FilesController extends MVVController
public function upload_attachment_action() public function upload_attachment_action()
{ {
if ($GLOBALS['user']->id === "nobody") { $user = User::findCurrent();
if (!$user) {
throw new AccessDeniedException(); throw new AccessDeniedException();
} }
$mvvfile_id = Request::option('mvvfile_id');
$document_id = Request::option('document_id');
$file = $_FILES['file']; $file = $_FILES['file'];
$output = [ $output = [
'name' => $file['name'], 'name' => $file['name'],
'size' => $file['size']]; 'size' => $file['size'],
'mvvfile_id' => $mvvfile_id,
$mvvfile_id = Request::option('mvvfile_id'); 'range_id' => Request::option('range_id', $mvvfile_id),
$output['mvvfile_id'] = $mvvfile_id; ];
$range_id = Request::option('range_id', $mvvfile_id);
$output['range_id'] = $range_id;
$file_language = Request::option('file_language');
$top_folder = $this->getTopFolder($mvvfile_id); $top_folder = $this->getTopFolder($output['mvvfile_id']);
$user = User::findCurrent();
if ($document_id) {
$file = File::find($document_id);
$file->mime_type = $_FILES['file']['type'] ?? get_mime_type($_FILES['file']['name']);
$file->size = $_FILES['file']['size'] ?? filesize($_FILES['file']['tmp_name']);
$file->connectWithDataFile($_FILES['file']['tmp_name']);
} else {
$file = StandardFile::create($_FILES['file']); $file = StandardFile::create($_FILES['file']);
$error = $top_folder->validateUpload($file, $GLOBALS['user']->id); }
if ($error != null) { $error = $top_folder->validateUpload($file, $user->id);
if ($error !== null) {
if (!$document_id) {
$file->delete(); $file->delete();
}
$this->response->set_status(400); $this->response->set_status(400);
$this->render_json(compact('error')); $this->render_json(compact('error'));
return; return;
...@@ -399,18 +407,15 @@ class Materialien_FilesController extends MVVController ...@@ -399,18 +407,15 @@ class Materialien_FilesController extends MVVController
return; return;
} }
$mvv_file_fileref = new MvvFileFileref([$mvvfile_id, $file_language]); $mvv_file_fileref = new MvvFileFileref([
$mvv_file_fileref->fileref_id = $file->getId(); $output['mvvfile_id'],
Request::option('file_language'),
]);
$mvv_file_fileref->fileref_id = $file->id;
$mvv_file_fileref->store(); $mvv_file_fileref->store();
$output['document_id'] = $file->getId(); $output['document_id'] = $file->id;
$output['icon'] = $file->getIcon(Icon::ROLE_CLICKABLE)->asImg(['class' => 'text-bottom']);
$output['icon'] = Icon::create(
FileManager::getIconNameForMimeType(
$file->getMimeType()
),
'clickable'
)->asImg(['class' => "text-bottom"]);
$this->render_json($output); $this->render_json($output);
} }
......
app/views/materialien/files/add_dokument.php
+ 24
− 2
View file @ cca44456
...@@ -47,7 +47,18 @@ ...@@ -47,7 +47,18 @@
<span class="icon"></span> <span class="icon"></span>
<span class="name"></span> <span class="name"></span>
<span class="size"></span> <span class="size"></span>
<a class="remove_attachment"><?= Icon::create('trash')->asImg(['class' => 'text-bottom']) ?></a> <button class="refresh_attachment as-link" data-language="<?= htmlReady($key) ?>">
<?= Icon::create('refresh')->asImg([
'class' => 'text-bottom',
'title' => _('Datei aktualisieren'),
]) ?>
</button>
<button class="remove_attachment as-link">
<?= Icon::create('trash')->asImg([
'class' => 'text-bottom',
'title' => _('Datei löschen'),
]) ?>
</button>
</li> </li>
</ul> </ul>
<div id="statusbar_container"> <div id="statusbar_container">
...@@ -79,7 +90,18 @@ ...@@ -79,7 +90,18 @@
<span class="icon"><?= Icon::create('file', Icon::ROLE_INFO, ['class' => 'text-bottom']); ?></span> <span class="icon"><?= Icon::create('file', Icon::ROLE_INFO, ['class' => 'text-bottom']); ?></span>
<span class="name"><?= htmlReady($documents[$key]->filename) ?></span> <span class="name"><?= htmlReady($documents[$key]->filename) ?></span>
<span class="size"></span> <span class="size"></span>
<a class="remove_attachment"><?= Icon::create('trash', 'clickable')->asImg(['class' => "text-bottom"]) ?></a> <button class="refresh_attachment as-link" data-language="<?= htmlReady($key) ?>">
<?= Icon::create('refresh')->asImg([
'class' => 'text-bottom',
'title' => _('Datei aktualisieren'),
]) ?>
</button>
<button class="remove_attachment as-link">
<?= Icon::create('trash')->asImg([
'class' => 'text-bottom',
'title' => _('Datei löschen'),
]) ?>
</button>
</li> </li>
<? endif; ?> <? endif; ?>
</ul> </ul>
......
lib/filesystem/MVVFolder.php
+ 1
− 1
View file @ cca44456
...@@ -82,7 +82,7 @@ class MVVFolder extends StandardFolder ...@@ -82,7 +82,7 @@ class MVVFolder extends StandardFolder
if ($folder) { if ($folder) {
$topfolder = $folder->getTypedFolder(); $topfolder = $folder->getTypedFolder();
} }
return $topfolder ?: null; return $topfolder ?? null;
} }
/** /**
......
lib/filesystem/StandardFolder.php
+ 5
− 4
View file @ cca44456
...@@ -224,14 +224,13 @@ class StandardFolder implements FolderType ...@@ -224,14 +224,13 @@ class StandardFolder implements FolderType
} }
/** /**
* @param FileType $newfile
* @param string $user_id * @param string $user_id
* @return string * @return string
*/ */
public function validateUpload(FileType $newfile, $user_id) public function validateUpload(FileType $file, $user_id)
{ {
$upload_type = FileManager::getUploadTypeConfig($this->range_id, $user_id); $upload_type = FileManager::getUploadTypeConfig($this->range_id, $user_id);
return $this->getValidationMessages($upload_type, $newfile); return $this->getValidationMessages($upload_type, $file);
} }
protected function getValidationMessages($upload_type, $newfile) protected function getValidationMessages($upload_type, $newfile)
...@@ -256,6 +255,8 @@ class StandardFolder implements FolderType ...@@ -256,6 +255,8 @@ class StandardFolder implements FolderType
if (in_array($ext, $types) && $upload_type['type'] === 'allow') { if (in_array($ext, $types) && $upload_type['type'] === 'allow') {
return sprintf(_('Sie dürfen den Dateityp %s nicht hochladen!'), $ext); return sprintf(_('Sie dürfen den Dateityp %s nicht hochladen!'), $ext);
} }
return null;
} }
/** /**
......
resources/assets/javascripts/mvv.js
+ 138
− 91
View file @ cca44456
...@@ -67,10 +67,17 @@ jQuery(function ($) { ...@@ -67,10 +67,17 @@ jQuery(function ($) {
}); });
$(document).on('click', '.stgfile .remove_attachment', function($event) { $(document).on('click', '.stgfile .remove_attachment', function($event) {
STUDIP.MVV.Document.remove_attachment($(this)); STUDIP.Dialog.confirm($gettext('Soll die Datei wirklich gelöscht werden?')).done(() => {
STUDIP.MVV.Document.remove_attachment(this);
});
return false; return false;
}); });
$(document).on('click', '.stgfile .refresh_attachment', (event) => {
STUDIP.MVV.Document.refresh_attachment(event.target);
event.preventDefault();
});
STUDIP.dialogReady( STUDIP.dialogReady(
function() { function() {
...@@ -663,27 +670,55 @@ STUDIP.MVV.Document = { ...@@ -663,27 +670,55 @@ STUDIP.MVV.Document = {
}) })
}, 100); }, 100);
}, },
remove_attachment: function(item) { refresh_attachment(item) {
jQuery.ajax({ const language = item.closest('button').dataset.language;
url: STUDIP.ABSOLUTE_URI_STUDIP + 'dispatch.php/materialien/files/delete_attachment', const document_id = item.closest('.stgfile').querySelector('[name="document_id"]').value;
data: {
mvvfile_id: jQuery('#mvvfile_id').val(), const input = document.createElement('input');
fileref_id: item.closest('li') input.type = 'file';
.find('input[name=document_id]') input.hidden = true;
.val()
}, input.addEventListener('cancel', () => {
type: 'POST' input.remove();
}); });
item.parents('td').find('.attachments').toggle(); input.addEventListener('change', () => {
item.closest('li') const fd = new FormData();
.fadeOut(300, function() { fd.append('file', input.files[0], input.files[0].name);
jQuery(this).remove(); fd.append('mvvfile_id', jQuery('#mvvfile_id').val());
fd.append('range_id', jQuery('#range_id').val());
fd.append('document_id', document_id);
fd.append('file_language', language);
const statusbar = $('#statusbar_container .statusbar')
.first()
.clone()
.show();
statusbar.appendTo('#statusbar_container');
STUDIP.MVV.Document.upload_file(fd, statusbar, true).then(() => {
input.remove();
});
});
item.parentNode.after(input);
input.click();
},
remove_attachment(item) {
const url = STUDIP.URLHelper.getURL('dispatch.php/materialien/files/delete_attachment', {
mvvfile_id: document.getElementById('mvvfile_id').value,
fileref_id: item.closest('li').querySelector('input[name=document_id]').value,
});
$.post(url).done(() => {
$(item).closest('td').find('.attachments').toggle();
$(item).closest('li').fadeOut(300, function () {
this.remove();
jQuery('#upload_chooser').show(); jQuery('#upload_chooser').show();
}); });
});
}, },
upload_from_input: function(input, file_language) { upload_from_input(input, file_language) {
STUDIP.MVV.Document.upload_files(input.files, file_language); STUDIP.MVV.Document.upload_files(input.files, file_language);
jQuery(input).val(''); input.value = '';
}, },
fileIDQueue: 1, fileIDQueue: 1,
upload_files: function(files, file_language) { upload_files: function(files, file_language) {
...@@ -701,9 +736,10 @@ STUDIP.MVV.Document = { ...@@ -701,9 +736,10 @@ STUDIP.MVV.Document = {
STUDIP.MVV.Document.upload_file(fd, statusbar); STUDIP.MVV.Document.upload_file(fd, statusbar);
} }
}, },
upload_file: function(formdata, statusbar) { upload_file(formdata, statusbar, update = false) {
return new Promise((resolve, reject) => {
$.ajax({ $.ajax({
xhr: function() { xhr() {
var xhrobj = $.ajaxSettings.xhr(); var xhrobj = $.ajaxSettings.xhr();
if (xhrobj.upload) { if (xhrobj.upload) {
xhrobj.upload.addEventListener( xhrobj.upload.addEventListener(
...@@ -733,8 +769,9 @@ STUDIP.MVV.Document = { ...@@ -733,8 +769,9 @@ STUDIP.MVV.Document = {
cache: false, cache: false,
data: formdata, data: formdata,
dataType: 'json' dataType: 'json'
}) }).done((data) => {
.done(function(data) { const language = formdata.get('file_language');
statusbar.find('.progress').css({ 'min-width': '100%', 'max-width': '100%' }); statusbar.find('.progress').css({ 'min-width': '100%', 'max-width': '100%' });
var file = jQuery('#fileselector_'+formdata.get('file_language')).find('.stgfiles > .stgfile') var file = jQuery('#fileselector_'+formdata.get('file_language')).find('.stgfiles > .stgfile')
.first() .first()
...@@ -754,18 +791,25 @@ STUDIP.MVV.Document = { ...@@ -754,18 +791,25 @@ STUDIP.MVV.Document = {
} }
file.find('.icon').html(data.icon); file.find('.icon').html(data.icon);
file.find('input[name=document_id]').attr('value', data.document_id); file.find('input[name=document_id]').attr('value', data.document_id);
jQuery('#fileviewer_'+formdata.get('file_language')).find('.stgfiles').append(file); if (update) {
jQuery('#fileselector_'+formdata.get('file_language')).toggle(); $(`#fileviewer_${language} .stgfiles`).empty().append(file);
jQuery('#fileselector_'+formdata.get('file_language')).parents('.attachments').toggle(); file.show();
jQuery('#fileselector_'+formdata.get('file_language')).parents('.attachments').find('span').toggle(); } else {
$(`#fileviewer_${language}`).find('.stgfiles').append(file);
$(`#fileselector_${language}`)
.toggle()
.parents('.attachments').toggle()
.find('span').toggle();
file.fadeIn(300); file.fadeIn(300);
}
statusbar.find('.progresstext').text(jQuery('#upload_received_data').text()); statusbar.find('.progresstext').text(jQuery('#upload_received_data').text());
statusbar.delay(1000).fadeOut(300, function() { statusbar.delay(1000).fadeOut(300, function() {
jQuery('#upload_chooser').hide(); $('#upload_chooser').hide();
jQuery(this).remove(); this.remove();
}); });
})
.fail(function(jqxhr, status, errorThrown) { resolve();
}).fail(function(jqxhr, status, errorThrown) {
var error = jqxhr.responseJSON.error; var error = jqxhr.responseJSON.error;
statusbar statusbar
...@@ -778,6 +822,9 @@ STUDIP.MVV.Document = { ...@@ -778,6 +822,9 @@ STUDIP.MVV.Document = {
jQuery(this).remove(); jQuery(this).remove();
}); });
}); });
reject(new Error(error));
});
}); });
} }
}; };
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment