From d873f0da3500e1f9eef89d4b73ec635f9c927636 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Schie=C3=9Fle?= <schiessle@owncloud.com>
Date: Fri, 14 Dec 2012 13:13:01 +0100
Subject: [PATCH] autofocus to password input after clicking the checkbox; code
 restructured; debug output added

---
 core/js/share.js | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

diff --git a/core/js/share.js b/core/js/share.js
index c218c2c123..98ceceb3a4 100644
--- a/core/js/share.js
+++ b/core/js/share.js
@@ -526,29 +526,23 @@ $(document).ready(function() {
 			var itemType = $('#dropdown').data('item-type');
 			var itemSource = $('#dropdown').data('item-source');
 			OC.Share.share(itemType, itemSource, OC.Share.SHARE_TYPE_LINK, '', OC.PERMISSION_READ);
+		} else {
+			$('#linkPassText').focus();
 		}
 	});
 
 	$('#linkPassText').live('focusout keyup', function(event) {
-		if ( event.type == 'focusout' || event.keyCode == 13 ) {
+		if ( $('#linkPassText').val() != '' && (event.type == 'focusout' || event.keyCode == 13) ) {
 			var itemType = $('#dropdown').data('item-type');
 			var itemSource = $('#dropdown').data('item-source');
-			if ( $('#linkPassText').attr('passwordChanged') == 'true' ) {
-				OC.Share.share(itemType, itemSource, OC.Share.SHARE_TYPE_LINK, $(this).val(), OC.PERMISSION_READ, function() {
-					$('#linkPassText').val('');
-					$('#linkPassText').attr('placeholder', t('core', 'Password protected'));
-					$('#linkPassText').attr('passwordChanged', 'false');
-				});
-			}
+			OC.Share.share(itemType, itemSource, OC.Share.SHARE_TYPE_LINK, $(this).val(), OC.PERMISSION_READ, function() {
+				console.log("password set to: '" + $('#linkPassText').val() +"' by event: " + event.type);
+				$('#linkPassText').val('');
+				$('#linkPassText').attr('placeholder', t('core', 'Password protected'));
+			});
 		}
 	});
 
-	$('#linkPassText').live('focusout keyup', function(event) {
-		if ( event.keyCode != 13 ) {
-			$('#linkPassText').attr('passwordChanged', 'true');	
-		}
-	});
-	
 	$('#expirationCheckbox').live('click', function() {
 		if (this.checked) {
 			OC.Share.showExpirationDate('');
-- 
GitLab