Back out last change, it causes more problems than it solves

diff --git a/net/systemeD/potlatch2/mapfeatures/editors/ChoiceEditor.mxml b/net/systemeD/potlatch2/mapfeatures/editors/ChoiceEditor.mxml
index 43d6ff8eb1ca1c17601f5ab5cc3712f41bcd4ae8..d43741dd96b81689a6658d62520d12c58216500d 100644 (file)
--- a/net/systemeD/potlatch2/mapfeatures/editors/ChoiceEditor.mxml
+++ b/net/systemeD/potlatch2/mapfeatures/editors/ChoiceEditor.mxml
@@ -12,21 +12,7 @@
       <edit:itemRenderer>
         <mx:Component>
         <mx:HBox toolTip="{data.description}" horizontalScrollPolicy="off">
-          <!-- Use an object proxy to wrap calls to data.icon, otherwise you get a slew of binding warnings. Something to do
-                with it being passed through the dataProvider to an itemRenderer.
-                It's not clear whether this fixes the problem or just supresses the warnings!
-                see http://designovermatter.com/post.cfm/warning-unable-to-bind-to-property-somevar-on-object-object-class-is-not-an-ieventdispatcher -->
-          <mx:Script><![CDATA[
-            import mx.utils.ObjectProxy;
-
-            [Bindable]
-            public var dataProxy:ObjectProxy;
-
-            private function init():void {
-                dataProxy = new ObjectProxy(data);
-            }
-          ]]></mx:Script>
-          <mx:Image source="{dataProxy.icon}"/>
+          <mx:Image source="{data.icon}"/>
           <mx:Label htmlText="{data.label}"/>
         </mx:HBox>
         </mx:Component>