From 5b0b30468431ac1a52bc4c8228d032393d2e335c Mon Sep 17 00:00:00 2001
From: Timo Mihaljov <timo.mihaljov@solita.fi>
Date: Wed, 29 Jun 2016 14:47:21 +0300
Subject: [PATCH] Give users admin permission if it's missing

---
 library/solita_jenkins_user | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/library/solita_jenkins_user b/library/solita_jenkins_user
index 182ad9d..a07a4db 100644
--- 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)
 """