[jbossseam-issues] [JBoss JIRA] Created: (JBSEAM-689) Seam framework Query too sensitive about keyword case
1:43 a.m.
Seam framework Query too sensitive about keyword case
-----------------------------------------------------
Key: JBSEAM-689
URL: http://jira.jboss.com/jira/browse/JBSEAM-689
Project: JBoss Seam
Issue Type: Bug
Components: Framework
Affects Versions: 1.1.1.GA
Reporter: Leo Baschy
When the org.jboss.seam.framework.Query uses anything but lower case "from" and
"order" then in Query method getCountEjbql calls to indexOf return -1, which
then causes an exception when calling substring.
Further, indexOf("from") and indexOf("order") also would match e.g.
fromAddress or orderNumber, which would be wrong.
Upper case or any case "FROM" and "ORDER" should be tolerated. Names
that happen to contain letter sequences "from" or "order" should be
ignored.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
2 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Updated: (JBSEAM-689) Seam framework Query too sensitive about keyword case
[ http://jira.jboss.com/jira/browse/JBSEAM-689?page=all ]
Leo Baschy updated JBSEAM-689:
------------------------------
Attachment: Query.java.patch
This patch fixes the problem.
The regular expression could use some checking on their exact logic matching EJBQL.
The actual patch may have wacky line breaks due to CVS issues.
Seam framework Query too sensitive about keyword case
-----------------------------------------------------
Key: JBSEAM-689
URL: http://jira.jboss.com/jira/browse/JBSEAM-689
Project: JBoss Seam
Issue Type: Bug
Components: Framework
Affects Versions: 1.1.1.GA
Reporter: Leo Baschy
Attachments: Query.java.patch
When the org.jboss.seam.framework.Query uses anything but lower case "from" and
"order" then in Query method getCountEjbql calls to indexOf return -1, which
then causes an exception when calling substring.
Further, indexOf("from") and indexOf("order") also would match e.g.
fromAddress or orderNumber, which would be wrong.
Upper case or any case "FROM" and "ORDER" should be tolerated. Names
that happen to contain letter sequences "from" or "order" should be
ignored.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
3:37 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Updated: (JBSEAM-689) Seam framework Query too sensitive about keyword case
[ http://jira.jboss.com/jira/browse/JBSEAM-689?page=all ]
Gavin King updated JBSEAM-689:
------------------------------
Fix Version/s: 1.1.5.GA
Assignee: Gavin King
Seam framework Query too sensitive about keyword case
-----------------------------------------------------
Key: JBSEAM-689
URL: http://jira.jboss.com/jira/browse/JBSEAM-689
Project: JBoss Seam
Issue Type: Bug
Components: Framework
Affects Versions: 1.1.1.GA
Reporter: Leo Baschy
Assigned To: Gavin King
Fix For: 1.1.5.GA
Attachments: Query.java.patch
When the org.jboss.seam.framework.Query uses anything but lower case "from" and
"order" then in Query method getCountEjbql calls to indexOf return -1, which
then causes an exception when calling substring.
Further, indexOf("from") and indexOf("order") also would match e.g.
fromAddress or orderNumber, which would be wrong.
Upper case or any case "FROM" and "ORDER" should be tolerated. Names
that happen to contain letter sequences "from" or "order" should be
ignored.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
8:04 p.m.
New subject: [jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-689) Seam framework Query too sensitive about keyword case
[ http://jira.jboss.com/jira/browse/JBSEAM-689?page=comments#action_12351589 ]
Leo Baschy commented on JBSEAM-689:
-----------------------------------
The regular expressions in the attached patch covers way more cases than the original
indexOf, possibly all cases that real programmers will write in the next five years.
For complete correctness though, EJBQL identifiers are defined in section 4.4.1
Identifiers of JSR 220: Enterprise JavaBeans, Version 3.0 Java Persistence API.
As one can recognize, for perfection one probably should use
Character.isJavaIdentifierPart() and related methods to check the characters before and
after the keyword (i.e. "from" or "order"). Still should use case
insensitive regular expression for the keyword itself. Complete correctness may be
complex to figure out and complex to test.
Seam framework Query too sensitive about keyword case
-----------------------------------------------------
Key: JBSEAM-689
URL: http://jira.jboss.com/jira/browse/JBSEAM-689
Project: JBoss Seam
Issue Type: Bug
Components: Framework
Affects Versions: 1.1.1.GA
Reporter: Leo Baschy
Assigned To: Gavin King
Fix For: 1.1.5.GA
Attachments: Query.java.patch
When the org.jboss.seam.framework.Query uses anything but lower case "from" and
"order" then in Query method getCountEjbql calls to indexOf return -1, which
then causes an exception when calling substring.
Further, indexOf("from") and indexOf("order") also would match e.g.
fromAddress or orderNumber, which would be wrong.
Upper case or any case "FROM" and "ORDER" should be tolerated. Names
that happen to contain letter sequences "from" or "order" should be
ignored.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
11:15 a.m.
New subject: [jbossseam-issues] [JBoss JIRA] Closed: (JBSEAM-689) Seam framework Query too sensitive about keyword case
[ http://jira.jboss.com/jira/browse/JBSEAM-689?page=all ]
Gavin King closed JBSEAM-689.
-----------------------------
Resolution: Done
Thanks, applied this with minor changes.
Seam framework Query too sensitive about keyword case
-----------------------------------------------------
Key: JBSEAM-689
URL: http://jira.jboss.com/jira/browse/JBSEAM-689
Project: JBoss Seam
Issue Type: Bug
Components: Framework
Affects Versions: 1.1.1.GA
Reporter: Leo Baschy
Assigned To: Gavin King
Fix For: 1.1.5.GA
Attachments: Query.java.patch
When the org.jboss.seam.framework.Query uses anything but lower case "from" and
"order" then in Query method getCountEjbql calls to indexOf return -1, which
then causes an exception when calling substring.
Further, indexOf("from") and indexOf("order") also would match e.g.
fromAddress or orderNumber, which would be wrong.
Upper case or any case "FROM" and "ORDER" should be tolerated. Names
that happen to contain letter sequences "from" or "order" should be
ignored.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
7032
days inactive
7033
days old
4 comments
2 participants
participants (2)
-
Gavin King (JIRA) -
Leo Baschy (JIRA)