changeset 2:abd59cfa1e70

Wowhead has better type detection, now tells you if no results were found.
author Atul Varma <varmaa@toolness.com>
date Fri, 20 Mar 2009 00:12:42 -0700
parents b498a74c7410
children 7ac908bdb3d3
files wowhead/results.js
diffstat 1 files changed, 11 insertions(+), 7 deletions(-) [+]
line wrap: on
line diff
--- a/wowhead/results.js	Fri Mar 20 00:02:10 2009 -0700
+++ b/wowhead/results.js	Fri Mar 20 00:12:42 2009 -0700
@@ -1,18 +1,22 @@
-TYPE_MAP = {
-  3: "item",
-  5: "quest",
-  6: "spell",
-  10: "achievement"
-};
+var typeRegexp = /.*\(([A-Za-z]+)\).*/;
 
 function showResults(data) {
+  var query = data[0];
   var names = data[1];
+  if (!query) {
+    $("#results").html("Searches Wowhead.");
+    return;
+  }
+  if (!names.length) {
+    $("#results").html("No results found.");
+    return;
+  }
   var meta = data[7];
   var list = $("<ul></ul>");
   for (var i = 0; i < names.length; i++) {
     var name = names[i];
     var item_meta = meta[i];
-    var type = TYPE_MAP[item_meta[0]];
+    var type = name.match(typeRegexp)[1].toLowerCase();
     if (type == "spell" && item_meta[3])
       name = name.replace("(Spell)", "(" + item_meta[3] + ")");
     var id = item_meta[1];