Skip to content

Commit

Permalink
Fix compiler errors/warnings in o.g.w.d.l.AttributeEditPage and Attri…
Browse files Browse the repository at this point in the history
…buteDescription
  • Loading branch information
smithkm committed Jan 18, 2016
1 parent 93c0a21 commit 1513b64
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 17 deletions.
Expand Up @@ -28,7 +28,7 @@
@SuppressWarnings("serial") @SuppressWarnings("serial")
class AttributeDescription implements Serializable { class AttributeDescription implements Serializable {


static final List BINDINGS = Arrays.asList(String.class, Boolean.class, Integer.class, static final List<Class<?>> BINDINGS = Arrays.asList(String.class, Boolean.class, Integer.class,
Long.class, Float.class, Double.class, Date.class, Time.class, Timestamp.class, Long.class, Float.class, Double.class, Date.class, Time.class, Timestamp.class,
Geometry.class, Point.class, LineString.class, Polygon.class, MultiPoint.class, Geometry.class, Point.class, LineString.class, Polygon.class, MultiPoint.class,
MultiLineString.class, MultiPolygon.class, GeometryCollection.class); MultiLineString.class, MultiPolygon.class, GeometryCollection.class);
Expand All @@ -45,7 +45,7 @@ class AttributeDescription implements Serializable {


String name; String name;


Class binding = String.class; Class<?> binding = String.class;


boolean nullable = true; boolean nullable = true;


Expand All @@ -59,7 +59,7 @@ class AttributeDescription implements Serializable {
* @param binding * @param binding
* @return * @return
*/ */
static String getLocalizedName(Class binding) { static String getLocalizedName(Class<?> binding) {
if (binding == null) { if (binding == null) {
return "-"; return "-";
} else if (BINDINGS.contains(binding)) { } else if (BINDINGS.contains(binding)) {
Expand All @@ -78,11 +78,11 @@ public void setName(String name) {
this.name = name; this.name = name;
} }


public Class getBinding() { public Class<?> getBinding() {
return binding; return binding;
} }


public void setBinding(Class binding) { public void setBinding(Class<?> binding) {
this.binding = binding; this.binding = binding;
} }


Expand Down
Expand Up @@ -38,11 +38,11 @@ public class AttributeEditPage extends GeoServerSecuredPage {


boolean newAttribute; boolean newAttribute;


private TextField nameField; private TextField<String> nameField;


String size; String size;


private TextField sizeField; private TextField<String> sizeField;


private CRSPanel crsField; private CRSPanel crsField;


Expand All @@ -58,12 +58,12 @@ public AttributeEditPage(final AttributeDescription attribute,
this.attribute = attribute; this.attribute = attribute;
this.size = String.valueOf(attribute.getSize()); this.size = String.valueOf(attribute.getSize());


final Form form = new Form("form", new CompoundPropertyModel(attribute)); final Form<AttributeDescription> form = new Form<>("form", new CompoundPropertyModel<>(attribute));
form.setOutputMarkupId(true); form.setOutputMarkupId(true);
add(form); add(form);


form.add(nameField = new TextField("name")); form.add(nameField = new TextField<>("name"));
DropDownChoice binding = new DropDownChoice("binding", AttributeDescription.BINDINGS, DropDownChoice<Class<?>> binding = new DropDownChoice<>("binding", AttributeDescription.BINDINGS,
new BindingChoiceRenderer()); new BindingChoiceRenderer());
binding.add(new AjaxFormSubmitBehavior("onchange") { binding.add(new AjaxFormSubmitBehavior("onchange") {


Expand Down Expand Up @@ -92,7 +92,7 @@ protected void onSubmit(AjaxRequestTarget target) {
sizeContainer = new WebMarkupContainer("sizeContainer"); sizeContainer = new WebMarkupContainer("sizeContainer");
sizeContainer.setOutputMarkupId(true); sizeContainer.setOutputMarkupId(true);
form.add(sizeContainer); form.add(sizeContainer);
sizeContainer.add(sizeField = new TextField("size", new PropertyModel(this, "size"))); sizeContainer.add(sizeField = new TextField<>("size", new PropertyModel<>(this, "size")));
sizeContainer.setVisible(String.class.equals(attribute.getBinding())); sizeContainer.setVisible(String.class.equals(attribute.getBinding()));


crsContainer = new WebMarkupContainer("crsContainer"); crsContainer = new WebMarkupContainer("crsContainer");
Expand All @@ -116,7 +116,7 @@ public void onSubmit() {
}; };
form.setDefaultButton(submit); form.setDefaultButton(submit);
form.add(submit); form.add(submit);
form.add(new Link("cancel") { form.add(new Link<Void>("cancel") {


@Override @Override
public void onClick() { public void onClick() {
Expand Down Expand Up @@ -157,14 +157,14 @@ protected boolean validate() {
return valid; return valid;
} }


static class BindingChoiceRenderer extends ChoiceRenderer { static class BindingChoiceRenderer extends ChoiceRenderer<Class<?>> {


public Object getDisplayValue(Object object) { public Object getDisplayValue(Class<?> object) {
return AttributeDescription.getLocalizedName((Class) object); return AttributeDescription.getLocalizedName((Class<?>) object);
} }


public String getIdValue(Object object, int index) { public String getIdValue(Class<?> object, int index) {
return ((Class) object).getName(); return object.getName();
} }


} }
Expand Down

0 comments on commit 1513b64

Please sign in to comment.