forked from geoserver/geoserver
-
Notifications
You must be signed in to change notification settings - Fork 1
Wicket migration code sprint
Jody Garnett edited this page Jan 15, 2016
·
28 revisions
Getting started: https://dzone.com/refcardz/getting-started-apache-wicket Wicket official docs: https://ci.apache.org/projects/wicket/guide/7.x/guide/single.html
Migration references:
Old branch migrating a 2013 GeoServer towards 6.0 and full diff against its baseline
Justin's previous page Wicket-UI-TODOs on this subject.
ResourceReference -> PackageResourceReference- ParamResourceModel -> StringResourceModel (the Wicket 7 class is fluent enough that we don't need ParamResourceModel anymore)
PageParameters -> has been moved in a different pakage, the way to fill it is differentAjaxRequestTarget.addComponent --> AjaxRequestTarget.addSimpleAttributeModifier --> AttributeModifier.replace
- Add all missing generics
- Check internationalization
- Check that WicketCallback changes did not break the callbacks (some are pretty important)
- Check that the PageParameters migration picked the right
to<TargetType>
vstoOptional<TargetType>
form - AbstractValidator is no more there, see the guide on how to implement from interface
- Component visit has changed, see this bit of the Wicket 1.5 migration guide for reference
- IAjaxCallDecorator -> IAjaxCallListener, see the Ajax guide for details
Some ideas for making use of Java 8 features when updating wicket code.
Use of lambda to replace IVisitor:
visitChildren((Component c, final IVisit<Void> visit) -> {
c.setEnabled(enabled);
});
©2020 Open Source Geospatial Foundation