fix: reversing road upgrade should now be fine #100

Author: ST-Apps

ParallelRoadTool/Extensions/Vector3Extensions.cs

@@ -40,13 +40,14 @@ public static Vector3 RotateXZ(this Vector3 vector, float angle = 90)
     ///     I guess it applies the offset to vector while also normalizing it.
     /// </summary>
     /// <param name="vector"></param>
-    /// <param name="offset"></param>
+    /// <param name="horizontalOffset"></param>
+    /// <param name="verticalOffset"></param>
     /// <returns></returns>
-    public static Vector3 NormalizeWithOffset(this Vector3 vector, float offset)
+    public static Vector3 NormalizeWithOffset(this Vector3 vector, float horizontalOffset, float verticalOffset = 0)
     {
         if (vector.magnitude <= 0) return vector;
 
-        var offsetMagnitude = offset / vector.magnitude;
-        return vector with { x = vector.x * offsetMagnitude, z = vector.z * offsetMagnitude };
+        var offsetMagnitude = horizontalOffset / vector.magnitude;
+        return vector with { x = vector.x * offsetMagnitude, y = vector.y + verticalOffset, z = vector.z * offsetMagnitude };
     }
 }

ParallelRoadTool/ParallelRoadTool.csproj

@@ -69,7 +69,6 @@
     <Compile Include="Patches\NetAIPatch.cs" />
     <Compile Include="Patches\NetManagerPatch.cs" />
     <Compile Include="Patches\NetToolCameraPatch.cs" />
-    <Compile Include="Patches\NetToolNodePatch.cs" />
     <Compile Include="Patches\NetToolsPrefabPatch.cs" />
     <Compile Include="Patches\ToolControllerPatch.cs" />
     <Compile Include="Extensions\NetInfoExtensions.cs" />

ParallelRoadTool/Patches/NetManagerPatch.cs

@@ -119,6 +119,14 @@ private static void Postfix(out ushort     segment,
                 return;
             }
 
+            if (ToolsModifierControl.GetTool<NetTool>().m_mode == NetTool.Mode.Upgrade && invert)
+            {
+                // HACK - [ISSUE-100] Prevent executing if user is reversing road direction using the upgrade tool
+                Log._Debug($"[{nameof(NetManagerPatch)}.{nameof(Postfix)}] Skipping because we're reversing a road's direction using update tool.");
+
+                return;
+            }
+
             Log._Debug($"[{nameof(NetManagerPatch)}.{nameof(Postfix)}] Adding {Singleton<ParallelRoadToolManager>.instance.SelectedNetworkTypes.Count} parallel segments");
 
             if (Singleton<ParallelRoadToolManager>.instance.IsLeftHandTraffic)