From 0e2e83ef56f2c69338c3a1497f8533f2667d1261 Mon Sep 17 00:00:00 2001
From: Matchu <matchu1993@gmail.com>
Date: Wed, 20 Feb 2013 00:52:37 -0600
Subject: [PATCH] bring needed items queries up-to-date with new species
 support ID syntax

---
 app/models/pet_type.rb | 42 ++++++++++++++++++++++--------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/app/models/pet_type.rb b/app/models/pet_type.rb
index 89edd9f5..9df9de77 100644
--- a/app/models/pet_type.rb
+++ b/app/models/pet_type.rb
@@ -89,26 +89,28 @@ class PetType < ActiveRecord::Base
   end
 
   def needed_items
-    items = Item.arel_table
-    species_matchers = [
-      "#{species_id},%",
-      "%,#{species_id},%",
-      "%,#{species_id}"
-    ]
-    species_condition = nil
-    species_matchers.each do |matcher|
-      condition = items[:species_support_ids].matches(matcher)
-      if species_condition
-        species_condition = species_condition.or(condition)
-      else
-        species_condition = condition
-      end
-    end
-    unneeded_item_ids = Item.select(items[:id]).
-      joins(:parent_swf_asset_relationships => :swf_asset).
-      where(SwfAsset.arel_table[:body_id].in([0, self.body_id])).map(&:id)
-    Item.where(items[:id].not_in(unneeded_item_ids)).
-      where(species_condition)
+    # If I need this item on a pet type, that means that we've already seen it
+    # and it's body-specific. So, there's a body-specific asset for the item,
+    # but no asset that fits this pet type.
+    i = Item.arel_table
+    psa = ParentSwfAssetRelationship.arel_table
+    sa = SwfAsset.arel_table
+
+    # Close, but no cigar: if we just check for the presence of *one* other
+    # body-specific asset, it'll also include single-species items for other
+    # species. We should check for more than one... but I'm not sure how to
+    # do that in Arel...
+    Item.where('(' + ParentSwfAssetRelationship.select('count(*)').joins(:swf_asset).
+               where(
+                 psa[:parent_id].eq(i[:id]).and(
+                 psa[:parent_type].eq('Item').and(
+                 sa[:body_id].not_eq(self.body_id)))
+               ).to_sql + ') > 1').
+         where(ParentSwfAssetRelationship.joins(:swf_asset).where(
+                 psa[:parent_id].eq(i[:id]).and(
+                 psa[:parent_type].eq('Item').and(
+                 sa[:body_id].in([self.body_id, 0])))
+               ).exists.not)
   end
 
   def add_pet_state_from_biology!(biology)