From e31d74037ad0e7be60393bc5c6c5b74ef686dd94 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B6rn=20Friedrich=20Dreyer?= <jfd@butonic.de>
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 4fdb76b102..70c3332cca 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;
-- 
GitLab