Merge pull request #76 from danieljamesrees/use_template_id_not_name

danieljamesrees · web-flow · commit b15b7ee12c4d · 2018-10-17T12:48:42.000+01:00
Use templateId not templateName
diff --git a/jobs/sonarqube/templates/config/ConfigureUsers.groovy b/jobs/sonarqube/templates/config/ConfigureUsers.groovy
@@ -33,7 +33,7 @@ static void addPermission(login, permission, permissionsUrl, template) {
     // Currently this seems only to be needed for project permissions.
     // The API claims both are optional, but when not included, an error is specified indicating one of the other, not both, is required.
     if (template && !template.allWhitespace) {
-        def templatePair = SonarApiClient.buildSingleValuedKeyPair('templateName', template)
+        def templatePair = SonarApiClient.buildSingleValuedKeyPair('templateId', template)
         queryValues = [loginPair, permissionsPair, templatePair]
     }
 

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ static void addPermission(login, permission, permissionsUrl, template) {`
`33`	`33`	`// Currently this seems only to be needed for project permissions.`
`34`	`34`	`// The API claims both are optional, but when not included, an error is specified indicating one of the other, not both, is required.`
`35`	`35`	`if (template && !template.allWhitespace) {`
`36`		`- def templatePair = SonarApiClient.buildSingleValuedKeyPair('templateName', template)`
	`36`	`+ def templatePair = SonarApiClient.buildSingleValuedKeyPair('templateId', template)`
`37`	`37`	`queryValues = [loginPair, permissionsPair, templatePair]`
`38`	`38`	`}`
`39`	`39`