Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Make it possible to disable legacy Property.toString and related warning message #3475
Comments
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vaadin-bot
Apr 19, 2013
Collaborator
Originally by @Artur-
Review meeting decided this has the potential to break too much and should be reverted.
Originally by @Artur- Review meeting decided this has the potential to break too much and should be reverted. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vaadin-bot
Apr 23, 2013
Collaborator
Originally by @Artur-
Decided to keep the current behavior for now to avoid breaking legacy code but to add a way to disable the feature and related log message.
Originally by @Artur- Decided to keep the current behavior for now to avoid breaking legacy code but to add a way to disable the feature and related log message. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
Show comment Hide comment
vaadin-bot
Apr 24, 2013
Collaborator
Originally by @Artur-
The init parameter "legacyPropertyToString" now controls the behavior of Property.toString for old Property implementations. A value of "true" or "warning" enables the legacy behavior, "false" disables it. If set to "warning" (the default) then a warning will be emitted when toString is called (like in 7.0).
Originally by @Artur- The init parameter "legacyPropertyToString" now controls the behavior of Property.toString for old Property implementations. A value of "true" or "warning" enables the legacy behavior, "false" disables it. If set to "warning" (the default) then a warning will be emitted when toString is called (like in 7.0). |
vaadin-bot commentedFeb 1, 2013
Originally by @Artur-
The toString() method in AbstractField and Label logs an error message and returns the property value in 7.0. The log message promises neither will happen in 7.1 anymore.
Imported from https://dev.vaadin.com/ issue #10916