Changeset 18834 in josm

Timestamp:

2023-09-20T16:30:51+02:00 (15 months ago)

Author:

taylor.smock

Message:

Fix #23149: Add line_arrangement to power supports presets (patch by gaben)

Location:

trunk/resources/data

Files:

• defaultpresets.xml (modified) (7 diffs)
• validator/deprecated.mapcss (modified) (2 diffs)
• validator/geometry.mapcss (modified) (1 diff)

trunk/resources/data/defaultpresets.xml

@@ -391 +391 @@
         <reference ref="cables" />
         <combo key="circuits" text="Amount of circuits" values="1,2,3,4" />
+    </chunk>
+    <chunk id="line_properties">
+        <combo key="line_attachment" text="Line attachment" values="suspension,anchor,pin,pulley" values_sort="false" values_context="power" />
+        <combo key="line_arrangement" text="Line arrangement" values="horizontal,semi_horizontal,vertical,semi_vertical,triangular,delta,square,unarranged" values_sort="false" values_context="power" />
+        <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
     </chunk>
     <chunk id="power_loc">
@@ -465 +470 @@
     </chunk>
     <chunk id="power_pole">
-        <combo key="line_attachment" text="Line attachment" values="anchor,pin,pulley,suspension" values_context="power" />
-        <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
+        <reference ref="line_properties" />
         <check key="location:transition" text="Location transition" disable_off="true" />
         <reference ref="pole" />
@@ -6021 +6025 @@
             <reference ref="pole" />
             <reference ref="utilities" />
+            <optional>
+                <reference ref="line_properties"/>
+            </optional>
             <preset_link preset_name="Pole" text="Similar preset:" />
         </item> <!-- Utility Pole -->
@@ -6871 +6878 @@
             <key key="power" value="terminal" />
             <optional>
-                <combo key="line_attachment" text="Line attachment" values="anchor,pin,pulley,suspension" values_context="power" />
-                <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
+                <reference ref="line_properties" />
                 <reference ref="ref_operator" />
             </optional>
@@ -6886 +6892 @@
             <reference ref="ref_operator" />
             <combo key="design" text="Portal design" values="portal,portal_two-level,portal_three-level" values_context="power" />
-            <combo key="line_attachment" text="Line attachment" values="anchor,pin,pulley,suspension" values_context="power" />
-            <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
+            <reference ref="line_properties" />
         </item> <!-- Power Portal -->
         <item name="Power Tower" icon="presets/power/tower.svg" type="node" preset_name_label="true">
@@ -6900 +6905 @@
             <optional>
                 <combo key="design" text="Tower design" values="one-level,two-level,donau,three-level,barrel,asymmetric,triangle,flag,donau;one-level,four-level,six-level,nine-level,delta,delta_two-level,y-frame,x-frame,h-frame,guyed_h-frame,portal,portal_two-level,portal_three-level,guyed_v-frame,bipole,monopolar" values_context="power" />
-                <combo key="line_attachment" text="Line attachment" values="anchor,pin,pulley,suspension" values_context="power" />
-                <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
+                <reference ref="line_properties" />
                 <check key="triple_tower" text="Triple tower" />
                 <check key="design:incomplete" text="Incomplete tower" />
@@ -6961 +6965 @@
             <space />
             <key key="power" value="insulator" />
-            <combo key="line_attachment" text="Line attachment" values="anchor,pin,pulley,suspension" values_context="power" />
-            <combo key="line_management" text="Line management" values="branch,cross,split,straight,termination,transition,transpose" values_context="Line management" />
+            <reference ref="line_properties" />
             <reference ref="ref_operator" />
         </item> <!-- Insulator -->

trunk/resources/data/validator/deprecated.mapcss

@@ -1898 +1898 @@
   suggestAlternative: "design";
   suggestAlternative: "structure";
+  suggestAlternative: "line_arrangement";
+  suggestAlternative: "line_attachment";
   suggestAlternative: "line_management";
-  suggestAlternative: "line_attachment";
   set generic_power_tower_type_warning;
   group: tr("deprecated tagging");
@@ -1958 +1959 @@
 *[pole:type][power][power=~/^(tower|pole|insulator|portal|terminal)$/]!.power_pole_type_warning!.generic_power_tower_type_warning {
   throwWarning: tr("{0} is deprecated for {1}", "{0.key}", "{1.tag}");
+  suggestAlternative: "line_arrangement";
+  suggestAlternative: "line_attachment";
   suggestAlternative: "line_management";
-  suggestAlternative: "line_attachment";
   group: tr("deprecated tagging");
 }

trunk/resources/data/validator/geometry.mapcss

@@ -128 +128 @@
 way[voltage:tertiary],
 way[transformer],
+way[line_arrangement],
 way[line_attachment],
 way[line_management],