summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMorris Jobke <hey@morrisjobke.de>2017-03-13 16:03:11 -0600
committerGitHub <noreply@github.com>2017-03-13 16:03:11 -0600
commit5730c587ec72f36f57ea07b894d5b6052f806a88 (patch)
treefd3afd3001dcf146a633652dfdfd129bfc79d82e
parentaf16416de51a40e3fbb1e8499bbe8bf3b77ab5af (diff)
parentbeb3f92c4da65a93a6f8df9b3772a221cf3637e5 (diff)
downloadnextcloud-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.js1
-rw-r--r--settings/js/admin.js9
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');
+ }
+ });
}
}
});