]> source.dussan.org Git - vaadin-framework.git/commit
Large server side refactoring for DateField
authorMatti Tahvonen <matti.tahvonen@itmill.com>
Tue, 4 Jan 2011 21:55:57 +0000 (21:55 +0000)
committerMatti Tahvonen <matti.tahvonen@itmill.com>
Tue, 4 Jan 2011 21:55:57 +0000 (21:55 +0000)
commitd1b310fb07e08d060c4d325c1059c007954303f3
tree5bccbaf328e41761973413eec43c2fec80b1660c
parentd2a5faac24e9e643f03e1d77032646d4d9219d23
Large server side refactoring for DateField

Changes summarized:
 - removed invalid implementation of isEmpty(). Can't work with validation/buffering. Added in #4311, fixes #5927
 - removed invalid overridden implementation of valueChange. Added in #5810 and #5277.
 - moved a lot of flag updates and other internal field updates to setInternalValue. Also tasks removed from valueChange(), now possible since #6002 is fixed.
 - n non-reported bugs fixed (at least some related to read buffering)
 - m regressions caused just a moment before large milestone?

Trivial minor enhancements to improve maintainability:
 - removed obsolete toString() method (same as in AbstractField)
 - added <?> to getType() method as in super implementation, fixes a compilation warning
 - decreased visibility of setValue(Object, boolean), should be exposed to extending classes only - probably opened accidentally
 - removed obsolete implementation of setValue(Object), did exactly the same thing as the parent
 - javadoc typo

Sorry for a large patch. Don't like these either. I just don't see how this could have been fixed in smaller patches (except for the trivial fixes).

svn changeset:16797/svn branch:6.5
src/com/vaadin/ui/DateField.java