DESIGNATE Class Fix

DESIGNATE
- GetRootMenu() function seems obsolete and caused a crash in DESIGNATE class. Needs to be replaced by GetMenu() Function. This bug only appeared when a mission is given in DESIGNATE:New(...)

Moose Development/Moose/Core/Menu.lua

@@ -89,12 +89,15 @@ function MENU_INDEX:PrepareCoalition( CoalitionSide )
     self.Coalition[CoalitionSide].Menus = self.Coalition[CoalitionSide].Menus or {}
 end
 
-
+---
+-- @param Wrapper.Group#GROUP Group
 function MENU_INDEX:PrepareGroup( Group )
+  if Group and Group:IsAlive()  then
     local GroupName = Group:GetName()
     self.Group[GroupName] = self.Group[GroupName] or {}
     self.Group[GroupName].Menus = self.Group[GroupName].Menus or {}
   end
+end
 
 
 
@@ -133,10 +136,12 @@ end
 
 
 function MENU_INDEX:HasGroupMenu( Group, Path )
-
+  if Group and Group:IsAlive() then
     local MenuGroupName = Group:GetName()
     return self.Group[MenuGroupName].Menus[Path]
   end
+  return nil
+end
 
 function MENU_INDEX:SetGroupMenu( Group, Path, Menu )
 

Moose Development/Moose/Core/Set.lua

@@ -798,6 +798,7 @@ function SET_GROUP:GetAliveSet()
   
   -- Clean the Set before returning with only the alive Groups.
   for GroupName, GroupObject in pairs( self.Set ) do
+    local GroupObject=GroupObject --Wrapper.Group#GROUP
     if GroupObject then
       if GroupObject:IsAlive() then
         AliveSet:Add( GroupName, GroupObject )

Moose Development/Moose/Functional/Designate.lua

@@ -175,7 +175,7 @@ do -- DESIGNATE
   -- Smoke will fire for 5 minutes.
   -- Each available recce within range will smoke a target.
   -- Smoking can be requested while lasing targets.
-  -- Smoke will appear “around” the targets, because of accuracy limitations.
+  -- Smoke will appear "around" the targets, because of accuracy limitations.
   -- 
   -- 
   -- Have FUN!
@@ -952,7 +952,8 @@ do -- DESIGNATE
         local MissionMenu = nil
         
         if self.Mission then
-          MissionMenu = self.Mission:GetRootMenu( AttackGroup )
+          --MissionMenu = self.Mission:GetRootMenu( AttackGroup )
+          MissionMenu = self.Mission:GetMenu( AttackGroup )
         end
         
         local MenuTime = timer.getTime()

Moose Development/Moose/Tasking/CommandCenter.lua

@@ -426,6 +426,7 @@ end
 
 --- Gets the commandcenter menu structure governed by the HQ command center.
 -- @param #COMMANDCENTER self
+-- @param Wrapper.Group#Group TaskGroup Task Group.
 -- @return Core.Menu#MENU_COALITION
 function COMMANDCENTER:GetMenu( TaskGroup )
 

Moose Development/Moose/Tasking/Mission.lua

@@ -611,13 +611,14 @@ function MISSION:RemoveTaskMenu( Task )
 end
 
 
---- Gets the root mission menu for the TaskGroup.
+--- Gets the root mission menu for the TaskGroup. Obsolete?! Originally no reference to TaskGroup parameter!
 -- @param #MISSION self
+-- @param Wrapper.Group#GROUP TaskGroup Task group.
 -- @return Core.Menu#MENU_COALITION self
 function MISSION:GetRootMenu( TaskGroup ) -- R2.2
 
   local CommandCenter = self:GetCommandCenter()
-  local CommandCenterMenu = CommandCenter:GetMenu()
+  local CommandCenterMenu = CommandCenter:GetMenu( TaskGroup )
 
   local MissionName = self:GetText()
   --local MissionMenu = CommandCenterMenu:GetMenu( MissionName )
@@ -629,6 +630,7 @@ end
 
 --- Gets the mission menu for the TaskGroup.
 -- @param #MISSION self
+-- @param Wrapper.Group#GROUP TaskGroup Task group.
 -- @return Core.Menu#MENU_COALITION self
 function MISSION:GetMenu( TaskGroup ) -- R2.1 -- Changed Menu Structure