Merge pull request #579 from vinod2800/2.4.x

Resolution of GRAILS-11583 - removing of web flow param settings

Author: graemerocher

grails-plugin-gsp/src/main/groovy/org/codehaus/groovy/grails/plugins/web/taglib/ApplicationTagLib.groovy

@@ -359,8 +359,6 @@ class ApplicationTagLib implements ApplicationContextAware, InitializingBean, Gr
         }
         def params = urlAttrs.params && urlAttrs.params instanceof Map ? urlAttrs.params : [:]
         if (request.flowExecutionKey) {
-            params.execution = request.flowExecutionKey
-            urlAttrs.params = params
             if (attrs.controller == null && attrs.action == null && attrs.url == null && attrs.uri == null) {
                 urlAttrs[LinkGenerator.ATTRIBUTE_ACTION] = GrailsWebRequest.lookup().actionName
             }

grails-test-suite-web/src/test/groovy/org/codehaus/groovy/grails/web/taglib/ApplicationTagLibTests.groovy

@@ -291,32 +291,27 @@ class ApplicationTagLibTests extends AbstractGrailsTagTests {
 
     void testCreateLinkWithFlowExecutionKeyAndEvent() {
         unRegisterRequestDataValueProcessor()
-        request.flowExecutionKey = '12345'
-
+      
         def template = '<g:createLink controller="foo" action="bar" event="boo" />'
-        assertOutputEquals('/foo/bar?execution=12345&_eventId=boo', template)
+        assertOutputEquals('/foo/bar?_eventId=boo', template)
     }
 
     void testCreateLinkWithFlowExecutionKeyAndEventAndRequestDataValueProcessor() {
-        request.flowExecutionKey = '12345'
 
         def template = '<g:createLink controller="foo" action="bar" event="boo" />'
-        assertOutputEquals('/foo/bar?execution=12345&_eventId=boo&requestDataValueProcessorParamName=paramValue', template)
+        assertOutputEquals('/foo/bar?_eventId=boo&requestDataValueProcessorParamName=paramValue', template)
     }
 
     void testLinkWithFlowExecutionKeyAndEvent() {
         unRegisterRequestDataValueProcessor()
-        request.flowExecutionKey = '12345'
 
         def template = '<g:link controller="foo" action="bar" event="boo" >link</g:link>'
-        assertOutputEquals('<a href="/foo/bar?execution=12345&amp;_eventId=boo">link</a>', template)
+        assertOutputEquals('<a href="/foo/bar?_eventId=boo">link</a>', template)
     }
 
     void testLinkWithFlowExecutionKeyAndEventAndRequestDataValueProcessor() {
-        request.flowExecutionKey = '12345'
-
         def template = '<g:link controller="foo" action="bar" event="boo" >link</g:link>'
-        assertOutputEquals('<a href="/foo/bar?execution=12345&amp;_eventId=boo&amp;requestDataValueProcessorParamName=paramValue">link</a>', template)
+        assertOutputEquals('<a href="/foo/bar?_eventId=boo&amp;requestDataValueProcessorParamName=paramValue">link</a>', template)
     }
 
     void testSetTag() {