change color reading and execution id setting

Author: birm

apps/port/xml2geo.js

@@ -1,7 +1,7 @@
 parser = new DOMParser();
 
 function generateRandomId(length = 6) {
-  return Math.random().toString(36).substr(2, length);
+  return Array.from({length}, () => Math.floor(Math.random() * 36).toString(36)).join('');
 }
 
 var template = {
@@ -107,7 +107,13 @@ function xml2geo() {
     let annotationLineColor = annotation.getAttribute('LineColor') || '0';
     let annotationId = annotation.getAttribute('Id');
 
-    let hexColor = `#${parseInt(annotationLineColor).toString(16).padStart(6, '0')}`;
+    // Aperio LineColor is a Windows COLORREF (0x00BBGGRR), so channels must be
+    // unpacked individually and reassembled as RRGGBB rather than hex-stringified directly.
+    let rawColor = parseInt(annotationLineColor) || 0;
+    let r = rawColor & 0xFF;
+    let g = (rawColor >> 8) & 0xFF;
+    let b = (rawColor >> 16) & 0xFF;
+    let hexColor = `#${[r, g, b].map((c) => c.toString(16).padStart(2, '0')).join('')}`;
 
     let colorKey = apolloColors ? classifyColor(hexColor) : hexColor;