Give users admin permission if it's missing

jokimaki · Jun 29, 2016 · 5b0b304 · 5b0b304
1 parent c5f4592
commit 5b0b304
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/library/solita_jenkins_user b/library/solita_jenkins_user
@@ -26,20 +26,21 @@ if (params.state == 'present') {
     if (!(params.name in currentUsers)) {
         user = securityRealm.createAccount(params.name, params.password)
         user.save()
+        result.changed = true
+    }
 
+    if (!(authorizationStrategy.hasPermission(params.name, Jenkins.ADMINISTER))) {
         authorizationStrategy.add(Jenkins.ADMINISTER, params.name)
-        jenkins.save()
-
         result.changed = true
     }
 } else {
     if (params.name in currentUsers) {
         user = securityRealm.getUser(params.name)
         user.delete()
-
         result.changed = true
     }
 }
+jenkins.save()
 
 println JsonOutput.toJson(result)
 """