Github user wolpi commented on the pull request:
https://github.com/apache/struts/pull/35#issuecomment-74502592
could you add a unit test, please?
---
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
Github user wolpi commented on the pull request:
https://github.com/apache/struts/pull/50#issuecomment-142278975
Usually it is not supported to use struts2 utility classes outside of
struts context. I would rather not merge this as it increases maintanance
burden.
That it
Github user wolpi commented on the pull request:
https://github.com/apache/struts/pull/50#issuecomment-142507218
> It make struts more robust.
When LocalizedTextUtil is used as intended and supported it is already
robust :wink:
> It doesn'
Github user wolpi commented on the pull request:
https://github.com/apache/struts/pull/54#issuecomment-146840650
Please create a JIRA issue. That makes it easier to track changes and
contributions.
https://issues.apache.org/jira/browse/WW
---
If your project is set up for
Github user wolpi commented on the issue:
https://github.com/apache/struts/pull/128
+1 for merging ð
---
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
