Merge branch 'release/0.2.4'

Changelog.md

@@ -1,3 +1,6 @@
+# 0.2.4
+Minor update with additional checks on presence of elevation increment flag. Additional logging. 
+
 # 0.2.3
 Fix #3. When a grid is present, snap the projected destination point to the center of the grid. This keeps the distances consistent for different cardinalities. Rounding still occurs with grid measurements such that adjacent squares may have the same measured 3-D distance despite appearing to be closer/further from the origin point. 
 

scripts/segments.js

@@ -7,6 +7,8 @@ import { MODULE_ID, log } from "./module.js";
  * - ending elevation of the segment 
  */
 export function elevationRulerAddProperties(wrapped, ...args) {
+  log(`elevationRulerAddProperties: this`, this);
+
   if(this.segment_num < 0) {
     console.error(`${MODULE_ID}|libRulerAddProperties: this.segment_num is less than zero`, this);
     return;
@@ -30,12 +32,16 @@ UX goals:
     would go from 5 to 55. 
   */
 
-  const elevation_increments = duplicate(this.ruler.getFlag(MODULE_ID, "elevation_increments"));
+  let elevation_increments = duplicate(this.ruler.getFlag(MODULE_ID, "elevation_increments"));
+  if(!elevation_increments || elevation_increments.length < 1) {
+    elevation_increments = [0];
+  } 
+
   log(`${elevation_increments.length} elevation increments for ruler flag.`)
 
-  const destination_elevation_increment = this.ruler.getFlag(MODULE_ID, "destination_elevation_increment");
+  const destination_elevation_increment = this.ruler.getFlag(MODULE_ID, "destination_elevation_increment") || 0;
   elevation_increments.push(destination_elevation_increment);
-  log(`${this.ruler.getFlag(MODULE_ID, "elevation_increments").length} elevation increments for ruler flag.`);
+  log(`${this.ruler.getFlag(MODULE_ID, "elevation_increments").length} destination elevation increments for ruler flag.`);
 
 
   elevation_increments.shift(); //first increment is 0 for the origin waypoint
@@ -46,6 +52,7 @@ UX goals:
     // starting elevation equals the token elevation 
     // if no token, use elevation at the point. 
     starting_elevation = ElevationAtPoint(this.ray.A, this.ruler._getMovementToken(), 0) // 0 starting elevation otherwise
+    log(`Starting elevation using origin ${this.ray.A.x}, ${this.ray.A.y}`, this.ruler._getMovementToken());
 
   } else {
     // starting elevation is the prior segment end elevation
@@ -57,6 +64,7 @@ UX goals:
   const incremental_elevation = toGridDistance(elevation_increments[this.segment_num]);
   const current_point_elevation = ElevationAtPoint(this.ray.B, undefined, starting_elevation); // no starting token; assume we are at the elevation from the last segment
   const ending_elevation = current_point_elevation + incremental_elevation;
+  log(`Current elevation using point ${this.ray.B.x}, ${this.ray.B.y}`);
 
   log(`elevationRulerAddProperties segment ${this.segment_num}: ${starting_elevation}[start]; ${incremental_elevation}[incremental]; ${current_point_elevation}[current point]`);