Author: pete.muir(a)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 + "\")"); } }