[seam-commits] Seam SVN: r8081 - trunk/src/main/org/jboss/seam/framework and 1 other directory.

[seam-commits at lists.jboss.org]

Author: pete.muir at jboss.org
Date: 2008-04-30 11:26:46 -0400 (Wed, 30 Apr 2008)
New Revision: 8081

Modified:
   branches/Seam_2_0/src/main/org/jboss/seam/framework/Query.java
   trunk/src/main/org/jboss/seam/framework/Query.java
Log:
JBSEAM-2931

Modified: branches/Seam_2_0/src/main/org/jboss/seam/framework/Query.java
===================================================================
--- branches/Seam_2_0/src/main/org/jboss/seam/framework/Query.java	2008-04-30 15:12:49 UTC (rev 8080)
+++ branches/Seam_2_0/src/main/org/jboss/seam/framework/Query.java	2008-04-30 15:26:46 UTC (rev 8081)
@@ -369,7 +369,7 @@
    public void setOrder(String order)
    {      
        if (order!= null && !ORDER_CLAUSE_PATTERN.matcher(order).find()) {
-           throw new IllegalArgumentException("invalid order clause");
+          throw new IllegalArgumentException("invalid order column (\"" + order + "\" must match the regular expression \"" + ORDER_CLAUSE_PATTERN + "\")");
        }
        this.order = order;
        refresh();

Modified: trunk/src/main/org/jboss/seam/framework/Query.java
===================================================================
--- trunk/src/main/org/jboss/seam/framework/Query.java	2008-04-30 15:12:49 UTC (rev 8080)
+++ trunk/src/main/org/jboss/seam/framework/Query.java	2008-04-30 15:26:46 UTC (rev 8081)
@@ -442,7 +442,7 @@
        } else if (ORDER_COLUMN_PATTERN.matcher(columnName).find()) {
            return columnName;
        } else {
-           throw new IllegalArgumentException("invalid order column");
+           throw new IllegalArgumentException("invalid order column (\"" + columnName + "\" must match the regular expression \"" + ORDER_COLUMN_PATTERN + "\")");
        }
    }