From c35965ea9a083cec65d9db075d4c0a6dac61ed90 Mon Sep 17 00:00:00 2001
From: Michael Spinelli <michael.spinelli@us.army.mil>
Date: Fri, 16 Mar 2018 16:20:01 -0400
Subject: [PATCH] Fix for: If 2525B is not loaded, getBasicSymbolID errors out
 when called with no symStd parameter #29

---
 .../c2sd/renderer/utilities/SymbolDefTable.js |  4 +--
 .../renderer/utilities/SymbolUtilities.js     | 31 ++++++++-----------
 .../c2sd/renderer/utilities/UnitDefTable.js   |  4 +--
 3 files changed, 17 insertions(+), 22 deletions(-)

diff --git a/web/js/src/armyc2/c2sd/renderer/utilities/SymbolDefTable.js b/web/js/src/armyc2/c2sd/renderer/utilities/SymbolDefTable.js
index 8ec27db2..07e63192 100644
--- a/web/js/src/armyc2/c2sd/renderer/utilities/SymbolDefTable.js
+++ b/web/js/src/armyc2/c2sd/renderer/utilities/SymbolDefTable.js
@@ -232,7 +232,7 @@ armyc2.c2sd.renderer.utilities.SymbolDefTable = (function () {
             else
                 symbolMap = symbolMapC;
             
-            if(symbolMap[symbolID] !== undefined)
+            if(symbolMap && symbolMap[symbolID] !== undefined)
             {
                 return symbolMap[symbolID];
             }
@@ -259,7 +259,7 @@ armyc2.c2sd.renderer.utilities.SymbolDefTable = (function () {
             else
                 symbolMap = symbolMapC;
             
-            if(symbolMap[symbolID] !== undefined)
+            if(symbolMap && symbolMap[symbolID] !== undefined)
             {
                 return true;
             }
diff --git a/web/js/src/armyc2/c2sd/renderer/utilities/SymbolUtilities.js b/web/js/src/armyc2/c2sd/renderer/utilities/SymbolUtilities.js
index dd2327fb..e2d1dd82 100644
--- a/web/js/src/armyc2/c2sd/renderer/utilities/SymbolUtilities.js
+++ b/web/js/src/armyc2/c2sd/renderer/utilities/SymbolUtilities.js
@@ -28,36 +28,31 @@ armyc2.c2sd.renderer.utilities.SymbolUtilities.rendererSettings = armyc2.c2sd.re
                 {
                     basic = scheme + '*' + basic.charAt(2) + '*' + basic.substring(4, 10) + "*****";
 
-                    var i = symStd || 0;
-                    for(; i < 2; i++)
+                    var std = armyc2.c2sd.renderer.utilities.RendererSettings.getSymbologyStandard();
+                    if(symStd !== undefined)
+                        std = symStd;
+
+                    var has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(basic, std);
+                    var temp = null;
+                    if(!has)
                     {
-                        var has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(basic, i);
-                        var temp = null;
+                        temp = basic.substr(0,10) + "H****";
+                        has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(temp, std);
                         if(has)
                         {
-                            i=3;
+                            basic = temp;
                         }
                         else
                         {
-                            temp = basic.substr(0,10) + "H****";
-                            has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(temp, i);
+                            temp = basic.substr(0,10) + "MO***";
+                            has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(temp, std);
                             if(has)
                             {
                                 basic = temp;
-                                i=3;
-                            }
-                            else
-                            {
-                                temp = basic.substr(0,10) + "MO***";
-                                has = armyc2.c2sd.renderer.utilities.UnitDefTable.hasUnitDef(temp, i);
-                                if(has)
-                                {
-                                    basic = temp;
-                                    i=3;
-                                }
                             }
                         }
                     }
+                    
                 }
                 else if (scheme === 'G') //tactical graphic
                 {
diff --git a/web/js/src/armyc2/c2sd/renderer/utilities/UnitDefTable.js b/web/js/src/armyc2/c2sd/renderer/utilities/UnitDefTable.js
index f455492e..dd8cbee2 100644
--- a/web/js/src/armyc2/c2sd/renderer/utilities/UnitDefTable.js
+++ b/web/js/src/armyc2/c2sd/renderer/utilities/UnitDefTable.js
@@ -140,7 +140,7 @@ armyc2.c2sd.renderer.utilities.UnitDefTable = (function () {
             else
                 symbolMap = symbolMapC;
             
-            if(symbolMap[symbolID] !== undefined)
+            if(symbolMap && symbolMap[symbolID] !== undefined)
             {
                 return symbolMap[symbolID];
             }
@@ -167,7 +167,7 @@ armyc2.c2sd.renderer.utilities.UnitDefTable = (function () {
             else
                 symbolMap = symbolMapC;
             
-            if(symbolMap[symbolID] !== undefined)
+            if(symbolMap && symbolMap[symbolID] !== undefined)
             {
                 return true;
             }