Do not use localized strings for core operations

cesarBLG · cesarBLG · commit c1f443e59ab5 · 2025-09-10T13:25:18.000+02:00
diff --git a/Source/Orts.Simulation/Common/Scripting/PowerSupply/ControlCarPowerSupply.cs b/Source/Orts.Simulation/Common/Scripting/PowerSupply/ControlCarPowerSupply.cs
@@ -49,6 +49,7 @@ protected override void SetCurrentMainPowerSupplyState(PowerSupplyState state) {
         protected override void SetCurrentAuxiliaryPowerSupplyState(PowerSupplyState state) {}
         protected override void SetCurrentElectricTrainSupplyState(PowerSupplyState state) {}
         protected override void SetCurrentDynamicBrakeAvailability(bool avail) {}
+        public override PowerSupplyState GetPowerStatus() => PowerSupplyState.Unavailable;
         public void SignalEventToControlActiveLocomotive(PowerSupplyEvent evt)
         {
             ControlActiveLocomotive?.LocomotivePowerSupply.HandleEvent(evt);
diff --git a/Source/Orts.Simulation/Common/Scripting/PowerSupply/DieselPowerSupply.cs b/Source/Orts.Simulation/Common/Scripting/PowerSupply/DieselPowerSupply.cs
@@ -90,7 +90,27 @@ public float DieselEngineMinRpm
         /// Closing authorization of the traction cut-off relay
         /// </summary>
         public bool TractionCutOffRelayClosingAuthorization() => TractionCutOffRelay.ClosingAuthorization;
-        
+
+        public override PowerSupplyState GetPowerStatus()
+        {
+            var status = base.GetPowerStatus();
+            PowerSupplyState engineStatus;
+            switch (CurrentDieselEnginesState())
+            {
+                case DieselEngineState.Running:
+                    engineStatus = PowerSupplyState.PowerOn;
+                    break;
+                case DieselEngineState.Starting:
+                    engineStatus = PowerSupplyState.PowerOnOngoing;
+                    break;
+                default:
+                    engineStatus = PowerSupplyState.PowerOff;
+                    break;
+            }
+            if (status == engineStatus) return status;
+            return PowerSupplyState.PowerOnOngoing;
+        }
+
         /// <summary>
         /// Sends an event to all diesel engines
         /// </summary>
diff --git a/Source/Orts.Simulation/Common/Scripting/PowerSupply/ElectricPowerSupply.cs b/Source/Orts.Simulation/Common/Scripting/PowerSupply/ElectricPowerSupply.cs
@@ -128,6 +128,34 @@ protected float PantographVoltageVDC
         /// </summary>
         protected void SetFilterVoltageV(float voltage) => EpsHost.FilterVoltageV = voltage;
 
+        public override PowerSupplyState GetPowerStatus()
+        {
+            var status = base.GetPowerStatus();
+            PowerSupplyState electricStatus;
+            switch (CurrentPantographState())
+            {
+                case PantographState.Up:
+                    switch (CurrentCircuitBreakerState())
+                    {
+                        case CircuitBreakerState.Closed:
+                            electricStatus = PowerSupplyState.PowerOn;
+                            break;
+                        default:
+                            electricStatus = PowerSupplyState.PowerOnOngoing;
+                            break;
+                    }
+                    break;
+                case PantographState.Raising:
+                    electricStatus = PowerSupplyState.PowerOnOngoing;
+                    break;
+                default:
+                    electricStatus = PowerSupplyState.PowerOff;
+                    break;
+            }
+            if (status == electricStatus) return status;
+            return PowerSupplyState.PowerOnOngoing;
+        }
+
         /// <summary>
         /// Sends an event to the circuit breaker
         /// </summary>
diff --git a/Source/Orts.Simulation/Common/Scripting/PowerSupply/LocomotivePowerSupply.cs b/Source/Orts.Simulation/Common/Scripting/PowerSupply/LocomotivePowerSupply.cs
@@ -293,6 +293,20 @@ protected void SetCustomizedCabviewControlName(int index, string name)
         /// </summary>
         protected virtual void SetCurrentDynamicBrakeAvailability(bool avail) => LpsHost.DynamicBrakeAvailable = avail;
 
+        /// <summary>
+        /// Called by other subsystems to determine whether the locomotive is powered
+        /// </summary>
+        /// <returns>
+        /// PowerSupplyState.PowerOff if the locomotive is unpowered
+        /// PowerSupplyState.PowerOnOngoing if the locomotive is in a power on sequence, but not all subsystems are ready
+        /// PowerSupplyState.PowerOn if the locomotive is ready for service (all necessary subystems are connected)
+        /// </returns>
+        public virtual PowerSupplyState GetPowerStatus()
+        {
+            if (LpsHost.MainPowerSupplyState == LpsHost.AuxiliaryPowerSupplyState) return LpsHost.MainPowerSupplyState;
+            return PowerSupplyState.PowerOnOngoing;
+        }
+
         /// <summary>
         /// Sends an event to the master switch
         /// </summary>
diff --git a/Source/Orts.Simulation/Simulation/RollingStocks/SubSystems/PowerSupplies/ILocomotivePowerSupply.cs b/Source/Orts.Simulation/Simulation/RollingStocks/SubSystems/PowerSupplies/ILocomotivePowerSupply.cs
@@ -47,6 +47,8 @@ public interface ILocomotivePowerSupply : IPowerSupply
         bool ServiceRetentionCancellationButton { get; set; }
         bool ServiceRetentionActive { get; set; }
 
+        PowerSupplyState GetPowerStatus();
+
         void HandleEventFromTcs(PowerSupplyEvent evt);
         void HandleEventFromTcs(PowerSupplyEvent evt, int id);
         void HandleEventFromTcs(PowerSupplyEvent evt, string message);
diff --git a/Source/Orts.Simulation/Simulation/RollingStocks/SubSystems/PowerSupplies/LocomotivePowerSupply.cs b/Source/Orts.Simulation/Simulation/RollingStocks/SubSystems/PowerSupplies/LocomotivePowerSupply.cs
@@ -266,6 +266,8 @@ public virtual void Update(float elapsedClockSeconds)
             ElectricTrainSupplySwitch.Update(elapsedClockSeconds);
         }
 
+        public PowerSupplyState GetPowerStatus() => AbstractScript?.GetPowerStatus() ?? PowerSupplyState.Unavailable;
+
         public void HandleEvent(PowerSupplyEvent evt)
         {
             AbstractScript?.HandleEvent(evt);
diff --git a/Source/RunActivity/Viewer3D/Popups/TrainCarOperationsViewerWindow.cs b/Source/RunActivity/Viewer3D/Popups/TrainCarOperationsViewerWindow.cs
@@ -27,10 +27,12 @@
 using Orts.Simulation.RollingStocks.SubSystems.PowerSupplies;
 using ORTS.Common;
 using ORTS.Common.Input;
+using ORTS.Scripting.Api;
 using System;
 using System.Collections.Generic;
 using System.IO;
 using System.Linq;
+using System.Windows.Forms;
 
 namespace Orts.Viewer3D.Popups
 {
@@ -81,10 +83,9 @@ public class TrainCarOperationsViewerWindow : Window
 
         public List<bool> AngleCockAPartiallyOpened = new List<bool>();
         public List<bool> AngleCockBPartiallyOpened = new List<bool>();
-        public string BatteryStatus;
-        string CircuitBreakerState;
+        public bool BatterySwitchOn;
+        public PowerSupplyState PowerSupplyStatus;
         public int LocoRowCount;
-        public string PowerSupplyStatus;
         public int RowsCount;
         public int SpacerRowCount;
         public int SymbolsRowCount;
@@ -537,10 +538,8 @@ public override void PrepareFrame(ElapsedTime elapsedTime, bool updateFull)
                     carOperations.CarOperationChanged = carOperations.Visible && carOperations.CarOperationChanged;
                 }
                 // Updates power supply status
-                else if (isElectricDieselLocomotive &&
-                     (PowerSupplyStatus != null && PowerSupplyStatus != Owner.Viewer.PlayerTrain.Cars[CarPosition].GetStatus()
-                      || (BatteryStatus != null && BatteryStatus != Owner.Viewer.PlayerTrain.Cars[CarPosition].GetStatus())
-                      || (CircuitBreakerState != null && CircuitBreakerState != (trainCar as MSTSElectricLocomotive).ElectricPowerSupply.CircuitBreaker.State.ToString())))
+                else if ((trainCar is MSTSWagon wagon && wagon.PowerSupply != null && BatterySwitchOn != wagon.PowerSupply.BatterySwitch.On)
+                    || (isElectricDieselLocomotive && PowerSupplyStatus != (trainCar as MSTSLocomotive).LocomotivePowerSupply.GetPowerStatus()))
                 {
                     Layout();
                     UpdateWindowSize();
@@ -1149,10 +1148,10 @@ void buttonTogglePower_Click(Control arg1, Point arg2)
                 if ((CurrentCar is MSTSElectricLocomotive) || (CurrentCar is MSTSDieselLocomotive))
                 {
                     MSTSLocomotive locomotive = CurrentCar as MSTSLocomotive;
+                    bool powerOn = locomotive.LocomotivePowerSupply.GetPowerStatus() == PowerSupplyState.PowerOff;
 
-                    new PowerCommand(Viewer.Log, locomotive, !locomotive.LocomotivePowerSupply.MainPowerSupplyOn);
-                    var mainPowerSupplyOn = locomotive.LocomotivePowerSupply.MainPowerSupplyOn;
-                    if (mainPowerSupplyOn)
+                    new PowerCommand(Viewer.Log, locomotive, powerOn);
+                    if (!powerOn)
                         Viewer.Simulator.Confirmer.Information(Viewer.Catalog.GetString("Power OFF command sent"));
                     else
                         Viewer.Simulator.Confirmer.Information(Viewer.Catalog.GetString("Power ON command sent"));
@@ -1165,36 +1164,9 @@ void buttonTogglePower_Click(Control arg1, Point arg2)
             }
             public Texture2D locomotiveStatusPower(int CarPosition)
             {
-                string locomotiveStatus = CurrentCar.GetStatus();
-                foreach (string data in locomotiveStatus.Split('\n').Where((string d) => !string.IsNullOrWhiteSpace(d)))
-                {
-                    string[] parts = data.Split(new string[] { " = " }, 2, StringSplitOptions.None);
-                    string keyPart = parts[0];
-                    string valuePart = parts?[1];
-                    if (keyPart.Contains(Viewer.Catalog.GetParticularString("DieselEngine","Engine")))
-                    {
-                        TrainCarViewer.PowerSupplyStatus = locomotiveStatus;
-                        Texture = valuePart.Contains(Viewer.Catalog.GetParticularString("DieselEngine", "Running")) ? PowerOn
-                           : valuePart.Contains(Viewer.Catalog.GetParticularString("DieselEngine", "Stopped")) ? PowerOff
-                           : PowerChanging;
-                        break;
-                    }
-
-                    MSTSElectricLocomotive locomotive = CurrentCar as MSTSElectricLocomotive;
-                    switch (locomotive.ElectricPowerSupply.CircuitBreaker.State)
-                    {
-                        case ORTS.Scripting.Api.CircuitBreakerState.Closed:
-                            Texture = PowerOn;
-                            break;
-                        case ORTS.Scripting.Api.CircuitBreakerState.Closing:
-                            Texture = PowerChanging;
-                            break;
-                        case ORTS.Scripting.Api.CircuitBreakerState.Open:
-                            Texture = PowerOff;
-                            break;
-                    }
-                    TrainCarViewer.CircuitBreakerState = locomotive.ElectricPowerSupply.CircuitBreaker.State.ToString();
-                }
+                var powerStatus = (CurrentCar as MSTSLocomotive).LocomotivePowerSupply.GetPowerStatus();
+                TrainCarViewer.PowerSupplyStatus = powerStatus;
+                Texture = powerStatus == PowerSupplyState.PowerOn ? PowerOn : powerStatus == PowerSupplyState.PowerOff ? PowerOff : PowerChanging;
                 return Texture;
             }
         }
@@ -1254,18 +1226,11 @@ void ToggleBatterySwitch_Click(Control arg1, Point arg2)
             }
             public Texture2D locomotiveStatusBattery(int CarPosition)
             {
-                string locomotiveStatus = CurrentCar.GetStatus();
-                foreach (string data in locomotiveStatus.Split('\n').Where((string d) => !string.IsNullOrWhiteSpace(d)))
+                if (CurrentCar is MSTSWagon wagon && wagon.PowerSupply != null)
                 {
-                    string[] parts = data.Split(new string[] { " = " }, 2, StringSplitOptions.None);
-                    string keyPart = parts[0];
-                    string valuePart = parts?[1];
-                    if (keyPart.Contains(Viewer.Catalog.GetString("Battery switch")))
-                    {
-                        TrainCarViewer.BatteryStatus = locomotiveStatus;
-                        Texture = valuePart.Contains(Viewer.Catalog.GetString("On")) ? BattOn32 : BattOff32;
-                        break;
-                    }
+                    bool on = wagon.PowerSupply.BatterySwitch.On;
+                    TrainCarViewer.BatterySwitchOn = on;
+                    Texture = on ? BattOn32 : BattOff32;
                 }
                 return Texture;
             }
diff --git a/Source/RunActivity/Viewer3D/Popups/TrainCarOperationsWindow.cs b/Source/RunActivity/Viewer3D/Popups/TrainCarOperationsWindow.cs

Original file line number	Diff line number	Diff line change
`@@ -49,6 +49,7 @@ protected override void SetCurrentMainPowerSupplyState(PowerSupplyState state) {`
`49`	`49`	`protected override void SetCurrentAuxiliaryPowerSupplyState(PowerSupplyState state) {}`
`50`	`50`	`protected override void SetCurrentElectricTrainSupplyState(PowerSupplyState state) {}`
`51`	`51`	`protected override void SetCurrentDynamicBrakeAvailability(bool avail) {}`
	`52`	`+ public override PowerSupplyState GetPowerStatus() => PowerSupplyState.Unavailable;`
`52`	`53`	`public void SignalEventToControlActiveLocomotive(PowerSupplyEvent evt)`
`53`	`54`	`{`
`54`	`55`	`ControlActiveLocomotive?.LocomotivePowerSupply.HandleEvent(evt);`
Original file line number	Diff line number	Diff line change
`@@ -266,6 +266,8 @@ public virtual void Update(float elapsedClockSeconds)`
`266`	`266`	`ElectricTrainSupplySwitch.Update(elapsedClockSeconds);`
`267`	`267`	`}`
`268`	`268`
	`269`	`+ public PowerSupplyState GetPowerStatus() => AbstractScript?.GetPowerStatus() ?? PowerSupplyState.Unavailable;`
	`270`	`+`
`269`	`271`	`public void HandleEvent(PowerSupplyEvent evt)`
`270`	`272`	`{`
`271`	`273`	`AbstractScript?.HandleEvent(evt);`