| Summary: | TRIM() no longer works on SQL Server | ||||||
|---|---|---|---|---|---|---|---|
| Product: | z_Archived | Reporter: | Mitesh Meswani <mitesh.meswani> | ||||
| Component: | Eclipselink | Assignee: | Project Inbox <eclipselink.foundation-inbox> | ||||
| Status: | CLOSED FIXED | QA Contact: | |||||
| Severity: | blocker | ||||||
| Priority: | P3 | CC: | andrei.ilitchev, david.minsky, tom.ware | ||||
| Version: | unspecified | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Mitesh Meswani
Created attachment 187877 [details]
Suggested patch.
Rolled back the code to use leftTrim(rightTrim, again.
Also fixes in JUnitJPQLComplexTestSuite:
introduced supportsStoredFunction method so that setup passes on SQLServer (which doesn't support stored functions);
updated variableReferencedOnlyInParameterTest so that it now passes on SQLServer. That's done by excluding the qpql strings known not been supported by the db platform. For SQLSerfver those are:
"SELECT TRIM(LEADING 'A' FROM e.firstName) FROM Employee e"
and
"SELECT TRIM(TRAILING 'A' FROM e.firstName) FROM Employee e"
Fixed in trunk rev.8902 and 2.2 revision: 8903 Checked into 2.1 trunk at revision: 9722 The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink |