active_record-json_associations 0.6.7 → 0.6.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: db8990fae5a00349dbfb5f3fd9066c590126e637
-  data.tar.gz: cd5fa6ecb84d09a2bc72066dea8f3a133b02d0d3
+  metadata.gz: b1d844089bbc9addaaa6680fff31aac70e44ecf8
+  data.tar.gz: 53bacf65b6bbfe4a2e47cc300e88235733e7c20e
 SHA512:
-  metadata.gz: 8cccd447bcdb4d412e0bcf77fbb2c6ae84497fc7cd34e28993788dd2473ec55ed9b8fb72332186f6612f11e8d4bbd78631d57e22d14527b7e20f4d1b36de3c1b
-  data.tar.gz: 3800a7453d7046e4a733b54f901dce2e71f88b061309071148c38eb19a0b2acd0394d2435402f464615cfe9997d1a71238ddb628b1efeb7bd660394a5ce6f3cb
+  metadata.gz: 1e88b1f2079405a05c2b75490845f7c2dc2918110b3aebd977148c9b2f630a22bdec21ca276c75c3c2e20992a2e523b57961af054cc844caa8b27e57c1f7e723
+  data.tar.gz: 98cc39387cfab3b919c44d44e0065d2f5b76fc2f24fc9d72053e6cadbd057f0524d12c744a4f77eac3d9ddcd0bb4153f533fd36682dd955085c8496d28a8a168

data/lib/active_record/json_associations.rb

@@ -25,7 +25,16 @@ module ActiveRecord
 
         define_method many do
           klass = class_name.constantize
-          klass.where(klass.primary_key => send(one_ids))
+          scope = klass.all
+
+          ids = send(one_ids)
+          scope.where!(klass.primary_key => ids)
+
+          fragments = []
+          fragments += ["#{klass.primary_key} NOT IN (#{ids.map(&:to_s).join(",")})"] if ids.any?
+          fragments += ids.reverse.map { |id| "#{klass.primary_key}=#{id}" }
+          order_by_ids = fragments.join(", ")
+          scope.order!(order_by_ids)
         end
 
         define_method many_equals do |collection|

data/lib/active_record/json_associations/version.rb

@@ -1,5 +1,5 @@
 module ActiveRecord
   module JsonAssociations
-    VERSION = "0.6.7"
+    VERSION = "0.6.8"
   end
 end

data/spec/json_associations_spec.rb

@@ -74,6 +74,11 @@ describe ActiveRecord::JsonAssociations do
         expect(subject.children).to eq children
       end
 
+      it "finds the children by id order" do
+        subject.child_ids = [3,2,1]
+        expect(subject.children).to eq children.reverse
+      end
+
       it "is an accessor" do
         subject.children = children
         expect(subject.children).to eq children

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: active_record-json_associations
 version: !ruby/object:Gem::Version
-  version: 0.6.7
+  version: 0.6.8
 platform: ruby
 authors:
 - Micah Geisel
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-03 00:00:00.000000000 Z
+date: 2017-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord