From a3862c43dada59f2f740b53c187f6955aa4084e9 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=B6rn=20Friedrich=20Dreyer?= Date: Wed, 27 Mar 2013 15:55:44 +0100 Subject: [PATCH] use local variable to walk DOM only once --- apps/files/js/filelist.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) 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; -- 2.39.5