Uploaded image for project: 'Struts 2'
  1. Struts 2
  2. WW-4808

Allow define more than one Action suffix

    Details

      Description

      Right now the struts.convention.action.suffix accepts only single string, it would be good if I could define more suffixes, e.g. Action,Endpoint,Controller

        Issue Links

          Activity

          Hide
          githubbot ASF GitHub Bot added a comment -

          GitHub user lukaszlenart opened a pull request:

          https://github.com/apache/struts/pull/148

          WW-4808: Multiple suffixes

          Implements multiple Action suffixes in the Convention Plugin, right now you can define suffixes as follow

          ```xml
          <constant name="struts.convention.action.suffix" value="Action,Controller,Endpoint"/>
          ```

          WW-4808(https://issues.apache.org/jira/browse/WW-4808)

          You can merge this pull request into a Git repository by running:

          $ git pull https://github.com/lukaszlenart/struts multiple-suffixes

          Alternatively you can review and apply these changes as the patch at:

          https://github.com/apache/struts/pull/148.patch

          To close this pull request, make a commit to your master/trunk branch
          with (at least) the following in the commit message:

          This closes #148


          commit cf10f68e766957fdd7f1bf70f25d4cc899a40b87
          Author: Lukasz Lenart <lukaszlenart@apache.org>
          Date: 2017-07-07T08:23:54Z

          Extends action suffix to support multiple suffixes

          commit b708d844c292e064f5ef90bf65bfcd53b32b6659
          Author: Lukasz Lenart <lukaszlenart@apache.org>
          Date: 2017-07-07T08:24:02Z

          Uses multiple suffixes

          commit f522fbc45417b836463553e3d9110ee58fcf2673
          Author: Lukasz Lenart <lukaszlenart@apache.org>
          Date: 2017-07-07T08:58:26Z

          Extract common code

          commit b5be431f540271685eed803c7e55bf5a4d3138e6
          Author: Lukasz Lenart <lukaszlenart@apache.org>
          Date: 2017-07-07T09:01:03Z

          Converts comments into logging statements


          Show
          githubbot ASF GitHub Bot added a comment - GitHub user lukaszlenart opened a pull request: https://github.com/apache/struts/pull/148 WW-4808 : Multiple suffixes Implements multiple Action suffixes in the Convention Plugin, right now you can define suffixes as follow ```xml <constant name="struts.convention.action.suffix" value="Action,Controller,Endpoint"/> ``` WW-4808 ( https://issues.apache.org/jira/browse/WW-4808 ) You can merge this pull request into a Git repository by running: $ git pull https://github.com/lukaszlenart/struts multiple-suffixes Alternatively you can review and apply these changes as the patch at: https://github.com/apache/struts/pull/148.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #148 commit cf10f68e766957fdd7f1bf70f25d4cc899a40b87 Author: Lukasz Lenart <lukaszlenart@apache.org> Date: 2017-07-07T08:23:54Z Extends action suffix to support multiple suffixes commit b708d844c292e064f5ef90bf65bfcd53b32b6659 Author: Lukasz Lenart <lukaszlenart@apache.org> Date: 2017-07-07T08:24:02Z Uses multiple suffixes commit f522fbc45417b836463553e3d9110ee58fcf2673 Author: Lukasz Lenart <lukaszlenart@apache.org> Date: 2017-07-07T08:58:26Z Extract common code commit b5be431f540271685eed803c7e55bf5a4d3138e6 Author: Lukasz Lenart <lukaszlenart@apache.org> Date: 2017-07-07T09:01:03Z Converts comments into logging statements
          Hide
          jira-bot ASF subversion and git services added a comment -

          Commit 62900ee9868c4d12ef76b8a7a8bbc60b694cbcaa in struts's branch refs/heads/master from Lukasz Lenart
          [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=62900ee ]

          WW-4808 Allows define multiple suffixes when using
          Convention plugin

          Show
          jira-bot ASF subversion and git services added a comment - Commit 62900ee9868c4d12ef76b8a7a8bbc60b694cbcaa in struts's branch refs/heads/master from Lukasz Lenart [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=62900ee ] WW-4808 Allows define multiple suffixes when using Convention plugin
          Hide
          lukaszlenart Lukasz Lenart added a comment -

          PR got merged

          Show
          lukaszlenart Lukasz Lenart added a comment - PR got merged
          Hide
          githubbot ASF GitHub Bot added a comment -

          Github user asfgit closed the pull request at:

          https://github.com/apache/struts/pull/148

          Show
          githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/148

            People

            • Assignee:
              lukaszlenart Lukasz Lenart
              Reporter:
              lukaszlenart Lukasz Lenart
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development