[OpenMap Users] [PATCH] MultiShapeLayer.java

- background and removable properties were not set in setProperties

There is a larger problem because MultiShapeLayer does not call
super.setProperties(). This will lead to inevitable problems because there
are properties (render policy, etc.) set in OMGraphicHandlerLayer that
aren't set because they haven't been copied into MultiShapeLayer's
setProperties() method.

I can use the java.lang.reflect to skip ShapLayer.setProperties() and call
OMGraphicHandlerLayer.setProperties(), but that is not very clean. I suggest
ShapeLayer and MultiShapeLayer get merged into one ShapeLayer that can
handle multiple layers. I'll try to do this soon.

Until then, here is a patch that simply sets the background and removable
properties.

>>>>>>>>>>>>>>>>>>>>>>>>>>

Index: MultiShapeLayer.java
===================================================================
RCS file:
/cvs/openmap/openmap/src/openmap/com/bbn/openmap/layer/shape/MultiShapeLayer
.java,v
retrieving revision 1.8
diff -u -w -r1.8 MultiShapeLayer.java
--- MultiShapeLayer.java 14 May 2004 20:18:48 -0000 1.8
+++ MultiShapeLayer.java 17 May 2004 14:36:13 -0000
@@ -123,6 +123,10 @@
 
         setAddToBeanContext(PropUtils.booleanFromProperties(props,
realPrefix + AddToBeanContextProperty, addToBeanContext));
 
+ setAddAsBackground(PropUtils.booleanFromProperties(props,
realPrefix + AddAsBackgroundProperty, addAsBackground));
+
+ setRemoveable(PropUtils.booleanFromProperties(props, realPrefix +
RemoveableProperty, removeable));
+
         autoPalette = PropUtils.booleanFromProperties(props, realPrefix +
AutoPaletteProperty, autoPalette);
 
         /// end from Layer.java

<<<<<<<<<<<<<<<<<<<<<<<<<<

Thanks,
Michael

--
[To unsubscribe to this list send an email to "majdart@bbn.com"
with the following text in the BODY of the message "unsubscribe openmap-users"]