Merge pull request #4236 from nextcloud/backport-3827-remove-double-p…

…assword-confirmation [stable11] Remove the double password confirmation on changing cron
nextcloud · Apr 6, 2017 · aabcb04 · aabcb04
2 parents 125c0eb + c5d9807
commit aabcb04
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/core/js/public/appconfig.js b/core/js/public/appconfig.js
@@ -38,6 +38,7 @@ OCP.AppConfig = {
 			return;
 		}
 
+		options = options || {};
 		$.ajax({
 			type: method.toUpperCase(),
 			url: OC.linkToOCS('apps/provisioning_api/api/v1', 2) + 'config/apps' + endpoint,

diff --git a/settings/js/admin.js b/settings/js/admin.js
@@ -46,9 +46,12 @@ $(document).ready(function(){
 		if($(this).is(':checked')){
 			var mode = $(this).val();
 			if (mode === 'ajax' || mode === 'webcron' || mode === 'cron') {
-				OCP.AppConfig.setValue('core', 'backgroundjobs_mode', mode);
-				// clear cron errors on background job mode change
-				OCP.AppConfig.deleteKey('core', 'cronErrors');
+				OCP.AppConfig.setValue('core', 'backgroundjobs_mode', mode, {
+					success: function() {
+						// clear cron errors on background job mode change
+						OCP.AppConfig.deleteKey('core', 'cronErrors');
+					}
+				});
 			}
 		}
 	});