Ticket #3676: deprecate-osmprimitive-selection-methods.patch

(a) core/build.xml~deprecate-osmprimitive-selection-methods vs. (b) core-dave/build.xml

@@ -66 +66 @@
                 <javac srcdir="src" classpathref="classpath" destdir="build"
                                 target="1.5" source="1.5" debug="on" encoding="UTF-8">
                         <compilerarg value="-Xlint:deprecation"/>
-                        <compilerarg value="-Xlint:unchecked"/>
+                        <!-- compilerarg value="-Xlint:unchecked"/ -->
                 </javac>
         </target>
 

(a) core/src/org/openstreetmap/josm/actions/search/SearchAction.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/actions/search/SearchAction.java

@@ -25 +25 @@
 import org.openstreetmap.josm.tools.GBC;
 import org.openstreetmap.josm.tools.Shortcut;
 import org.openstreetmap.josm.data.osm.Filter;
+import org.openstreetmap.josm.data.osm.DataSet;
 
 public class SearchAction extends JosmAction{
 
@@ -88 +89 @@
         left.add(label, GBC.eop());
         left.add(input, GBC.eop().fill(GBC.HORIZONTAL));
         left.add(replace, GBC.eol());
+        DataSet ds = Main.main.getCurrentDataSet();
         left.add(add, GBC.eol());
         left.add(remove, GBC.eol());
         left.add(in_selection, GBC.eop());
@@ -236 +238 @@
         //            }
         //        }
 
-        Collection<OsmPrimitive> sel = Main.main.getCurrentDataSet().getSelected();
+        final DataSet ds = Main.main.getCurrentDataSet();
+        Collection<OsmPrimitive> sel = ds.getSelected();
         int foundMatches = getSelection(s, sel, new Function(){
             public Boolean isSomething(OsmPrimitive o){
-                return o.isSelected();
+                return ds.isSelected(o);
             }
         });
-        Main.main.getCurrentDataSet().setSelected(sel);
+        ds.setSelected(sel);
         if (foundMatches == 0) {
             String msg = null;
             if (s.mode == SearchMode.replace) {

(a) core/src/org/openstreetmap/josm/actions/search/SearchCompiler.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

@@ -423 +423 @@
 
     private static class Selected extends Match {
         @Override public boolean match(OsmPrimitive osm) {
-            return osm.isSelected();
+            return Main.main.getCurrentDataSet().isSelected(osm);
         }
         @Override public String toString() {return "selected";}
     }

(a) core/src/org/openstreetmap/josm/actions/UnGlueAction.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/actions/UnGlueAction.java

@@ -151 +151 @@
 
         Node c = new Node(selectedNode);
         c.removeAll();
-        c.setSelected(false);
+        getCurrentDataSet().clearSelection(c);
         cmds.add(new ChangeCommand(selectedNode, c));
 
         Node n = new Node(selectedNode);

(a) core/src/org/openstreetmap/josm/data/osm/DataSet.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/data/osm/DataSet.java

@@ -216 +216 @@
         }
     }
 
+    public boolean addSelected(OsmPrimitive osm) {
+        osm.setSelected(true);
+        return true;
+    }
+
+    public boolean toggleSelected(OsmPrimitive osm) {
+        osm.setSelected(!osm.isSelected());
+        return true;
+    }
+    public boolean isSelected(OsmPrimitive osm) {
+        return osm.isSelected();
+    }
+
     public void setDisabled(OsmPrimitive... osm) {
         if (osm.length == 1 && osm[0] == null) {
             setDisabled();
@@ -282 +295 @@
      * Remove the selection from every value in the collection.
      * @param list The collection to remove the selection from.
      */
+    public void clearSelection(OsmPrimitive... osm) {
+        clearSelection(Arrays.asList(osm));
+    }
     private void clearSelection(Collection<? extends OsmPrimitive> list) {
         if (list == null)
             return;

(a) core/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/data/osm/OsmPrimitive.java

@@ -220 +220 @@
      * @param selected  true, if this primitive is selected; false, otherwise
      * @since 1899
      */
-    public void setSelected(boolean selected) {
+    @Deprecated public void setSelected(boolean selected) {
         if (selected) {
             flags |= FLAG_SELECTED;
         } else {
@@ -233 +233 @@
      * @return true, if this primitive is selected
      * @since 1899
      */
-    public boolean isSelected() {
+    @Deprecated public boolean isSelected() {
         return (flags & FLAG_SELECTED) != 0;
     }
 
@@ -773 +773 @@
      * @return the display name
      */
     public abstract String getDisplayName(NameFormatter formatter);
-}
- No newline at end of file
+}

(a) core/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java

@@ -127 +127 @@
                         RenderingHints.VALUE_ANTIALIAS_ON : RenderingHints.VALUE_ANTIALIAS_OFF);
     }
 
+    DataSet ds;
     public void visitAll(DataSet data, Boolean virtual) {
+        this.ds = data;
         //boolean profiler = Main.pref.getBoolean("simplepaint.profiler",false);
         //long profilerStart = java.lang.System.currentTimeMillis();
         //long profilerLast = profilerStart;
@@ -148 +150 @@
            require changing the colour while painting... */
         //profilerN = 0;
         for (final OsmPrimitive osm : data.relations)
-            if (!osm.isDeleted() && !osm.isSelected() && !osm.isFiltered())
+            if (!osm.isDeleted() && !ds.isSelected(osm) && !osm.isFiltered())
             {
                 osm.visit(this);
                 //        profilerN++;
@@ -162 +164 @@
 
         //profilerN = 0;
         for (final OsmPrimitive osm : data.ways)
-            if (!osm.isDeleted() && !osm.isSelected() && !osm.isFiltered() && osm.isTagged())
+            if (!osm.isDeleted() && !ds.isSelected(osm) && !osm.isFiltered() && osm.isTagged())
             {
                 osm.visit(this);
                 //        profilerN++;
@@ -170 +172 @@
         displaySegments();
 
         for (final OsmPrimitive osm : data.ways)
-            if (!osm.isDeleted() && !osm.isSelected() && !osm.isFiltered() && !osm.isTagged())
+            if (!osm.isDeleted() && !ds.isSelected(osm) && !osm.isFiltered() && !osm.isTagged())
             {
                 osm.visit(this);
                 //        profilerN++;
@@ -201 +203 @@
 
         //profilerN = 0;
         for (final OsmPrimitive osm : data.nodes)
-            if (!osm.isDeleted() && !osm.isSelected() && !osm.isFiltered())
+            if (!osm.isDeleted() && !ds.isSelected(osm) && !osm.isFiltered())
             {
                 osm.visit(this);
                 //        profilerN++;
@@ -252 +254 @@
             drawNode(n, inactiveColor, unselectedNodeSize, unselectedNodeRadius, fillUnselectedNode);
         } else if (n.highlighted) {
             drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
-        } else if (n.isSelected()) {
+        } else if (ds.isSelected(n)) {
             drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
         } else if(n.isTagged()) {
             drawNode(n, nodeColor, taggedNodeSize, taggedNodeRadius, fillUnselectedNode);
@@ -304 +306 @@
         /* show direction arrows, if draw.segment.relevant_directions_only is not set, the way is tagged with a direction key
            (even if the tag is negated as in oneway=false) or the way is selected */
 
-        boolean showThisDirectionArrow = w.isSelected()
+        boolean showThisDirectionArrow = ds.isSelected(w)
         || (showDirectionArrow && (!showRelevantDirectionsOnly || w.hasDirectionKeys()));
         /* head only takes over control if the option is true,
            the direction should be shown at all and not only because it's selected */
-        boolean showOnlyHeadArrowOnly = showThisDirectionArrow && !w.isSelected() && showHeadArrowOnly;
+        boolean showOnlyHeadArrowOnly = showThisDirectionArrow && !ds.isSelected(w) && showHeadArrowOnly;
         Color wayColor;
 
         if (inactive || w.isDisabled()) {
             wayColor = inactiveColor;
         } else if(w.highlighted) {
             wayColor = highlightColor;
-        } else if(w.isSelected()) {
+        } else if(ds.isSelected(w)) {
             wayColor = selectedColor;
         } else if (!w.isTagged()) {
             wayColor = untaggedWayColor;
@@ -346 +348 @@
         Color col;
         if (inactive || r.isDisabled()) {
             col = inactiveColor;
-        } else if (r.isSelected()) {
+        } else if (ds.isSelected(r)) {
             col = selectedColor;
         } else {
             col = relationColor;

(a) core/src/org/openstreetmap/josm/gui/NavigatableComponent.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/gui/NavigatableComponent.java

@@ -317 +317 @@
             }
             // when multiple nodes on one point, prefer new or selected nodes
             else if(dist == minDistanceSq && minPrimitive != null
-                    && ((n.getId() == 0 && n.isSelected())
-                            || (!minPrimitive.isSelected() && (n.isSelected() || n.getId() == 0)))) {
+                    && ((n.getId() == 0 && ds.isSelected(n))
+                            || (!ds.isSelected(minPrimitive) && (ds.isSelected(n) || n.getId() == 0)))) {
                 minPrimitive = n;
             }
         }
@@ -359 +359 @@
                 double b = p.distanceSq(A);
                 double perDist = a-(a-b+c)*(a-b+c)/4/c; // perpendicular distance squared
                 if (perDist < snapDistance && a < c+snapDistance && b < c+snapDistance) {
-                    if(w.isSelected()) {
+                    if(ds.isSelected(w)) {
                         perDist -= 0.00001;
                     }
                     List<WaySegment> l;

(a) core/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/actions/mapmode/DrawAction.java

@@ -324 +324 @@
         updateKeyModifiers(e);
         mousePos = e.getPoint();
 
-        Collection<OsmPrimitive> selection = getCurrentDataSet().getSelected();
+        DataSet ds = getCurrentDataSet();
+        Collection<OsmPrimitive> selection = ds.getSelected();
         Collection<Command> cmds = new LinkedList<Command>();
 
         ArrayList<Way> reuseWays = new ArrayList<Way>(),
@@ -404 +405 @@
                     // but pressing ALT prevents this. Therefore we must de-select the way manually
                     // here so /only/ the new way will be selected after this method finishes.
                     if(alt) {
-                        wnew.setSelected(false);
+                        ds.addSelected(wnew);
                     }
 
                     cmds.add(new ChangeCommand(insertPoint.getKey(), wnew));
@@ -518 +519 @@
 
                 extendedWay = true;
                 getCurrentDataSet().setSelected(way);
-                DataSet.fireSelectionChanged(getCurrentDataSet().getSelected());
+                DataSet.fireSelectionChanged(ds.getSelected());
             }
         }
 
@@ -531 +532 @@
             } else {
                 title = tr("Add node into way");
                 for (Way w : reuseWays) {
-                    w.setSelected(false);
+                    ds.clearSelection(w);
                 }
             }
             getCurrentDataSet().setSelected(n);

(a) core/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

@@ -151 +151 @@
         //    return;
 
         if (nodeStyle != null && isZoomOk(nodeStyle) && showIcons > dist) {
-            drawNode(n, nodeStyle.icon, nodeStyle.annotate, n.isSelected());
+            drawNode(n, nodeStyle.icon, nodeStyle.annotate, data.isSelected(n));
         } else if (n.highlighted) {
             drawNode(n, highlightColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
-        } else if (n.isSelected()) {
+        } else if (data.isSelected(n)) {
             drawNode(n, selectedColor, selectedNodeSize, selectedNodeRadius, fillSelectedNode);
         } else if (n.isTagged()) {
             drawNode(n, nodeColor, taggedNodeSize, taggedNodeRadius, fillUnselectedNode);
@@ -226 +226 @@
             /* way without style */
             //profilerVisibleWays++;
             //if(!profilerOmitDraw)
-            drawWay(w, null, untaggedColor, w.isSelected());
+            drawWay(w, null, untaggedColor, data.isSelected(w));
         }
         else if(wayStyle instanceof LineElemStyle)
         {
             /* way with line style */
             //profilerVisibleWays++;
             //if(!profilerOmitDraw)
-            drawWay(w, (LineElemStyle)wayStyle, untaggedColor, w.isSelected());
+            drawWay(w, (LineElemStyle)wayStyle, untaggedColor, data.isSelected(w));
         }
         else if (wayStyle instanceof AreaElemStyle)
         {
@@ -244 +244 @@
             if (fillAreas > dist)
             {
                 //    profilerVisibleAreas++;
-                drawArea(w, w.isSelected() ? selectedColor : areaStyle.color);
+                drawArea(w, data.isSelected(w) ? selectedColor : areaStyle.color);
                 if(!w.isClosed()) {
                     w.putError(tr("Area style way is not closed."), true);
                 }
             }
-            drawWay(w, areaStyle.line, areaStyle.color, w.isSelected());
+            drawWay(w, areaStyle.line, areaStyle.color, data.isSelected(w));
             //}
         }
     }
@@ -258 +258 @@
         /* show direction arrows, if draw.segment.relevant_directions_only is not set,
            the way is tagged with a direction key
            (even if the tag is negated as in oneway=false) or the way is selected */
-        boolean showDirection = w.isSelected() || ((!useRealWidth) && (showDirectionArrow
+        boolean showDirection = data.isSelected(w) || ((!useRealWidth) && (showDirectionArrow
                 && (!showRelevantDirectionsOnly || w.hasDirectionKeys())));
         /* head only takes over control if the option is true,
            the direction should be shown at all and not only because it's selected */
-        boolean showOnlyHeadArrowOnly = showDirection && !w.isSelected() && showHeadArrowOnly;
+        boolean showOnlyHeadArrowOnly = showDirection && !data.isSelected(w) && showHeadArrowOnly;
         int width = defaultSegmentWidth;
         int realWidth = 0; /* the real width of the element in meters */
         float dashed[] = new float[0];
@@ -306 +306 @@
 
         if(w.highlighted) {
             color = highlightColor;
-        } else if(w.isSelected()) {
+        } else if(data.isSelected(w)) {
             color = selectedColor;
         } else if(w.isDisabled()) {
             color = inactiveColor;
@@ -324 +324 @@
                     {
                         if(lastN != null)
                         {
-                            drawSeg(lastN, n, s.color != null  && !w.isSelected() ? s.color : color,
+                            drawSeg(lastN, n, s.color != null  && !data.isSelected(w) ? s.color : color,
                                     false, s.getWidth(width), s.dashed, s.dashedColor);
                         }
                         lastN = n;
@@ -358 +358 @@
                     {
                         if(lastN != null)
                         {
-                            drawSeg(lastN, n, s.color != null && !w.isSelected() ? s.color : color,
+                            drawSeg(lastN, n, s.color != null && !data.isSelected(w) ? s.color : color,
                                     false, s.getWidth(width), s.dashed, s.dashedColor);
                         }
                         lastN = n;
@@ -404 +404 @@
                     {
                         Way c = (Way)joinArray[i];
                         if(w == null)
-                        { w = c; selected = w.isSelected(); joinArray[i] = null; --left; }
+                        { w = c; selected = data.isSelected(w); joinArray[i] = null; --left; }
                         else
                         {
                             int mode = 0;
@@ -440 +440 @@
                             {
                                 joinArray[i] = null;
                                 joined = true;
-                                if(c.isSelected()) {
+                                if(data.isSelected(c)) {
                                     selected = true;
                                 }
                                 --left;
@@ -473 +473 @@
             {
                 w = new Way(w);
                 w.setNodes(n);
-                w.setSelected(selected);
+                if (selected)
+                    data.setSelected(w);
+                else
+                    data.clearSelection(w);
             }
             if(!w.isClosed())
             {
@@ -550 +553 @@
             drawRestriction(r);
         }
 
-        if(r.isSelected()) /* draw ways*/
+        if(data.isSelected(r)) /* draw ways*/
         {
             for (RelationMember m : r.getMembers())
             {
@@ -827 +830 @@
         int w = smallIcon.getIconWidth(), h=smallIcon.getIconHeight();
         smallIcon.paintIcon ( Main.map.mapView, g, (int)(pVia.x+vx+vx2)-w/2, (int)(pVia.y+vy+vy2)-h/2 );
 
-        if (r.isSelected())
+        if (data.isSelected(r))
         {
             g.setColor (  selectedColor );
             g.drawRect ((int)(pVia.x+vx+vx2)-w/2-2,(int)(pVia.y+vy+vy2)-h/2-2, w+4, h+4);
@@ -871 +874 @@
                                 m.getRole(), w.getDisplayName(DefaultNameFormatter.getInstance())), true);
                         if(!m.hasRole()) {
                             outer.add(w);
-                        } else if(r.isSelected()) {
+                        } else if(data.isSelected(r)) {
                             drawSelectedMember(m.getMember(), styles != null
                                     ? getPrimitiveStyle(m.getMember()) : null, true, true);
                         }
@@ -1051 +1054 @@
                     Polygon p = pd.get();
                     if(isPolygonVisible(p))
                     {
-                        drawAreaPolygon(p, (pd.way.isSelected() || r.isSelected()) ? selectedColor
+                        drawAreaPolygon(p, (data.isSelected(pd.way) || data.isSelected(r)) ? selectedColor
                                 : areaStyle.color);
                         visible = true;
                     }
@@ -1077 +1080 @@
                             || outer.size() == 0))
                     {
                         drawWay(wInner, ((AreaElemStyle)wayStyle).line,
-                                ((AreaElemStyle)wayStyle).color, wInner.isSelected()
-                                || r.isSelected());
+                                ((AreaElemStyle)wayStyle).color, data.isSelected(wInner)
+                                || data.isSelected(r));
                     }
                     wInner.mappaintDrawnCode = paintid;
                 }
                 else
                 {
-                    if(r.isSelected())
+                    if(data.isSelected(r))
                     {
                         drawSelectedMember(wInner, innerStyle,
-                                !wayStyle.equals(innerStyle), wInner.isSelected());
+                                !wayStyle.equals(innerStyle), data.isSelected(wInner));
                     }
                     if(wayStyle.equals(innerStyle))
                     {
                         r.putError(tr("Style for inner way ''{0}'' equals multipolygon.",
                                 wInner.getDisplayName(DefaultNameFormatter.getInstance())), false);
-                        if(!r.isSelected()) {
+                        if(!data.isSelected(r)) {
                             wInner.mappaintDrawnAreaCode = paintid;
                         }
                     }
@@ -1107 +1110 @@
                     if(zoomok)
                     {
                         drawWay(wOuter, ((AreaElemStyle)wayStyle).line,
-                                ((AreaElemStyle)wayStyle).color, wOuter.isSelected()
-                                || r.isSelected());
+                                ((AreaElemStyle)wayStyle).color, data.isSelected(wOuter)
+                                || data.isSelected(r));
                     }
                     wOuter.mappaintDrawnCode = paintid;
                 }
@@ -1120 +1123 @@
                         r.putError(tr("Style for outer way ''{0}'' mismatches.",
                                 wOuter.getDisplayName(DefaultNameFormatter.getInstance())), true);
                     }
-                    if(r.isSelected())
+                    if(data.isSelected(r))
                     {
                         drawSelectedMember(wOuter, outerStyle, false, false);
                     }
@@ -1325 +1328 @@
         textColor = Main.pref.getColor (marktr("text"), Color.WHITE);
     }
 
+    DataSet data;
+
     /* Shows areas before non-areas */
     @Override
     public void visitAll(DataSet data, Boolean virtual) {
-
+        this.data = data;
         //boolean profiler = Main.pref.getBoolean("mappaint.profiler",false);
         //profilerOmitDraw = Main.pref.getBoolean("mappaint.profiler.omitdraw",false);
 
@@ -1449 +1454 @@
             /*** WAYS (filling disabled)  ***/
             //    profilerN = 0;
             for (final OsmPrimitive osm : data.ways)
-                if (!osm.incomplete && !osm.isDeleted() && !osm.isFiltered() && !osm.isSelected()
+                if (!osm.incomplete && !osm.isDeleted() && !osm.isFiltered() && !data.isSelected(osm)
                         && osm.mappaintVisibleCode != viewid )
                 {
                     osm.visit(this);
@@ -1488 +1493 @@
         /*** NODES ***/
         //profilerN = 0;
         for (final OsmPrimitive osm : data.nodes)
-            if (!osm.incomplete && !osm.isDeleted() && (osm.isSelected() || !osm.isFiltered())
+            if (!osm.incomplete && !osm.isDeleted() && (data.isSelected(osm) || !osm.isFiltered())
                     && osm.mappaintVisibleCode != viewid && osm.mappaintDrawnCode != paintid)
             {
                 osm.visit(this);

(a) core/src/org/openstreetmap/josm/gui/MapStatus.java~deprecate-osmprimitive-selection-methods vs. (b) core-dave/src/org/openstreetmap/josm/gui/MapStatus.java

@@ -299 +299 @@
          * @param mouse modifiers
          */
         private final void popupCycleSelection(Collection<OsmPrimitive> osms, int mods) {
+            DataSet ds = Main.main.getCurrentDataSet();
             // Find some items that are required for cycling through
             OsmPrimitive firstItem = null;
             OsmPrimitive firstSelected = null;
@@ -310 +311 @@
                 if(firstSelected != null && nextSelected == null) {
                     nextSelected = osm;
                 }
-                if(firstSelected == null && osm.isSelected()) {
+                if(firstSelected == null && ds.isSelected(osm)) {
                     firstSelected = osm;
                 }
             }
@@ -318 +319 @@
             // Clear previous selection if SHIFT (add to selection) is not
             // pressed. Cannot use "setSelected()" because it will cause a
             // fireSelectionChanged event which is unnecessary at this point.
-            if((mods & MouseEvent.SHIFT_DOWN_MASK) == 0) {
-                for(OsmPrimitive o : Main.main.getCurrentDataSet().getSelected()) {
-                    o.setSelected(false);
-                }
-            }
+            if((mods & MouseEvent.SHIFT_DOWN_MASK) == 0)
+                ds.clearSelection();
 
             // This will cycle through the available items.
             if(firstSelected == null) {
-                firstItem.setSelected(true);
+                ds.addSelected(firstItem);
             } else {
-                firstSelected.setSelected(false);
+                ds.clearSelection(firstSelected);
                 if(nextSelected != null) {
-                    nextSelected.setSelected(true);
+                    ds.addSelected(nextSelected);
                 }
             }
-            DataSet.fireSelectionChanged(Main.main.getCurrentDataSet().getSelected());
+            DataSet.fireSelectionChanged(ds.getSelected());
         }
 
         /**
@@ -397 +395 @@
          * @param osm The primitive to derive the colors from
          */
         private final void popupSetLabelColors(JLabel lbl, OsmPrimitive osm) {
-            if(osm.isSelected()) {
+            DataSet ds = Main.main.getCurrentDataSet();
+            if(ds.isSelected(osm)) {
                 lbl.setBackground(SystemColor.textHighlight);
                 lbl.setForeground(SystemColor.textHighlightText);
             } else {
@@ -459 +458 @@
                     popupSetLabelColors(l, osm);
                 }
                 @Override public void mouseClicked(MouseEvent e) {
+                    DataSet ds = Main.main.getCurrentDataSet();
                     // Let the user toggle the selection
-                    osm.setSelected(!osm.isSelected());
-                    DataSet.fireSelectionChanged(Main.main.getCurrentDataSet().getSelected());
+                    ds.toggleSelected(osm);
+                    DataSet.fireSelectionChanged(ds.getSelected());
                     l.validate();
                 }
             });