Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 331857 - fencepost error in getFunctionWithArguments
Summary: fencepost error in getFunctionWithArguments
Status: CLOSED INVALID
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-04 23:30 EST by Nathan Bryant CLA
Modified: 2022-06-09 10:30 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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