fix: condition explicitly uses attribute values and not public_send

Using public_send caused an issue when attribute names shadowed method
names. Closes #280

Author: flavorjones

lib/active_hash/condition.rb

@@ -19,7 +19,7 @@ def matches?(record)
       expectation_method = inverted ? :any? : :all?
 
       constraints.send(expectation_method) do |attribute, expected|
-        value = record.public_send(attribute)
+        value = record[attribute]
 
         matches_value?(value, expected)
       end
@@ -41,4 +41,4 @@ def matches_value?(value, comparison)
   def normalize(value)
     value.respond_to?(:to_s) ? value.to_s : value
   end
-end
+end

spec/active_hash/relation_spec.rb

@@ -50,4 +50,24 @@
       expect(array.size).to eq(2)
     end
   end
+
+  describe "colliding methods" do
+    it "should handle attributes named after existing methods" do
+      klass = Class.new(ActiveHash::Base) do
+        self.data = [
+          {
+            id: 1,
+            name: "Aaa",
+            display: true,
+          },
+          {
+            id: 2,
+            name: "Bbb",
+            display: false,
+          },
+        ]
+      end
+      expect(klass.where(display: true).length).to eq(1)
+    end
+  end
 end