active_node 2.3.3 → 2.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54ea207f36a379b3c584d9c37fe2c045da8e8a01
-  data.tar.gz: f484ee30f29b4ae820470f1dd1bc3aa583bf8aaa
+  metadata.gz: 068907cb909d3802f5a52c3b9ec3645bd5ab3246
+  data.tar.gz: 8fd0b5d063bcf6226b3f307804e89f843b3084bf
 SHA512:
-  metadata.gz: 82318d09eb4b0c34346514fc2c956d6e9f622e2d588941f85506b9e2d490788be7d1c0453e006c7ce1cd32f638911b47d1b53bbe113285f641bcced0b992bfe1
-  data.tar.gz: 25acaa945884d9ed45d2ff00e21c03f6dc7a2d062c891445e7b22c15dd140e6564d8b9e2a973dbc3f5af7881894fbde4e2fbaee762bdd8db119da605878fd2c8
+  metadata.gz: 9b949f9fb780fe090b81c6d99c5683fa271c26f01fdb1f521e3de177492b05c35dccf3fb8b5afd96754f8004908c920d2c7f4f3472054b137f76071de208fad6
+  data.tar.gz: 6de7ce32dbb5a33c8e11c2b07dfcb0219757aca660f09b99b81e0a6ce8d7e6e83560600fd919252edc876a70e28529506b5af79030a4d8142700fa8dfcfea4c4

data/lib/active_node/associations/association.rb

@@ -47,12 +47,17 @@ module ActiveNode
 
       # Implements the writer method, e.g. foo.items= for Foo.has_many :items
       def writer(records)
+        records = load_ids(records)
         validate_type(records)
         @dirty = true
         @rel_target = nil
         @target = records
       end
 
+      def load_ids(records)
+        records.map {|r| ActiveNode::Base === r ? r : reflection.klass.find(r)}
+      end
+
       def validate_type(records)
         unless records.all? { |r| r.is_a?(reflection.klass) }
           raise ArgumentError, "#{reflection.name} can only accept object(s) of class #{reflection.klass}"
@@ -64,7 +69,7 @@ module ActiveNode
       # Implements the ids writer method, e.g. foo.item_ids= for Foo.has_many :items
       def ids_writer(ids)
         @rel_target = nil
-        super_writer klass.find(ids.reject(&:blank?).map!(&:to_i))
+        super_writer klass.find(ids.reject(&:blank?))
       end
 
       def reader(*args)

data/lib/active_node/version.rb

@@ -1,3 +1,3 @@
 module ActiveNode
-  VERSION = "2.3.3"
+  VERSION = "2.3.4"
 end

data/spec/functional/associations_spec.rb

@@ -28,6 +28,18 @@ describe ActiveNode::Associations do
       expect(Client.find(client.id).users).to eq([user])
     end
 
+    it "can send id to association" do
+      user = NeoUser.create!(name: 'Heinrich')
+      client = Client.create!(name: 'a', users: [user.id])
+      expect(client.users.to_a).to eq([user])
+    end
+
+    it "can send node to association_id" do
+      user = NeoUser.create!(name: 'Heinrich')
+      client = Client.create!(name: 'a', user_ids: [user])
+      expect(client.users.to_a).to eq([user])
+    end
+
     it "can remove associated objects" do
       user = NeoUser.new(name: 'Heinrich')
       user.save

data/spec/functional/persistence_spec.rb

@@ -62,6 +62,7 @@ describe ActiveNode::Persistence do
     it 'should not record timestamp if not specified' do
       expect(Client.create!(name: 'abc').respond_to?(:created_at)).to be_falsey
     end
+
   end
 
   describe "#destroyed?" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_node
 version: !ruby/object:Gem::Version
-  version: 2.3.3
+  version: 2.3.4
 platform: ruby
 authors:
 - Heinrich Klobuczek
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-18 00:00:00.000000000 Z
+date: 2015-10-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: active_attr