Fix OAuth external storage auth for step 2

Parameters are only available after the onCompletion promise is run

1 files changed, 48 insertions, 46 deletions

diff --git a/apps/files_external/js/oauth2.js b/apps/files_external/js/oauth2.js
index 84941437420..2556bf45cae 100644
--- a/apps/files_external/js/oauth2.js
+++ b/apps/files_external/js/oauth2.js
@@ -1,6 +1,6 @@
 $(document).ready(function() {
 
-	OCA.External.Settings.mountConfig.whenSelectAuthMechanism(function($tr, authMechanism, scheme) {
+	OCA.External.Settings.mountConfig.whenSelectAuthMechanism(function($tr, authMechanism, scheme, onCompletion) {
 		if (authMechanism === 'oauth2::oauth2') {
 			var config = $tr.find('.configuration');
 			config.append($(document.createElement('input'))
@@ -10,54 +10,56 @@ $(document).ready(function() {
 				.attr('name', 'oauth2_grant')
 			);
 
-			var configured = $tr.find('[data-parameter="configured"]');
-			if ($(configured).val() == 'true') {
-				$tr.find('.configuration input').attr('disabled', 'disabled');
-				$tr.find('.configuration').append($('<span/>').attr('id', 'access')
-						.text(t('files_external', 'Access granted')));
-			} else {
-				var client_id = $tr.find('.configuration [data-parameter="client_id"]').val();
-				var client_secret = $tr.find('.configuration [data-parameter="client_secret"]')
-					.val();
-				if (client_id != '' && client_secret != '') {
-					var params = {};
-					window.location.href.replace(/[?&]+([^=&]+)=([^&]*)/gi, function(m, key, value) {
-						params[key] = value;
-					});
-					if (params['code'] !== undefined) {
-						var token = $tr.find('.configuration [data-parameter="token"]');
-						var statusSpan = $tr.find('.status span');
-						statusSpan.removeClass();
-						statusSpan.addClass('waiting');
-						$.post(OC.filePath('files_external', 'ajax', 'oauth2.php'),
-								{
-									step: 2,
-									client_id: client_id,
-									client_secret: client_secret,
-									redirect: location.protocol + '//' + location.host + location.pathname,
-									code: params['code'],
-								}, function(result) {
-									if (result && result.status == 'success') {
-										$(token).val(result.data.token);
-										$(configured).val('true');
-										OCA.External.Settings.mountConfig.saveStorageConfig($tr, function(status) {
-											if (status) {
-												$tr.find('.configuration input').attr('disabled', 'disabled');
-												$tr.find('.configuration').append($('<span/>')
-														.attr('id', 'access')
-														.text(t('files_external', 'Access granted')));
-											}
-										});
-									} else {
-										OC.dialogs.alert(result.data.message,
-												t('files_external', 'Error configuring OAuth2')
-												);
+			onCompletion.then(function() {
+				var configured = $tr.find('[data-parameter="configured"]');
+				if ($(configured).val() == 'true') {
+					$tr.find('.configuration input').attr('disabled', 'disabled');
+					$tr.find('.configuration').append($('<span/>').attr('id', 'access')
+							.text(t('files_external', 'Access granted')));
+				} else {
+					var client_id = $tr.find('.configuration [data-parameter="client_id"]').val();
+					var client_secret = $tr.find('.configuration [data-parameter="client_secret"]')
+						.val();
+					if (client_id != '' && client_secret != '') {
+						var params = {};
+						window.location.href.replace(/[?&]+([^=&]+)=([^&]*)/gi, function(m, key, value) {
+							params[key] = value;
+						});
+						if (params['code'] !== undefined) {
+							var token = $tr.find('.configuration [data-parameter="token"]');
+							var statusSpan = $tr.find('.status span');
+							statusSpan.removeClass();
+							statusSpan.addClass('waiting');
+							$.post(OC.filePath('files_external', 'ajax', 'oauth2.php'),
+									{
+										step: 2,
+										client_id: client_id,
+										client_secret: client_secret,
+										redirect: location.protocol + '//' + location.host + location.pathname,
+										code: params['code'],
+									}, function(result) {
+										if (result && result.status == 'success') {
+											$(token).val(result.data.token);
+											$(configured).val('true');
+											OCA.External.Settings.mountConfig.saveStorageConfig($tr, function(status) {
+												if (status) {
+													$tr.find('.configuration input').attr('disabled', 'disabled');
+													$tr.find('.configuration').append($('<span/>')
+															.attr('id', 'access')
+															.text(t('files_external', 'Access granted')));
+												}
+											});
+										} else {
+											OC.dialogs.alert(result.data.message,
+													t('files_external', 'Error configuring OAuth2')
+													);
+										}
 									}
-								}
-						);
+							);
+						}
 					}
 				}
-			}
+			});
 		}
 	});