From d7af6cfd4a0818d77324b21ce79b62cc176e6a3e Mon Sep 17 00:00:00 2001
From: Matchu <matchu1993@gmail.com>
Date: Wed, 2 Apr 2014 20:26:53 -0500
Subject: [PATCH] populate occupies/restricts selects

---
 app/assets/javascripts/outfits/edit.js       | 19 +++++++++++++++----
 app/assets/javascripts/wardrobe.js           | 10 ++++------
 app/controllers/item_zone_sets_controller.rb |  2 +-
 app/models/item/search/query.rb              | 12 +++++++++++-
 app/models/zone.rb                           |  8 ++++++--
 5 files changed, 37 insertions(+), 14 deletions(-)

diff --git a/app/assets/javascripts/outfits/edit.js b/app/assets/javascripts/outfits/edit.js
index 83c9c847..1e98883c 100644
--- a/app/assets/javascripts/outfits/edit.js
+++ b/app/assets/javascripts/outfits/edit.js
@@ -201,10 +201,6 @@ View.Fullscreen = function (wardrobe) {
   $('#preview').data('fit', fit);
 
   win.resize(fit).load(fit);
-  // run fit after search updates select fields
-  function fitSoon() { setTimeout(fit, 0) }
-  wardrobe.item_zone_sets.bind('update', fitSoon);
-  wardrobe.pet_attributes.bind('update', fitSoon);
   fit();
 }
 
@@ -1250,6 +1246,21 @@ View.Search = function (wardrobe) {
     wrapper.toggleClass('advanced');
     fit();
   });
+
+  wardrobe.item_zone_sets.bind('update', function (item_zone_sets) {
+    var selects = $('#advanced-search-occupies, #advanced-search-restricts');
+    console.log("yo", selects, item_zone_sets);
+    var sorted_item_zone_sets = item_zone_sets.slice(0);
+    item_zone_sets.sort(function(a, b) {
+      if (a.label < b.label) return -1;
+      else if (a.label > b.label) return 1;
+      else return 0;
+    });
+    item_zone_sets.forEach(function(set) {
+      $('<option/>', {value: set.plainLabel, text: set.label}).
+        appendTo(selects);
+    });
+  });
 }
 
 View.PrankColorMessage = function(wardrobe) {
diff --git a/app/assets/javascripts/wardrobe.js b/app/assets/javascripts/wardrobe.js
index 6794cbc4..ebc0e040 100644
--- a/app/assets/javascripts/wardrobe.js
+++ b/app/assets/javascripts/wardrobe.js
@@ -263,15 +263,13 @@ function Wardrobe() {
 
   Item.cache = {};
 
-  function ItemZoneSet(name) {
-    this.name = name;
-  }
+  var ItemZoneSet = {};
 
   ItemZoneSet.loadAll = function (success) {
     $.getJSON('/item_zone_sets.json', function (data) {
-      for(var i = 0, l = data.length; i < l; i++) {
-        ItemZoneSet.all.push(new ItemZoneSet(data[i]));
-      }
+      Object.keys(data).forEach(function(key) {
+        ItemZoneSet.all.push({plainLabel: key, label: data[key]});
+      });
       success(ItemZoneSet.all);
     });
   }
diff --git a/app/controllers/item_zone_sets_controller.rb b/app/controllers/item_zone_sets_controller.rb
index 8bbb30fb..6e346810 100644
--- a/app/controllers/item_zone_sets_controller.rb
+++ b/app/controllers/item_zone_sets_controller.rb
@@ -1,5 +1,5 @@
 class ItemZoneSetsController < ApplicationController
   def index
-    render :json => Zone.for_items.all_plain_labels
+    render :json => Zone.for_items.sets
   end
 end
diff --git a/app/models/item/search/query.rb b/app/models/item/search/query.rb
index a6d5965b..e77a86a9 100644
--- a/app/models/item/search/query.rb
+++ b/app/models/item/search/query.rb
@@ -243,8 +243,18 @@ class Item
         filters = params.values.map { |filter_params|
           if filter_params.has_key?(:key)
             key = filter_params[:key].to_sym
+            value = filter_params[:value]
+
+            # Ew, the hackiest of hacks!
+            if key == :occupied_zone_set_name
+              key = :occupied_zone_id
+              value = TEXT_QUERY_RESOURCE_FINDERS[:zone].call(value)
+            elsif key == :restricted_zone_set_name
+              key = :restricted_zone_id
+              value = TEXT_QUERY_RESOURCE_FINDERS[:zone].call(value)
+            end
+
             if FIELD_KEYS.include?(key)
-              value = filter_params[:value]
               # Ugh, this bit feels so hacky :P
               if key == :user_closet_hanger_ownership
                 value = (value == 'true')
diff --git a/app/models/zone.rb b/app/models/zone.rb
index 16c56f17..a8632583 100644
--- a/app/models/zone.rb
+++ b/app/models/zone.rb
@@ -19,8 +19,12 @@ class Zone < ActiveRecord::Base
     @sometimes ? "#{label} sometimes" : label
   end
   
-  def self.all_plain_labels
-    select([:id]).includes(:translations).all.map(&:plain_label).uniq.sort
+  def self.sets
+    {}.tap do |sets|
+      select([:id]).includes(:translations).each do |zone|
+        sets[zone.plain_label] = zone.label
+      end
+    end
   end
   
   def self.plainify_label(label)