Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 331857

Summary: fencepost error in getFunctionWithArguments
Product: z_Archived Reporter: Nathan Bryant <nbryant>
Component: EclipselinkAssignee: Nobody - feel free to take it <nobody>
Status: CLOSED INVALID QA Contact:
Severity: minor    
Priority: P3    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Nathan Bryant CLA 2010-12-04 23:30:52 EST
Build Identifier: 2.1.1

Expression.getFunctionWithArguments has a fence post error. For a function with two arguments it will insert two commas. I'm seeing this result in generation of syntactically incorrect SQL every time. The problem is here:

        for (int index = 0; index < arguments.size(); index++) {
            v.addElement(", ");
        }


Reproducible: Always

Steps to Reproduce:
1. Try to use getFunctionWithArguments
2. It can't possibly have ever worked.
3.
Comment 1 Nathan Bryant CLA 2010-12-05 00:01:25 EST
My bad... it's just not intended to be used from an ExpressionBuilder.
Comment 2 Eclipse Webmaster CLA 2022-06-09 10:30:28 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink