You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by me...@apache.org on 2009/12/30 23:06:33 UTC

svn commit: r894719 - in /incubator/click/trunk/click: documentation/docs/roadmap-changes.html framework/src/org/apache/click/Page.java framework/src/org/apache/click/control/AbstractControl.java

Author: medgar
Date: Wed Dec 30 22:06:32 2009
New Revision: 894719

URL: http://svn.apache.org/viewvc?rev=894719&view=rev
Log:
CLK-600

Modified:
    incubator/click/trunk/click/documentation/docs/roadmap-changes.html
    incubator/click/trunk/click/framework/src/org/apache/click/Page.java
    incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractControl.java

Modified: incubator/click/trunk/click/documentation/docs/roadmap-changes.html
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/documentation/docs/roadmap-changes.html?rev=894719&r1=894718&r2=894719&view=diff
==============================================================================
--- incubator/click/trunk/click/documentation/docs/roadmap-changes.html (original)
+++ incubator/click/trunk/click/documentation/docs/roadmap-changes.html Wed Dec 30 22:06:32 2009
@@ -394,6 +394,11 @@
           [<a target='_blank' href="https://issues.apache.org/jira/browse/CLK-535">535</a>].
       </li>
       <li class="change">
+          Improved Page and Control message handling with null args.
+          This issue was raised by WarnerJan Veldhuis
+          [<a target='_blank' href="https://issues.apache.org/jira/browse/CLK-600">600</a>].
+      </li>
+      <li class="change">
           Fixed resource deployment on JBoss 5 and up. The solution is based on
           the work done by the <a class="external" target="_blank" href="http://www.stripesframework.org">Stripes Framework</a>
           developers

Modified: incubator/click/trunk/click/framework/src/org/apache/click/Page.java
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/framework/src/org/apache/click/Page.java?rev=894719&r1=894718&r2=894719&view=diff
==============================================================================
--- incubator/click/trunk/click/framework/src/org/apache/click/Page.java (original)
+++ incubator/click/trunk/click/framework/src/org/apache/click/Page.java Wed Dec 30 22:06:32 2009
@@ -844,9 +844,6 @@
      * @return the named localized message for the page
      */
     public String getMessage(String name, Object[] args) {
-        if (args == null) {
-            throw new IllegalArgumentException("Null args parameter");
-        }
         String value = getMessage(name);
 
         return MessageFormat.format(value, args);

Modified: incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractControl.java
URL: http://svn.apache.org/viewvc/incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractControl.java?rev=894719&r1=894718&r2=894719&view=diff
==============================================================================
--- incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractControl.java (original)
+++ incubator/click/trunk/click/framework/src/org/apache/click/control/AbstractControl.java Wed Dec 30 22:06:32 2009
@@ -429,9 +429,6 @@
      * was found
      */
     public String getMessage(String name, Object[] args) {
-        if (args == null) {
-            throw new IllegalArgumentException("Null args parameter");
-        }
         String value = getMessage(name);
         if (value == null) {
             return null;