[GitHub] struts pull request #122: WW-4757: LocaleProvider factory

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 #122: WW-4757: LocaleProvider factory

sdutry
GitHub user lukaszlenart opened a pull request:

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

    WW-4757: LocaleProvider factory

    Replaces injectable `LocaleProvider` with `LocaleProviderFactory` to avoid problems with multiple beans implementing the `LocaleProvider` interface.
   
    Implements [WW-4757](https://issues.apache.org/jira/browse/WW-4757)

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

    $ git pull https://github.com/lukaszlenart/struts locale-factory

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

    https://github.com/apache/struts/pull/122.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 #122
   
----
commit b42b9f16c4598505e0b1cfef78e6639237b2d712
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T07:46:09Z

    Defines new interface for factory

commit 636821aa2dfc32a4531b6e65717c38625bb5b4cc
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T07:46:37Z

    Adds default implementation of the factory

commit 09be1b9b773f164d59114b0a73d3a4fde366b673
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T07:50:05Z

    Replaces LocaleProvider with LocaleProviderFactory

commit 3b738ad3ef01c434dbd673d5dca4659aa5779e7d
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T07:53:35Z

    Injects factory instead of the provider

commit 4b022bfbe5f575908e61d84de885ca171fcca7f3
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T08:11:31Z

    Replaces LocaleProvider with LocaleProviderFactory

commit f912dab8f3c77ef9667370c57cf8d35cf0cf4a32
Author: Lukasz Lenart <[hidden email]>
Date:   2017-03-15T08:18:24Z

    Fixes test

----


---
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 #122: WW-4757: LocaleProvider factory

sdutry
Github user asfgit closed the pull request at:

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


---
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...