Re-add destroying for unloading, other clean up:

* Cleaned some module terminate code
* Fixed "reload all" issues

modules/client_background/background.lua

@@ -25,7 +25,6 @@ function terminate()
 
   g_effects.cancelFade(background:getChildById('clientVersionLabel'))
   background:destroy()
-  background = nil
 
   Background = nil
 end

modules/client_entergame/entergame.lua

@@ -136,11 +136,17 @@ function EnterGame.terminate()
   removeEvent(autoLoginEvent)
   enterGame:destroy()
   enterGame = nil
+  enterGameButton:destroy()
+  enterGameButton = nil
   protocolBox = nil
   if motdWindow then
     motdWindow:destroy()
     motdWindow = nil
   end
+  if motdButton then
+    motdButton:destroy()
+    motdButton = nil
+  end
   if loadBox then
     loadBox:destroy()
     loadBox = nil

modules/client_modulemanager/modulemanager.lua

@@ -22,7 +22,7 @@ end
 
 function terminate()
   moduleManagerWindow:destroy()
-  moduleManagerWindow = nil
+  moduleManagerButton:destroy()
   moduleList = nil
 end
 

modules/client_options/options.lua

@@ -118,7 +118,8 @@ function terminate()
   --g_keyboard.unbindKeyDown('Ctrl+D')
   g_keyboard.unbindKeyDown('Ctrl+Shift+F')
   optionsWindow:destroy()
-  optionsWindow = nil
+  optionsButton:destroy()
+  audioButton:destroy()
   optionsTabBar = nil
   generalPanel = nil
   consolePanel = nil

modules/client_terminal/terminal.lua

@@ -150,10 +150,8 @@ function terminate()
   g_settings.setList('terminal-history', commandHistory)
   g_keyboard.unbindKeyDown('Ctrl+T')
   g_logger.setOnLog(nil)
-  commandTextEdit = nil
-  terminalBuffer = nil
   terminalWindow:destroy()
-  terminalWindow = nil
+  terminalButton:destroy()
   commandEnv = nil
 end
 

modules/client_topmenu/topmenu.lua

@@ -57,13 +57,7 @@ function terminate()
   disconnect(g_game, { onGameStart = showGameButtons,
                        onGameEnd = hideGameButtons })
 
-  leftButtonsPanel = nil
-  rightButtonsPanel = nil
-  leftGameButtonsPanel = nil
-  rightGameButtonsPanel = nil
-
   topMenu:destroy()
-  topMenu = nil
 end
 
 function addLeftButton(id, description, icon, callback, front)

modules/game_combatcontrols/combatcontrols.lua

@@ -51,6 +51,7 @@ function terminate()
 
   fightModeRadioGroup:destroy()
   combatControlsWindow:destroy()
+  combatControlsButton:destroy()
 
   disconnect(g_game, {
     onGameStart = online,

modules/game_console/console.lua

@@ -164,25 +164,17 @@ function terminate()
   
   if channelsWindow then
     channelsWindow:destroy()
-    channelsWindow = nil
   end
   
   if ignoreWindow then
     ignoreWindow:destroy()
-    ignoreWindow = nil
   end
 
   if violationWindow then
     violationWindow:destroy()
-    violationWindow = nil
   end
   
   consolePanel:destroy()
-  consolePanel = nil
-  consoleTextEdit = nil
-  consoleContentPanel = nil
-  consoleTabBar = nil
-
   ownPrivateName = nil
 
   Console = nil

modules/game_cooldown/cooldown.lua

@@ -36,6 +36,7 @@ function terminate()
                        onSpellCooldown = onSpellCooldown })
 
   cooldownWindow:destroy()
+  cooldownButton:destroy()
 end
 
 function onMiniWindowClose()

modules/game_healthinfo/healthinfo.lua

@@ -70,19 +70,7 @@ function terminate()
   disconnect(g_game, { onGameEnd = offline })
 
   healthInfoWindow:destroy()
-  healthInfoWindow = nil
+  healthInfoButton:destroy()
-  healthInfoButton = nil
-
-  healthBar = nil
-  manaBar = nil
-  soulBar = nil
-
-  healthLabel = nil
-  manaLabel = nil
-  soulLabel = nil
-  capLabel = nil
-
-  HealthInfo = nil
 end
 
 function toggle()

modules/game_hotkeys/hotkeys_manager.lua

@@ -88,6 +88,7 @@ function terminate()
   unload()
 
   hotkeysWindow:destroy()
+  hotkeysButton:destroy()
   mouseGrabberWidget:destroy()
 end
 

modules/game_interface/gameinterface.lua

@@ -123,11 +123,11 @@ function terminate()
 
   disconnect(gameLeftPanel, { onVisibilityChange = onLeftPanelVisibilityChange })
 
+  logoutButton:destroy()
   gameRootPanel:destroy()
 end
 
 function onGameStart()
-  logoutButton:setTooltip(tr('Logout'))
   show()
 
   -- open tibia has delay in auto walking
@@ -139,7 +139,6 @@ function onGameStart()
 end
 
 function onGameEnd()
-  logoutButton:setTooltip(tr('Exit'))
   setupViewMode(0)
   hide()
 end
@@ -151,10 +150,13 @@ function show()
   gameRootPanel:focus()
   gameMapPanel:followCreature(g_game.getLocalPlayer())
   updateStretchShrink()
+  logoutButton:setTooltip(tr('Logout'))
 end
 
 function hide()
   disconnect(g_app, { onClose = tryExit })
+  logoutButton:setTooltip(tr('Exit'))
+  
   if logoutWindow then
     logoutWindow:destroy()
     logoutWindow = nil

modules/game_inventory/inventory.lua

@@ -39,6 +39,7 @@ function terminate()
   g_keyboard.unbindKeyDown('Ctrl+I')
 
   inventoryWindow:destroy()
+  inventoryButton:destroy()
 end
 
 function refresh()

modules/game_market/market.lua

@@ -737,9 +737,7 @@ function terminate()
   disconnect(g_game, { onGameEnd = Market.reset })
   disconnect(g_game, { onGameEnd = Market.close })
 
-  if marketWindow then
   marketWindow:destroy()
-  end
 
   Market = nil
 end

modules/game_minimap/minimap.lua

@@ -49,6 +49,7 @@ function terminate()
   g_keyboard.unbindKeyDown('Ctrl+M')
 
   minimapWindow:destroy()
+  minimapButton:destroy()
 end
 
 function toggle()

modules/game_playerdeath/playerdeath.lua

@@ -17,7 +17,6 @@ end
 function reset()
   if deathWindow then
     deathWindow:destroy()
-    deathWindow = nil
   end
 end
 

modules/game_playermount/playermount.lua

@@ -11,6 +11,7 @@ function terminate()
     onGameStart = online,
     onGameEnd = offline
   })
+  offline()
 end
 
 function online()

modules/game_questlog/questlog.lua

@@ -18,17 +18,16 @@ function terminate()
                        onGameEnd = destroyWindows})
 
   destroyWindows()
+  questLogButton:destroy()
 end
 
 function destroyWindows()
   if questLogWindow then
     questLogWindow:destroy()
-    questLogWindow = nil
   end
 
   if questLineWindow then
     questLineWindow:destroy()
-    questLineWindow = nil
   end
 end
 

modules/game_skills/skills.lua

@@ -61,6 +61,7 @@ function terminate()
 
   g_keyboard.unbindKeyDown('Ctrl+S')
   skillsWindow:destroy()
+  skillsButton:destroy()
 end
 
 function expForLevel(level)
@@ -231,6 +232,8 @@ end
 
 function checkExpSpeed()
   local player = g_game.getLocalPlayer()
+  if not player then return end
+  
   local currentExp = player:getExperience()
   local currentTime = g_clock.seconds()
   if player.lastExps ~= nil then

modules/game_spelllist/spelllist.lua

@@ -173,41 +173,11 @@ function terminate()
                         end })   
 
   spelllistWindow:destroy()
-  spelllistWindow       = nil
+  spelllistButton:destroy()
   
   vocationRadioGroup:destroy()
-  vocationRadioGroup    = nil
   groupRadioGroup:destroy()
-  groupRadioGroup       = nil
   premiumRadioGroup:destroy()
-  premiumRadioGroup     = nil
-  
-  spellList             = nil
-  nameValueLabel        = nil
-  formulaValueLabel     = nil
-  vocationValueLabel    = nil
-  groupValueLabel       = nil
-  typeValueLabel        = nil
-  cooldownValueLabel    = nil
-  levelValueLabel       = nil
-  manaValueLabel        = nil
-  premiumValueLabel     = nil
-  descriptionValueLabel = nil
-
-  vocationBoxAny        = nil
-  vocationBoxSorcerer   = nil
-  vocationBoxDruid      = nil
-  vocationBoxPaladin    = nil
-  vocationBoxKnight     = nil
-
-  groupBoxAny           = nil
-  groupBoxAttack        = nil
-  groupBoxHealing       = nil
-  groupBoxSupport       = nil
-
-  premiumBoxAny         = nil
-  premiumBoxNo          = nil
-  premiumBoxYes         = nil
 end
 
 function initialiseSpelllist()

modules/game_viplist/viplist.lua

@@ -35,7 +35,6 @@ function refresh()
   end
 
   vipWindow:setContentMinimumHeight(38)
-  --vipWindow:setContentMaximumHeight(256)
 end
 
 function clear()