Changeset 36484 in osm for applications/editors/josm/plugins

Timestamp:

2026-02-12T17:33:22+01:00 (2 days ago)

Author:

stoecker

Message:

allow to select side for terracer, fix #11783, patch by Till_btn

Location:

applications/editors/josm/plugins/terracer

Files:

• src/org/openstreetmap/josm/plugins/terracer/HouseNumberInputDialog.java (modified) (2 diffs)
• src/org/openstreetmap/josm/plugins/terracer/HouseNumberInputHandler.java (modified) (3 diffs)
• src/org/openstreetmap/josm/plugins/terracer/TerracerAction.java (modified) (6 diffs)
• test/unit/org/openstreetmap/josm/plugins/terracer/ReverseTerraceActionTest.java (modified) (1 diff)

applications/editors/josm/plugins/terracer/src/org/openstreetmap/josm/plugins/terracer/HouseNumberInputDialog.java

@@ -95 +95 @@
         super(MainApplication.getMainFrame(),
                 tr("Terrace a house"),
-                new String[] {tr("OK"), tr("Cancel")},
+                new String[] {tr("Long side"), tr("Short side"), tr("Cancel")},
                 true
         );
@@ -107 +107 @@
         JPanel content = getInputPanel();
         setContent(content);
-        setButtonIcons("ok", "cancel");
+        setButtonIcons("ok", "ok", "cancel");
         getJContentPane();
         initialize();

applications/editors/josm/plugins/terracer/src/org/openstreetmap/josm/plugins/terracer/HouseNumberInputHandler.java

@@ -129 +129 @@
                         message);
 
-        JButton okButton = getButton(dialog, "OK");
-        if (okButton != null)
-            okButton.setEnabled(isOk);
+        JButton longSideButton = getButton(dialog, tr("Long side"));
+        if (longSideButton != null)
+            longSideButton.setEnabled(isOk);
+        JButton shortSideButton = getButton(dialog, tr("Short side"));
+        if (shortSideButton != null)
+            shortSideButton.setEnabled(isOk);
         if (isOk) {
 
@@ -272 +275 @@
         if (e.getSource() instanceof JButton) {
             JButton button = (JButton) e.getSource();
-            if (tr("OK").equals(button.getActionCommand()) && button.isEnabled()) {
+            if (tr("Long side").equals(button.getActionCommand()) && button.isEnabled()) {
                 if (validateInput()) {
                     saveValues();
@@ -288 +291 @@
                         streetName(),
                         doHandleRelation(),
-                        doKeepOutline(), buildingType());
+                        doKeepOutline(), buildingType(),
+                        false);
+
+                    this.dialog.setVisible(false);
+                }
+            } else if (tr("Short side").equals(button.getActionCommand()) && button.isEnabled()) {
+                if (validateInput()) {
+                    saveValues();
+
+                    terracerAction.terraceBuilding(
+                        outline,
+                        init,
+                        street,
+                        associatedStreet,
+                        segments(),
+                        dialog.lo.getText(),
+                        dialog.hi.getText(),
+                        stepSize(),
+                        housenumbers,
+                        streetName(),
+                        doHandleRelation(),
+                        doKeepOutline(), buildingType(),
+                        true);
 
                     this.dialog.setVisible(false);

applications/editors/josm/plugins/terracer/src/org/openstreetmap/josm/plugins/terracer/TerracerAction.java

@@ -216 +216 @@
             // Don't open the dialog
             terraceBuilding(outline, init, street, associatedStreet, 0, null, null, 0,
-                    housenumbers, streetname, associatedStreet != null, false, "yes");
+                    housenumbers, streetname, associatedStreet != null, false, "yes", false);
         } else {
             String title = trn("Change {0} object", "Change {0} objects", sel.size(), sel.size());
@@ -301 +301 @@
      * @param keepOutline If the outline way should be kept
      * @param buildingValue The value for {@code building} key to add
+     * @param invertSide If true, splits on the short side instead of the long side
      */
     public void terraceBuilding(final Way outline, Node init, Way street, Relation associatedStreet, Integer segments,
                 String start, String end, int step, List<Node> housenumbers, String streetName, boolean handleRelations,
-                boolean keepOutline, String buildingValue) {
+                boolean keepOutline, String buildingValue, boolean invertSide) {
         final int nb;
         Integer to;
@@ -327 +328 @@
 
         // now find which is the longest side connecting the first node
-        Pair<Way, Way> interp = findFrontAndBack(outline);
+        Pair<Way, Way> interp = findFrontAndBack(outline, invertSide);
 
         final boolean swap = init != null && (init.equals(interp.a.lastNode()) || init.equals(interp.b.lastNode()));
@@ -597 +598 @@
      *
      * @param w The way to analyse.
+     * @param invertSide If true, forces the use of the alternative (adjacent) pair
+     *                   of sides, effectively rotating the split axis.
      * @return A pair of ways (front, back) pointing in the same directions.
      */
-    private static Pair<Way, Way> findFrontAndBack(Way w) {
+    private static Pair<Way, Way> findFrontAndBack(Way w, boolean invertSide) {
         // calculate the "side-ness" score for each segment of the way
         double[] sideness = calculateSideness(w);
@@ -624 +627 @@
         if (sideLength(w, side1) > sideLength(w, side1 + 1)
                 && Math.abs(sideness[side1] - sideness[(side1 + 1) % (w.getNodesCount() - 1)]) < 0.001) {
+            side1 = (side1 + 1) % (w.getNodesCount() - 1);
+            side2 = (side2 + 1) % (w.getNodesCount() - 1);
+        }
+
+        // if inverted, shift the selection to the adjacent segments to use the
+        // alternative pair of sides (e.g. top/bottom instead of left/right).
+        if (invertSide) {
             side1 = (side1 + 1) % (w.getNodesCount() - 1);
             side2 = (side2 + 1) % (w.getNodesCount() - 1);
@@ -791 +801 @@
          * The value to sort
          */
-        public final double x;
+        private final double x;
         /**
          * The index in the original array
          */
-        public final int i;
+        private final int i;
 
         SortWithIndex(double a, int b) {

applications/editors/josm/plugins/terracer/test/unit/org/openstreetmap/josm/plugins/terracer/ReverseTerraceActionTest.java

@@ -52 +52 @@
         dataSet.setSelected(way2);
         new TerracerAction().terraceBuilding(way2, null, null, null, 5,
-                "80", "88", 2, Collections.emptyList(), null, false, false, "apartments");
+                "80", "88", 2, Collections.emptyList(), null, false, false, "apartments", false);
         final List<Way> addressedWays = dataSet.getWays().stream().filter(w -> w != way1)
                 .sorted(Comparator.comparingLong(w -> -w.getUniqueId())).collect(Collectors.toList());