Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364098 - Slush bucket property editor issue with case-insensitive possible values
Summary: Slush bucket property editor issue with case-insensitive possible values
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sapphire (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Konstantin Komissarchik CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-17 20:56 EST by Greg Amerson CLA
Modified: 2021-11-19 09:22 EST (History)
1 user (show)

See Also:

Attachments
Screenshot of the problem (60.73 KB, image/png)
2011-11-17 20:56 EST, Greg Amerson CLA 		no flags Details
Patch v1 (1.77 KB, patch)
2011-11-17 21:57 EST, Greg Amerson CLA 		no flags Details | Diff
Patch v2 (2.12 KB, patch)
2011-11-17 21:58 EST, Greg Amerson CLA 		konstantin: iplog+
Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Greg Amerson CLA 2011-11-17 20:56:17 EST 
If you have a custom possible values service that returns false for isCaseSensitive(), property editors for this property will not recognize a value as having already been "entered" or "taken" if the case doesn't match.  Will attach screenshot.
Comment 1 Greg Amerson CLA 2011-11-17 20:56:51 EST 
Created attachment 207192 [details]
Screenshot of the problem
Comment 2 Greg Amerson CLA 2011-11-17 21:17:55 EST 
Hey Konstantin, do you want me to submit a patch?  Or do you have this on your todo list for today?
Comment 3 Konstantin Komissarchik CLA 2011-11-17 21:19:15 EST 
If you can get this in by EOD today, then go for it. If not, we will be past the IP Log deadline for 0.4, so I would have to do it.
Comment 4 Greg Amerson CLA 2011-11-17 21:57:23 EST 
Created attachment 207194 [details]
Patch v1
Comment 5 Greg Amerson CLA 2011-11-17 21:58:36 EST 
Created attachment 207195 [details]
Patch v2

Forget to update header
Comment 6 Konstantin Komissarchik CLA 2011-11-17 22:41:45 EST 
Patch v2 accepted with a slight optimization.
Comment 7 Konstantin Komissarchik CLA 2011-11-17 22:42:25 EST 
Please verify.
Comment 8 Greg Amerson CLA 2011-11-18 00:05:18 EST 
Updated from head. Verified.
Comment 9 Konstantin Komissarchik CLA 2011-11-18 00:36:03 EST 
Thanks.