diff options
author | Morris Jobke <hey@morrisjobke.de> | 2017-03-13 16:03:11 -0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-03-13 16:03:11 -0600 |
commit | 5730c587ec72f36f57ea07b894d5b6052f806a88 (patch) | |
tree | fd3afd3001dcf146a633652dfdfd129bfc79d82e | |
parent | af16416de51a40e3fbb1e8499bbe8bf3b77ab5af (diff) | |
parent | beb3f92c4da65a93a6f8df9b3772a221cf3637e5 (diff) | |
download | nextcloud-server-5730c587ec72f36f57ea07b894d5b6052f806a88.tar.gz nextcloud-server-5730c587ec72f36f57ea07b894d5b6052f806a88.zip |
Merge pull request #3827 from nextcloud/issue-3826-remove-double-password-confirmation
Remove the double password confirmation on changing cron
-rw-r--r-- | core/js/public/appconfig.js | 1 | ||||
-rw-r--r-- | settings/js/admin.js | 9 |
2 files changed, 7 insertions, 3 deletions
diff --git a/core/js/public/appconfig.js b/core/js/public/appconfig.js index d84ddaab404..bba39c8b805 100644 --- 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 index 094b12b2bab..985e318e34b 100644 --- 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'); + } + }); } } }); |