Modify code to use global reference to deprecatedHeaderMap property instead of passing as parameter

bbellsct · bbellsct · commit 2a6fa7efdc95 · 2016-10-27T11:14:37.000-04:00
diff --git a/grails-app/controllers/net/hedtech/restfulapi/RestfulApiController.groovy b/grails-app/controllers/net/hedtech/restfulapi/RestfulApiController.groovy
@@ -459,7 +459,7 @@ class RestfulApiController {
         }
 
         // Add any deprecated response headers to the response holder
-        applyDeprecatedHeaderMap(responseHolder.headers, deprecatedHeaderMap)
+        applyDeprecatedHeaderMap(responseHolder.headers)
 
         responseHolder.headers.each { header ->
             header.value.each() { val ->
@@ -599,7 +599,7 @@ class RestfulApiController {
         }
 
         // Add any deprecated response headers to the response holder
-        applyDeprecatedHeaderMap(responseHolder.headers, deprecatedHeaderMap)
+        applyDeprecatedHeaderMap(responseHolder.headers)
 
         responseHolder.headers.each { header ->
             header.value.each() { val ->
@@ -848,7 +848,7 @@ class RestfulApiController {
     }
 
 
-    private void applyDeprecatedHeaderMap(headers, deprecatedHeaderMap) {
+    private void applyDeprecatedHeaderMap(headers) {
         deprecatedHeaderMap?.each { entry ->
             def value = headers[entry.key]
             if (value) {

Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,7 @@ class RestfulApiController {`
`459`	`459`	`}`
`460`	`460`
`461`	`461`	`// Add any deprecated response headers to the response holder`
`462`		`- applyDeprecatedHeaderMap(responseHolder.headers, deprecatedHeaderMap)`
	`462`	`+ applyDeprecatedHeaderMap(responseHolder.headers)`
`463`	`463`
`464`	`464`	`responseHolder.headers.each { header ->`
`465`	`465`	`header.value.each() { val ->`
`@@ -599,7 +599,7 @@ class RestfulApiController {`
`599`	`599`	`}`
`600`	`600`
`601`	`601`	`// Add any deprecated response headers to the response holder`
`602`		`- applyDeprecatedHeaderMap(responseHolder.headers, deprecatedHeaderMap)`
	`602`	`+ applyDeprecatedHeaderMap(responseHolder.headers)`
`603`	`603`
`604`	`604`	`responseHolder.headers.each { header ->`
`605`	`605`	`header.value.each() { val ->`
`@@ -848,7 +848,7 @@ class RestfulApiController {`
`848`	`848`	`}`
`849`	`849`
`850`	`850`
`851`		`- private void applyDeprecatedHeaderMap(headers, deprecatedHeaderMap) {`
	`851`	`+ private void applyDeprecatedHeaderMap(headers) {`
`852`	`852`	`deprecatedHeaderMap?.each { entry ->`
`853`	`853`	`def value = headers[entry.key]`
`854`	`854`	`if (value) {`