New assertion to diagnose issue 323. This assertion checks to see if GetItemType is going to return nil or not. If the assertion fails, nil is being returned and the item link is bad.

QuestHelper/collect_util.lua

@@ -27,9 +27,11 @@ end
 
 local function GetItemType(link, vague)
   QuestHelper:Assert(link, "Item did not have a link! Say WHAT?")
-  return tonumber(string.match(link,
+  local result = tonumber(string.match(link,
     (vague and "" or "^") .. "|cff%x%x%x%x%x%x|Hitem:(%d+):[%d:-]+|h%[[^%]]*%]|h|r".. (vague and "" or "$") 
   ))
+  QuestHelper:Assert(result, "Item does not have a type ('" .. link .. "')")
+  return result
 end
   
 local function GetQuestType(link)