[Pkg-owncloud-commits] [owncloud] 08/122: remove status's and adjust js

David Prévot taffit at moszumanska.debian.org
Sat May 9 00:00:01 UTC 2015


This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch stable8
in repository owncloud.

commit e3ec1a8bb8b913bd176b76bd59f8c0c209aff5cb
Author: Clark Tomlinson <fallen013 at gmail.com>
Date:   Wed Apr 22 10:41:47 2015 -0400

    remove status's and adjust js
---
 apps/encryption/controller/recoverycontroller.php | 42 ++++++++-------
 apps/encryption/js/settings-admin.js              | 65 +++++++++++++----------
 apps/encryption/js/settings-personal.js           | 48 ++++++++++-------
 3 files changed, 88 insertions(+), 67 deletions(-)

diff --git a/apps/encryption/controller/recoverycontroller.php b/apps/encryption/controller/recoverycontroller.php
index bf548f2..8ae37d9 100644
--- a/apps/encryption/controller/recoverycontroller.php
+++ b/apps/encryption/controller/recoverycontroller.php
@@ -91,14 +91,14 @@ class RecoveryController extends Controller {
 
 		if (isset($adminEnableRecovery) && $adminEnableRecovery === '1') {
 			if ($this->recovery->enableAdminRecovery($recoveryPassword)) {
-				return new DataResponse(['status' => 'success', 'data' => array('message' => (string)$this->l->t('Recovery key successfully enabled'))]);
+				return new DataResponse(['data' => ['message' => (string)$this->l->t('Recovery key successfully enabled')]]);
 			}
-			return new DataResponse(['data' => array('message' => (string)$this->l->t('Could not enable recovery key. Please check your recovery key password!'))], 500);
+			return new DataResponse(['data' => ['message' => (string)$this->l->t('Could not enable recovery key. Please check your recovery key password!')]], 500);
 		} elseif (isset($adminEnableRecovery) && $adminEnableRecovery === '0') {
 			if ($this->recovery->disableAdminRecovery($recoveryPassword)) {
-				return new DataResponse(['data' => array('message' => (string)$this->l->t('Recovery key successfully disabled'))]);
+				return new DataResponse(['data' => ['message' => (string)$this->l->t('Recovery key successfully disabled')]]);
 			}
-			return new DataResponse(['data' => array('message' => (string)$this->l->t('Could not disable recovery key. Please check your recovery key password!'))], 500);
+			return new DataResponse(['data' => ['message' => (string)$this->l->t('Could not disable recovery key. Please check your recovery key password!')]], 500);
 		}
 		// this response should never be sent but just in case.
 		return new DataResponse(['data' => ['message' => (string)$this->l->t('Missing parameters')]], 500);
@@ -114,22 +114,22 @@ class RecoveryController extends Controller {
 		//check if both passwords are the same
 		if (empty($oldPassword)) {
 			$errorMessage = (string)$this->l->t('Please provide the old recovery password');
-			return new DataResponse(array('data' => array('message' => $errorMessage)), 500);
+			return new DataResponse(['data' => ['message' => $errorMessage]], 500);
 		}
 
 		if (empty($newPassword)) {
 			$errorMessage = (string)$this->l->t('Please provide a new recovery password');
-			return new DataResponse (array('data' => array('message' => $errorMessage)), 500);
+			return new DataResponse (['data' => ['message' => $errorMessage]], 500);
 		}
 
 		if (empty($confirmPassword)) {
 			$errorMessage = (string)$this->l->t('Please repeat the new recovery password');
-			return new DataResponse(array('data' => array('message' => $errorMessage)), 500);
+			return new DataResponse(['data' => ['message' => $errorMessage]], 500);
 		}
 
 		if ($newPassword !== $confirmPassword) {
 			$errorMessage = (string)$this->l->t('Repeated recovery key password does not match the provided recovery key password');
-			return new DataResponse(array('data' => array('message' => $errorMessage)), 500);
+			return new DataResponse(['data' => ['message' => $errorMessage]], 500);
 		}
 
 		$result = $this->recovery->changeRecoveryKeyPassword($newPassword,
@@ -137,19 +137,18 @@ class RecoveryController extends Controller {
 
 		if ($result) {
 			return new DataResponse(
-				array(
-					'status' => 'success',
+				[
 					'data' => [
 						'message' => (string)$this->l->t('Password successfully changed.')]
-				)
+				]
 			);
 		}
 		return new DataResponse(
-			array(
+			[
 				'data' => [
 					'message' => (string)$this->l->t('Could not change the password. Maybe the old password was not correct.')
 				]
-			), 500);
+			], 500);
 	}
 
 	/**
@@ -164,22 +163,29 @@ class RecoveryController extends Controller {
 			$result = $this->recovery->setRecoveryForUser($userEnableRecovery);
 
 			if ($result) {
+				if ($userEnableRecovery === '0') {
+					return new DataResponse(
+						[
+							'data' => [
+								'message' => (string)$this->l->t('Recovery Key disabled')]
+						]
+					);
+				}
 				return new DataResponse(
-					array(
-						'status' => 'success',
+					[
 						'data' => [
 							'message' => (string)$this->l->t('Recovery Key enabled')]
-					)
+					]
 				);
 			}
 
 		}
 		return new DataResponse(
-			array(
+			[
 				'data' => [
 					'message' => (string)$this->l->t('Could not enable the recovery key, please try again or contact your administrator')
 				]
-			), 500);
+			], 500);
 	}
 
 }
diff --git a/apps/encryption/js/settings-admin.js b/apps/encryption/js/settings-admin.js
index bb539f6..fdc53c5 100644
--- a/apps/encryption/js/settings-admin.js
+++ b/apps/encryption/js/settings-admin.js
@@ -7,52 +7,59 @@
  * See the COPYING-README file.
  */
 
-$(document).ready(function(){
+$(document).ready(function () {
 
-	$( 'input:radio[name="adminEnableRecovery"]' ).change(
-		function() {
-			var recoveryStatus = $( this ).val();
-			var oldStatus = (1+parseInt(recoveryStatus, 10)) % 2;
-			var recoveryPassword = $( '#encryptionRecoveryPassword' ).val();
-			var confirmPassword = $( '#repeatEncryptionRecoveryPassword' ).val();
+	$('input:radio[name="adminEnableRecovery"]').change(
+		function () {
+			var recoveryStatus = $(this).val();
+			var oldStatus = (1 + parseInt(recoveryStatus)) % 2;
+			var recoveryPassword = $('#encryptionRecoveryPassword').val();
+			var confirmPassword = $('#repeatEncryptionRecoveryPassword').val();
 			OC.msg.startSaving('#encryptionSetRecoveryKey .msg');
 			$.post(
 				OC.generateUrl('/apps/encryption/ajax/adminRecovery'),
-				{ adminEnableRecovery: recoveryStatus,
+				{
+					adminEnableRecovery: recoveryStatus,
 					recoveryPassword: recoveryPassword,
-					confirmPassword: confirmPassword },
-				function( result ) {
-					OC.msg.finishedSaving('#encryptionSetRecoveryKey .msg', result);
-					if (result.status === "error") {
-						$('input:radio[name="adminEnableRecovery"][value="'+oldStatus.toString()+'"]')
-							.attr("checked", "true");
+					confirmPassword: confirmPassword
+				}
+			).done(function (data) {
+					OC.msg.finishedSuccess('#encryptionSetRecoveryKey .msg', data.data.message);
+
+					if (recoveryStatus === "0") {
+						$('p[name="changeRecoveryPasswordBlock"]').addClass("hidden");
 					} else {
-						if (recoveryStatus === "0") {
-							$('p[name="changeRecoveryPasswordBlock"]').addClass("hidden");
-						} else {
-							$('input:password[name="changeRecoveryPassword"]').val("");
-							$('p[name="changeRecoveryPasswordBlock"]').removeClass("hidden");
-						}
+						$('input:password[name="changeRecoveryPassword"]').val("");
+						$('p[name="changeRecoveryPasswordBlock"]').removeClass("hidden");
 					}
-				}
-			);
+				})
+				.fail(function (jqXHR) {
+					$('input:radio[name="adminEnableRecovery"][value="' + oldStatus.toString() + '"]').attr("checked", "true");
+					OC.msg.finishedError('#encryptionSetRecoveryKey .msg', JSON.parse(jqXHR.responseText).data.message);
+				});
 		}
 	);
 
 	// change recovery password
 
-	$('button:button[name="submitChangeRecoveryKey"]').click(function() {
+	$('button:button[name="submitChangeRecoveryKey"]').click(function () {
 		var oldRecoveryPassword = $('#oldEncryptionRecoveryPassword').val();
 		var newRecoveryPassword = $('#newEncryptionRecoveryPassword').val();
 		var confirmNewPassword = $('#repeatedNewEncryptionRecoveryPassword').val();
 		OC.msg.startSaving('#encryptionChangeRecoveryKey .msg');
 		$.post(
-				OC.generateUrl('/apps/encryption/ajax/changeRecoveryPassword'),
-			{ oldPassword: oldRecoveryPassword, newPassword: newRecoveryPassword, confirmPassword: confirmNewPassword },
-			function( data ) {
-					OC.msg.finishedSaving('#encryptionChangeRecoveryKey .msg', data);
-				}
-		);
+			OC.generateUrl('/apps/encryption/ajax/changeRecoveryPassword'),
+			{
+				oldPassword: oldRecoveryPassword,
+				newPassword: newRecoveryPassword,
+				confirmPassword: confirmNewPassword
+			}
+		).done(function (data) {
+				OC.msg.finishedSuccess('#encryptionChangeRecoveryKey .msg', data.data.message);
+			})
+			.fail(function (jqXHR) {
+				OC.msg.finishedError('#encryptionChangeRecoveryKey .msg', JSON.parse(jqXHR.responseText).data.message);
+			});
 	});
 
 });
diff --git a/apps/encryption/js/settings-personal.js b/apps/encryption/js/settings-personal.js
index e36f10a..4728da8 100644
--- a/apps/encryption/js/settings-personal.js
+++ b/apps/encryption/js/settings-personal.js
@@ -9,35 +9,43 @@ if (!OC.Encryption) {
 }
 
 OC.Encryption = {
-	updatePrivateKeyPassword: function() {
+	updatePrivateKeyPassword: function () {
 		var oldPrivateKeyPassword = $('input:password[id="oldPrivateKeyPassword"]').val();
 		var newPrivateKeyPassword = $('input:password[id="newPrivateKeyPassword"]').val();
 		OC.msg.startSaving('#encryption .msg');
 		$.post(
 			OC.generateUrl('/apps/encryption/ajax/updatePrivateKeyPassword'),
-			{oldPassword: oldPrivateKeyPassword, newPassword: newPrivateKeyPassword}
-		).success(function (response) {
-			OC.msg.finishedSuccess('#encryption .msg', response.message);
-		}).fail(function (response) {
-			OC.msg.finishedError('#encryption .msg', response.responseJSON.message);
-		});
+			{
+				oldPassword: oldPrivateKeyPassword,
+				newPassword: newPrivateKeyPassword
+			}
+		).done(function (data) {
+				OC.msg.finishedSuccess('#encryption .msg', data.data.message);
+			})
+			.fail(function (jqXHR) {
+				OC.msg.finishedError('#encryption .msg', JSON.parse(jqXHR.responseText).data.message);
+			});
 	}
 };
 
-$(document).ready(function(){
+$(document).ready(function () {
 
 	// Trigger ajax on recoveryAdmin status change
-	$( 'input:radio[name="userEnableRecovery"]' ).change(
-		function() {
-			var recoveryStatus = $( this ).val();
+	$('input:radio[name="userEnableRecovery"]').change(
+		function () {
+			var recoveryStatus = $(this).val();
 			OC.msg.startAction('#userEnableRecovery .msg', 'Updating recovery keys. This can take some time...');
 			$.post(
-					OC.generateUrl('/apps/encryption/ajax/userSetRecovery'),
-				{ userEnableRecovery: recoveryStatus },
-				function( data ) {
-					OC.msg.finishedAction('#userEnableRecovery .msg', data);
+				OC.generateUrl('/apps/encryption/ajax/userSetRecovery'),
+				{
+					userEnableRecovery: recoveryStatus
 				}
-			);
+			).done(function (data) {
+					OC.msg.finishedSuccess('#userEnableRecovery .msg', data.data.message);
+				})
+				.fail(function (jqXHR) {
+					OC.msg.finishedError('#userEnableRecovery .msg', JSON.parse(jqXHR.responseText).data.message);
+				});
 			// Ensure page is not reloaded on form submit
 			return false;
 		}
@@ -45,12 +53,12 @@ $(document).ready(function(){
 
 	// update private key password
 
-	$('input:password[name="changePrivateKeyPassword"]').keyup(function(event) {
+	$('input:password[name="changePrivateKeyPassword"]').keyup(function (event) {
 		var oldPrivateKeyPassword = $('input:password[id="oldPrivateKeyPassword"]').val();
 		var newPrivateKeyPassword = $('input:password[id="newPrivateKeyPassword"]').val();
-		if (newPrivateKeyPassword !== '' && oldPrivateKeyPassword !== '' ) {
+		if (newPrivateKeyPassword !== '' && oldPrivateKeyPassword !== '') {
 			$('button:button[name="submitChangePrivateKeyPassword"]').removeAttr("disabled");
-			if(event.which === 13) {
+			if (event.which === 13) {
 				OC.Encryption.updatePrivateKeyPassword();
 			}
 		} else {
@@ -58,7 +66,7 @@ $(document).ready(function(){
 		}
 	});
 
-	$('button:button[name="submitChangePrivateKeyPassword"]').click(function() {
+	$('button:button[name="submitChangePrivateKeyPassword"]').click(function () {
 		OC.Encryption.updatePrivateKeyPassword();
 	});
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/owncloud.git



More information about the Pkg-owncloud-commits mailing list