hack: allow times to be unset

Author: emersion

src/app/data-structures/technical.data.structures.ts

@@ -98,7 +98,7 @@ export interface TimeFormatter {
  * Represents the time data in TrainrunSectionDto.
  */
 export interface TimeLockDto {
-  time: number; // minutes [0..60]
+  time: number | null; // minutes [0..60]
   consecutiveTime: number; // automatically updated after any data changes in the application
   lock: boolean; // used to stop the time propagation (forward/backward)
   warning: WarningDto; // warning - if business logic detects an issue -> set human-readable warning

src/app/models/trainrunsection.model.ts

@@ -175,6 +175,9 @@ export class TrainrunSection {
   }
 
   private static formatDisplayText(time: TimeLockDto, offset: number): string {
+    if (time.time === null) {
+      return "?";
+    }
     if (!time?.timeFormatter?.stylePattern) {
       return undefined;
     }

src/app/perlenkette/perlenkette-section/perlenkette-section.component.ts

@@ -610,6 +610,9 @@ export class PerlenketteSectionComponent
   }
 
   getTravelTime() {
+    if (this.trainrunSectionTimesService.getTimeStructure().travelTime === null) {
+      return null;
+    }
     if (
       TrainrunSectionsView.getNode(this.trainrunSection, true).isNonStop(
         this.trainrunSection,
@@ -921,6 +924,9 @@ export class PerlenketteSectionComponent
   }
 
   roundTime(time: number) {
+    if (time === null) {
+      return time;
+    }
     return MathUtils.round(time, this.filterService.getTimeDisplayPrecision());
   }
 

src/app/services/data/trainrun-section-times.service.ts

@@ -491,22 +491,22 @@ export class TrainrunSectionTimesService {
       this.initialLeftAndRightElement ===
       LeftAndRightElement.LeftRightTrainrunName
     ) {
-      this.timeStructure.leftDepartureTime =
+      this.timeStructure.leftDepartureTime = this.timeStructure.leftDepartureTime === null ? null :
         (this.timeStructure.leftDepartureTime + this.offset) % 60;
-      this.timeStructure.rightArrivalTime =
+      this.timeStructure.rightArrivalTime = this.timeStructure.rightArrivalTime === null ? null :
         (this.timeStructure.rightArrivalTime + this.offset) % 60;
-      this.timeStructure.leftArrivalTime =
+      this.timeStructure.leftArrivalTime = this.timeStructure.leftArrivalTime === null ? null :
         (maxMinutes + this.timeStructure.leftArrivalTime - this.offset) % 60;
-      this.timeStructure.rightDepartureTime =
+      this.timeStructure.rightDepartureTime = this.timeStructure.rightDepartureTime === null ? null :
         (maxMinutes + this.timeStructure.rightDepartureTime - this.offset) % 60;
     } else {
-      this.timeStructure.leftDepartureTime =
+      this.timeStructure.leftDepartureTime = this.timeStructure.leftDepartureTime === null ? null :
         (maxMinutes + this.timeStructure.leftDepartureTime - this.offset) % 60;
-      this.timeStructure.rightArrivalTime =
+      this.timeStructure.rightArrivalTime = this.timeStructure.rightArrivalTime === null ? null :
         (maxMinutes + this.timeStructure.rightArrivalTime - this.offset) % 60;
-      this.timeStructure.leftArrivalTime =
+      this.timeStructure.leftArrivalTime = this.timeStructure.leftArrivalTime === null ? null :
         (this.timeStructure.leftArrivalTime + this.offset) % 60;
-      this.timeStructure.rightDepartureTime =
+      this.timeStructure.rightDepartureTime = this.timeStructure.rightDepartureTime === null ? null :
         (this.timeStructure.rightDepartureTime + this.offset) % 60;
     }
     this.offsetTransformationActive = true;
@@ -569,25 +569,26 @@ export class TrainrunSectionTimesService {
       this.timeStructure.travelTime,
       timeDisplayPrecision,
     );
+    // Populate travel time here, otherwise it'll be up to
+    // setTimeStructureToTrainrunSections() and it may overwrite values entered
+    // by the user
+    const minTravelTime = 1.0 / Math.pow(10, this.filterService.getTimeDisplayPrecision());
+    if (this.timeStructure.travelTime < 0.1) {
+      this.timeStructure.travelTime = 0.1;
+    }
   }
 
   private fixAllTimesPrecision() {
     const timeDisplayPrecision = 1000;
-    this.timeStructure.leftArrivalTime =
-      Math.round(this.timeStructure.leftArrivalTime * timeDisplayPrecision) /
-      timeDisplayPrecision;
-    this.timeStructure.leftDepartureTime =
-      Math.round(this.timeStructure.leftDepartureTime * timeDisplayPrecision) /
-      timeDisplayPrecision;
-    this.timeStructure.rightArrivalTime =
-      Math.round(this.timeStructure.rightArrivalTime * timeDisplayPrecision) /
-      timeDisplayPrecision;
-    this.timeStructure.rightDepartureTime =
-      Math.round(this.timeStructure.rightDepartureTime * timeDisplayPrecision) /
-      timeDisplayPrecision;
-    this.timeStructure.travelTime =
-      Math.round(this.timeStructure.travelTime * timeDisplayPrecision) /
-      timeDisplayPrecision;
+    const fixPrecision = (time) => {
+      if (time === null) return null;
+      return Math.round(time * timeDisplayPrecision) / timeDisplayPrecision;
+    };
+    this.timeStructure.leftArrivalTime = fixPrecision(this.timeStructure.leftArrivalTime);
+    this.timeStructure.leftDepartureTime = fixPrecision(this.timeStructure.leftDepartureTime);
+    this.timeStructure.rightArrivalTime = fixPrecision(this.timeStructure.rightArrivalTime);
+    this.timeStructure.rightDepartureTime = fixPrecision(this.timeStructure.rightDepartureTime);
+    this.timeStructure.travelTime = fixPrecision(this.timeStructure.travelTime);
   }
 
   private updateTrainrunSectionTime() {

src/app/services/data/trainrunsection.service.ts

@@ -934,7 +934,7 @@ export class TrainrunSectionService implements OnDestroy {
 
     const oldTotalTravelTime =
       this.trainrunService.getCumulativeTravelTime(trainrunSection);
-    const travelTimeFactor = newTotalTravelTime / oldTotalTravelTime;
+    const travelTimeFactor = newTotalTravelTime / (oldTotalTravelTime || 1);
 
     // prepare data structure for the first trainrunsection
     const bothLastNonStopNodes =

src/app/services/util/trainrunsection.helper.ts

@@ -303,7 +303,7 @@ export class TrainrunsectionHelper {
       leftArrivalTime: lastLeftNode.getArrivalTime(leftTrainrunSection),
       rightDepartureTime: lastRightNode.getDepartureTime(rightTrainrunSection),
       rightArrivalTime: lastRightNode.getArrivalTime(rightTrainrunSection),
-      travelTime: cumulativeTravelTime,
+      travelTime: cumulativeTravelTime || null,
     };
   }