[GitHub] struts pull request: [PATCH] First attempt at enabling value-label...

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

[GitHub] struts pull request: Feature: list value localization

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

[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-module"

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread lukaszlenart
Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/19#issuecomment-50593699 Two issues: - please open this PR against `develop` branch, not `master` - please register an issue in JIRA and reference this branch from it https://issues

[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 enab

[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 "struts-mo

[GitHub] struts pull request: Localized by classloader

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

[GitHub] struts pull request: Localized by classloader

2014-07-30 Thread lukaszlenart
Github user lukaszlenart commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575148 --- 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 lukaszlenart
Github user lukaszlenart commented on a diff in the pull request: https://github.com/apache/struts/pull/20#discussion_r15575196 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boole

[GitHub] struts pull request: Localized by classloader

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

[GitHub] struts pull request: Localized by classloader

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

[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 r

[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_r15575658 --- Diff: xwork-core/src/main/java/com/opensymphony/xwork2/util/LocalizedTextUtil.java --- @@ -136,13 +129,21 @@ public static void setReloadBundles(boolean r

[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 r

[GitHub] struts pull request: Localized by classloader

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

[GitHub] struts pull request: Localized by classloader

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

[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 a

[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 r

[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 r

[GitHub] struts pull request: Localized by classloader

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