Re-enabling version checks. It is easier to track other errors when people don't mix repository versions with 'productiion' versions, such as repository revisions 133 and 134 with alpha production version 132a.

QuestHelper/main.lua

@@ -333,9 +333,6 @@ QH_Event("ADDON_LOADED", function (addonid)
     
   local uninstallederr = ""
 
-  -- Why do we make the checks?
-  -- QH shouldn't care if an unexpected file is loaded. It is highly unlikely that an unexpected file or version is going to cause an issue.
-  --[[ Commenting out for now.
   for file, version in pairs(QuestHelper_File) do
     if not expected_files[file] then
       local errmsg = "Unexpected QuestHelper file: "..file
@@ -346,16 +343,15 @@ QH_Event("ADDON_LOADED", function (addonid)
       local errmsg = "Wrong version of QuestHelper file: "..file.." (found '"..version.."', should be '"..expected_version.."')"
       DEFAULT_CHAT_FRAME:AddMessage(errmsg)
       uninstallederr = uninstallederr .. "    " .. errmsg .. "\n"
-      if version ~= "Development Version" and expected_version ~= "Development Version" then
+ --[[ Nope, no one is allowed to do this. We've got a number of people doing just that... And they're not developers. Though this check always fails because I don't use "Development Version" to tell QH that it is a dev version.
+     if version ~= "Development Version" and expected_version ~= "Development Version" then
         -- Developers are allowed to mix dev versions with release versions
         file_problem_version = true
       end
+--]]
     end
   end
-  --]]
 
-  -- Again, what's the point? If a file doesn't load, WoW will bitch, piss and moan about something bein nil. If it doesn't load, there's probably an error in it.
-  --[[ Commenting for now.
   for file in pairs(expected_files) do
     if not QuestHelper_File[file] then
       local errmsg = "Missing QuestHelper file: "..file
@@ -364,7 +360,6 @@ QH_Event("ADDON_LOADED", function (addonid)
       if not (expected_version == "Development Version" and file:match("static.*")) then file_problem_version = true end
     end
   end
-  --]]
   
   -- Don't need this table anymore.
   QuestHelper_File = nil