[GitHub] struts pull request: Localized by classloader

2014-08-18 Thread foogaro
Github user foogaro commented on the pull request: https://github.com/apache/struts/pull/20#issuecomment-52499604 Hi Luckasz, I didn't notice this test, cause I just ran the __*xwork-core*__ test suite, not the all thing. Sorry about that. Backing to the issue, the test

[GitHub] struts pull request: Localized by classloader

2014-07-31 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15630020 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15576294 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15576177 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575975 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -246,35 +245,32 @@ public static String findDefaultText(String

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575705 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575658 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575513 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -893,6 +889,10 @@ public int hashCode

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575491 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
GitHub user foogaro opened a pull request: https://github.com/apache/struts/pull/20 Localized by classloader **This improvement enables struts to be used as JBoss AS 7/WildFly 8/JBoss EAP 6 module.** _**Issue:**_ having two or more applications using the same "s

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro commented on the pull request: https://github.com/apache/struts/pull/19#issuecomment-50594012 Redoing it, targeting develop branch. --- 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

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
Github user foogaro closed the pull request at: https://github.com/apache/struts/pull/19 --- 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

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread foogaro
GitHub user foogaro opened a pull request: https://github.com/apache/struts/pull/19 Localized by classloader **This update enables struts to be used as JBoss AS 7/WildFly 8/JBoss EAP 6 module.** _**Issue:**_ having two or more applications using the same "struts-m