[GitHub] struts pull request #148: WW-4808: Multiple suffixes

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] struts pull request #148: WW-4808: Multiple suffixes

cnenning
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 <[hidden email]>
Date:   2017-07-07T08:23:54Z

    Extends action suffix to support multiple suffixes

commit b708d844c292e064f5ef90bf65bfcd53b32b6659
Author: Lukasz Lenart <[hidden email]>
Date:   2017-07-07T08:24:02Z

    Uses multiple suffixes

commit f522fbc45417b836463553e3d9110ee58fcf2673
Author: Lukasz Lenart <[hidden email]>
Date:   2017-07-07T08:58:26Z

    Extract common code

commit b5be431f540271685eed803c7e55bf5a4d3138e6
Author: Lukasz Lenart <[hidden email]>
Date:   2017-07-07T09:01:03Z

    Converts comments into logging statements

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[GitHub] struts pull request #148: WW-4808: Multiple suffixes

cnenning
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Loading...