Regarding the bug, there is something I don't clearly understand in PropertyEditorSupport subclass , why does FieldStringEditor implement FocusListener and not ComboStringEditor ?
Do you know why ? On Tue, Aug 20, 2013 at 5:12 PM, sebb <[email protected]> wrote: > On 20 August 2013 13:11, Philippe Mouawad <[email protected]> > wrote: > > Hello Sebb, > > I don't think you broke it, I made some checks and it seems it is broken > in > > versions before your change. > > OK, however I think I did do some damage to ComboStringEditor when I > changed the static EDIT and UNDEFINED fields to instance variables - > the code relied on '==' comparisons which won't work with instance > variables. I'll fix that shortly. > > > Could be related to: > > https://issues.apache.org/bugzilla/show_bug.cgi?id=55103 > > Yes, looks quite likely. > > > Regards > > > > > > On Tue, Aug 20, 2013 at 12:10 AM, <[email protected]> wrote: > > > >> https://issues.apache.org/bugzilla/show_bug.cgi?id=55432 > >> > >> Sebb <[email protected]> changed: > >> > >> What |Removed |Added > >> > >> > ---------------------------------------------------------------------------- > >> Status|REOPENED |RESOLVED > >> Resolution|--- |FIXED > >> > >> --- Comment #5 from Sebb <[email protected]> --- > >> It's not just CSV Dataset - JSR223 Preproces has the same issue. > >> > >> Looks like I broke ComboStringEditor while implementing Bug 55440 > >> > >> I'll close this and re-open that. > >> > >> -- > >> You are receiving this mail because: > >> You are on the CC list for the bug. > >> You are watching the reporter of the bug. > >> > > > > > > > > -- > > Cordialement. > > Philippe Mouawad. > > Ubik-Ingénierie > > > > UBIK LOAD PACK Web Site <http://www.ubikloadpack.com/> > > > > UBIK LOAD PACK on TWITTER <https://twitter.com/ubikloadpack> > -- Cordialement. Philippe Mouawad.
