Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 289458 - Validators added to a Ridget does not always works correctly
Summary: Validators added to a Ridget does not always works correctly
Status: RESOLVED FIXED
Alias: None
Product: Riena
Classification: RT
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows Vista
: P2 normal (vote)
Target Milestone: 1.2.0.M3   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 289849 (view as bug list)
Depends on:
Blocks:
 
Reported: 2009-09-15 08:32 EDT by Thorsten Schenkel CLA
Modified: 2009-11-10 13:29 EST (History)
3 users (show)

See Also:


Attachments
Patch (3.26 KB, patch)
2009-10-20 23:20 EDT, Elias Volanakis CLA
no flags Details | Diff
Patch with tests (7.18 KB, patch)
2009-10-21 02:30 EDT, Sabine Achilles CLA
elias: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thorsten Schenkel CLA 2009-09-15 08:32:15 EDT
This bug can be reprocuded in the sub-module "UI-Filters/Examples".
In the group "Ridgets (sample)" you can add a validator to the text field behind the label "Sample Label".
(The validator is added with the help of an UIFilter)
Press last the toggle button "activate" of the group.
Now enter some letters in the text field; something like "abc".
Leaf the text field (focus lost).
No error marker is shown (error). But the added validator only allows the two digits "0" and "1".
Press the last toggle button again. (The text in the field disapears!)
Enter again "abc". Press once again the las toogle button. Now an error marker is visible (correct).
Comment 1 Elias Volanakis CLA 2009-10-20 23:20:02 EDT
Created attachment 150046 [details]
Patch
Comment 2 Sabine Achilles CLA 2009-10-21 02:30:30 EDT
Created attachment 150067 [details]
Patch with tests
Comment 3 Sabine Achilles CLA 2009-10-21 02:33:47 EDT
Added tests to Elias' patch. This should fix the problem.
Comment 4 Elias Volanakis CLA 2009-10-21 15:59:55 EDT
Thanks, patch applied.

( something useful to know: TestUtils.getLocalizedNumber("20,5") produces Strings that work for DE and US locales. Example: bean.setProperty(TestUtils.getLocalizedNumber("20,5")); )
Comment 5 Elias Volanakis CLA 2009-10-21 16:03:12 EDT
Resolved.
Comment 6 Sabine Achilles CLA 2009-10-22 01:04:30 EDT
Thanks for the tip!
Comment 7 Elias Volanakis CLA 2009-11-10 13:29:43 EST
*** Bug 289849 has been marked as a duplicate of this bug. ***