GRAILS-11748 - fix params.action for resource mappings

All of the mappings for a particular resource were sharing a Map so the action of the last one created was being shared.  Since delete was the last one created at https://github.com/grails/grails-core/blob/8c5d540797cd374c176e82aee7ca78e6dffaaf20/grails-web-url-mappings/src/main/groovy/org/codehaus/groovy/grails/web/mapping/DefaultUrlMappingEvaluator.java#L638, all of the mappings for that resource shared a params.action of "delete".

Author: jeffscottbrown

grails-web-url-mappings/src/main/groovy/org/codehaus/groovy/grails/web/mapping/DefaultUrlMappingEvaluator.java

@@ -836,7 +836,7 @@ private void configureUrlMapping(UrlMapping urlMapping) {
                 parameterValues.put("action", urlMapping.getActionName());
             }
 
-            urlMapping.setParameterValues(parameterValues);
+            urlMapping.setParameterValues(new LinkedHashMap(parameterValues));
             urlMappings.add(urlMapping);
         }
 

grails-web-url-mappings/src/test/groovy/org/codehaus/groovy/grails/web/mapping/RestfulResourceMappingSpec.groovy

@@ -14,6 +14,34 @@ import spock.lang.Specification
  */
 class RestfulResourceMappingSpec extends Specification{
 
+    @Issue('GRAILS-11748')
+    void 'Test params.action'() {
+        given:
+        def urlMappingsHolder = getUrlMappingsHolder {
+            "/books"(resources: "book")
+            "/owner"(resource: "person")
+        }
+        
+        when:
+        def urlMappings = urlMappingsHolder.urlMappings
+        
+        then:
+        urlMappingsHolder.matchAll('/books', 'GET')[0].parameters.action == 'index'
+        urlMappingsHolder.matchAll('/books/create', 'GET')[0].parameters.action == 'create'
+        urlMappingsHolder.matchAll('/books', 'POST')[0].parameters.action == 'save'
+        urlMappingsHolder.matchAll('/books/42', 'GET')[0].parameters.action == 'show'
+        urlMappingsHolder.matchAll('/books/42/edit', 'GET')[0].parameters.action == 'edit'
+        urlMappingsHolder.matchAll('/books/42', 'PUT')[0].parameters.action == 'update'
+        urlMappingsHolder.matchAll('/books/42', 'DELETE')[0].parameters.action == 'delete'
+        
+        urlMappingsHolder.matchAll('/owner/create', 'GET')[0].parameters.action == 'create'
+        urlMappingsHolder.matchAll('/owner', 'POST')[0].parameters.action == 'save'
+        urlMappingsHolder.matchAll('/owner', 'GET')[0].parameters.action == 'show'
+        urlMappingsHolder.matchAll('/owner/edit', 'GET')[0].parameters.action == 'edit'
+        urlMappingsHolder.matchAll('/owner', 'PUT')[0].parameters.action == 'update'
+        urlMappingsHolder.matchAll('/owner', 'DELETE')[0].parameters.action == 'delete'
+    }
+    
     @Issue('GRAILS-11680')
     @Ignore
     void 'Test mapping ordering problem'() {