From e31d74037ad0e7be60393bc5c6c5b74ef686dd94 Mon Sep 17 00:00:00 2001 From: Jörn Friedrich Dreyer Date: Wed, 27 Mar 2013 15:55:44 +0100 Subject: use local variable to walk DOM only once --- apps/files/js/filelist.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'apps/files/js') diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index 4fdb76b1026..70c3332ccaf 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -354,7 +354,8 @@ var FileList={ $(document).ready(function(){ // handle upload events - $('#file_upload_start').on('fileuploaddrop', function(e, data) { + var file_upload_start = $('#file_upload_start'); + file_upload_start.on('fileuploaddrop', function(e, data) { // only handle drop to dir if fileList exists if ($('#fileList').length > 0) { var dropTarget = $(e.originalEvent.target).closest('tr'); @@ -377,7 +378,7 @@ $(document).ready(function(){ } } }); - $('#file_upload_start').on('fileuploadadd', function(e, data) { + file_upload_start.on('fileuploadadd', function(e, data) { // only add to fileList if it exists if ($('#fileList').length > 0) { @@ -421,7 +422,7 @@ $(document).ready(function(){ } } }); - $('#file_upload_start').on('fileuploaddone', function(e, data) { + file_upload_start.on('fileuploaddone', function(e, data) { // only update the fileList if it exists if ($('#fileList').length > 0) { var response; @@ -472,7 +473,7 @@ $(document).ready(function(){ } } }); - $('#file_upload_start').on('fileuploadfail', function(e, data) { + file_upload_start.on('fileuploadfail', function(e, data) { // only update the fileList if it exists // cleanup files, error notification has been shown by fileupload code var tr = data.context; -- cgit v1.2.3