diff options
Diffstat (limited to 'data/assets/js/admin/tabs')
-rw-r--r-- | data/assets/js/admin/tabs/files.js | 59 |
1 files changed, 29 insertions, 30 deletions
diff --git a/data/assets/js/admin/tabs/files.js b/data/assets/js/admin/tabs/files.js index 6fcdde7..0e26b2f 100644 --- a/data/assets/js/admin/tabs/files.js +++ b/data/assets/js/admin/tabs/files.js @@ -165,40 +165,39 @@ jQuery(function($) { }); $('#file-actions').on('click', 'a', function() { - var action_id = $(this).data('id'); + var action_id = $(this).data('id'), + data = $('#files .active').data(); // hide dropdown $('body').trigger('click'); - if (action_id == 'download') { - // alert('TODO: download file'); - var url = $('#files .active').data('url'); - if (url) - location.href = url; - } else if (action_id == 'move') { - alert('TODO: move file'); - } else if (action_id == 'delete') { - var data = $('#files .active').data(); - - if (confirm(TEMPLATES.run('delete_confirm', data))) { - send('file/delete', { - path: data.path, - }).always(function() { - // TODO: need loading handler - }).fail(function(r) { - var error = r.responseText; - - try { - var data = $.parseJSON(r.responseText); - if (data.error) - error = data.error; - } catch (e) {} - - alert("Error: " + error); - }).done(function() { - // reload file list - reload(); - }); + if (data) { + if (action_id == 'download') { + if (data.url) + location.href = url; + } else if (action_id == 'move') { + alert('TODO: move file'); + } else if (action_id == 'delete') { + if (confirm(TEMPLATES.run('delete_confirm', data))) { + send('file/delete', { + path: data.path, + }).always(function() { + // TODO: need loading handler + }).fail(function(r) { + var error = r.responseText; + + try { + var data = $.parseJSON(r.responseText); + if (data.error) + error = data.error; + } catch (e) {} + + alert("Error: " + error); + }).done(function() { + // reload file list + reload(); + }); + } } } |