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

Bug 361168

Summary: SQL - Validation fails for get a,b,c from ds with #sql{}
Product: z_Archived Reporter: Paul Hoffman <hoffmanp>
Component: EDTAssignee: Joseph Vincens <jvincens>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: jspadea, jvincens, svihovec
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
test program for defect
none
Library to load database for test none

Description Paul Hoffman CLA 2011-10-17 13:21:04 EDT
Created attachment 205356 [details]
test program for defect

This variation came from Tim.    The "get scalar list from ds" should not require the variables in the list to be in an annotated record or entity.

See attached test program.
Comment 1 Paul Hoffman CLA 2011-10-17 13:22:20 EDT
Created attachment 205357 [details]
Library to load database for test
Comment 2 Joseph Vincens CLA 2011-10-18 14:21:22 EDT
I talked to Tim if a non entity variables are used a WITH clause must be used. 
We need to decide if some variables are non entity or if the variables point to multiple entities which map to different tables 
What does validation and generation do for the following
 -some variables are non entity 
 -if the variables point to multiple entities which map to different tables

Variables need to be validated that they are primitive types.

If there are multiple or no entities represented in the variables list then a WITH clause must be specified.

Are there other ambiguous situations? 

After validation changes are made reassign the defect back to me so I can test the gen/run.
Comment 3 Justin Spadea CLA 2011-10-19 14:25:49 EDT
Validation now allows primitives that are not within an entity type. If the targets do not map to a single table, then the FOR clause must be used for ADD, or the WITH clause must be used for GET.
Comment 4 Joseph Vincens CLA 2011-10-26 15:56:18 EDT
fixed
Comment 5 Paul Hoffman CLA 2011-11-01 16:44:42 EDT
Closed and verified