deferring 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 425c85b43ca58b47075b81fd98800f7ca2cb530e
-  data.tar.gz: 17649b71b066711330887c6a1170c2070baca519
+  metadata.gz: 0807e9de82ca4e5593921580b87eaa83bc6ed3b5
+  data.tar.gz: cba881489e3cbb1e43a6b87e617eb4a28a3f83ef
 SHA512:
-  metadata.gz: 4be333a99bf094965edf5f1636f84d039525d873844d4ba62ca75645676893d73ee1106948acd12b3f6302d231e441a323464663f2927e1b02a90fa1bc5bc709
-  data.tar.gz: 2c5d2de74c766c08956095a851a4f7405596327a5bc98e2ce9ad9e8b0d5b7b98f1b3040ec412423afe729e6f000c123293d36abe3e7a663bd5446d2c61c6bdef
+  metadata.gz: 3250b185fc57e85c66a1b25b84ad111a1d98b31a76fa434e6587c16a52551d0cc9b43a2ea2abd4b7f963ad723edfe8cbb1807e12e10010d5f37865b8e9d1b198
+  data.tar.gz: 137c400ee0f8270e69951d5e49c6c61ee5849285983c8121a0dd1fa2aba9692a84d340bd063719445630ed92144268c4ab1fd782b4777306bed47af8618da1cb

data/lib/deferring.rb

@@ -175,8 +175,10 @@ module Deferring
     define_method :"#{association_name.singularize}_ids=" do |ids|
       find_or_create_deferred_association(association_name, listeners, inverse_association_name)
 
+      ids ||= []
       klass = self.class.reflect_on_association(:"#{association_name}").klass
       objects = klass.find(ids.reject(&:blank?))
+
       send(:"deferred_#{association_name}").objects = objects
     end
 
@@ -192,6 +194,7 @@ module Deferring
     attr_accessor :"#{association_name}_checked"
     # collection_singular_checked=
     define_method(:"#{association_name}_checked=") do |ids|
+      ids ||= ''
       send(:"#{association_name.singularize}_ids=", ids.split(','))
     end
 

data/lib/deferring/version.rb

@@ -1,5 +1,5 @@
 # encoding: UTF-8
 
 module Deferring
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

data/spec/lib/deferring_habtm_spec.rb

@@ -101,14 +101,65 @@ RSpec.describe 'deferred has_and_belongs_to_many associations' do
         expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(0).to(1)
       end
 
-      describe '#collection_checked=' do
+      it 'unlinks all records when assigning empty array' do
+        bob.team_ids = [dba.id, operations.id]
+        bob.save
+
+        bob.team_ids = []
+        expect(bob.teams.length).to eq(0)
+
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(2).to(0)
+      end
+
+      it 'unlinks all records when assigning nil' do
+        bob.team_ids = [dba.id, operations.id]
+        bob.save
+
+        bob.team_ids = nil
+        expect(bob.teams.length).to eq(0)
+
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(2).to(0)
+      end
+    end
+
+    describe '#collection_checked=' do
+
+      it 'set associated records' do
+        bob.teams_checked = "#{dba.id},#{operations.id}"
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(0).to(2)
+      end
 
-        it 'set associated records' do
-          bob.teams_checked = [dba.id, operations.id]
+      it 'replace existing records when assigning a new set of ids of records' do
+        bob.team_ids = [dba.id, operations.id]
+        bob.save
+
+        # Replace DBA and Operations by Support.
+        bob.teams_checked = "#{support.id}"
+
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(2).to(1)
+        expect(bob.teams.first).to eq(support)
+      end
+
+      it 'unlinks all records when assigning empty string' do
+        bob.team_ids = [dba.id, operations.id]
+        bob.save
+
+        # Unlinks DBA and Operations.
+        bob.teams_checked = ""
+        expect(bob.teams.length).to eq(0)
+
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(2).to(0)
+      end
+
+      it 'unlinks all records when assigning nil' do
+        bob.team_ids = [dba.id, operations.id]
+        bob.save
 
-          expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(0).to(2)
-        end
+        # Unlinks DBA and Operations.
+        bob.teams_checked = nil
+        expect(bob.teams.length).to eq(0)
 
+        expect{ bob.save }.to change{ Person.where(name: 'Bob').first.teams.size }.from(2).to(0)
       end
 
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deferring
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Robin Roestenburg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-04 00:00:00.000000000 Z
+date: 2014-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord