reactive-record 0.7.27 → 0.7.28

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d768497702030401fd48ec8d9149f4c74e20c725
-  data.tar.gz: 4fbf024a9ade1663b8b2b0106f982d39b5642c92
+  metadata.gz: 4a5269da6b54feb09d691d5852e4b236b689d662
+  data.tar.gz: e6b947731add8eadc82e10b788d3880847665fb4
 SHA512:
-  metadata.gz: cdd59f7cc5d7704559c48099fb91c81b6ce124ad8817be2bf8a8b755d272dab3a8f2514cb3e48ddcf99dcd9ccd26b7f097fe56595ea94912f99cc5f6b65292bb
-  data.tar.gz: e09150e494aadd60d997b79473bd9cc9377027b4ea585f5ed243c34cf69629dc22be59e53c0aa39fc5772baea063c74b66c47d79fe4660efb3b57e00d6dfc16e
+  metadata.gz: 89102826cb196baffc99df77f910b6b5e620556c1ab7c2134fab12c46e05a0ee9b1ae564faddc51511f8674b136b1ad7b64235c709be7bda44b6ff41943aaf28
+  data.tar.gz: 4d8f4b1cf33dbb4391b2e094732cfdec9f329e88fff3eae8b45bfdac2c9a1e0212872c05de9d54fbfa1dcb459b56af1fc1b692820decc52a14eb957bfff377a0

data/lib/reactive_record/active_record/reactive_record/base.rb

@@ -35,6 +35,7 @@ module ReactiveRecord
     attr_accessor :destroyed
     attr_accessor :updated_during
     attr_accessor :synced_attributes
+    attr_accessor :virgin
 
     # While data is being loaded from the server certain internal behaviors need to change
     # for example records all record changes are synced as they happen.
@@ -86,7 +87,6 @@ module ReactiveRecord
         # and set the primary if we have one
         record.sync_attribute(model.primary_key, id) if id
       end
-
       # finally initialize and return the ar_instance
       record.ar_instance ||= infer_type_from_hash(model, record.attributes).new(record)
     end
@@ -214,6 +214,7 @@ module ReactiveRecord
           end
 
           aggregate_record = attributes[attribute].backing_record
+          aggregate_record.virgin = false
 
           if value
             value_attributes = value.backing_record.attributes

data/lib/reactive_record/active_record/reactive_record/isomorphic_base.rb

@@ -475,7 +475,8 @@ module ReactiveRecord
 
             saved_models = reactive_records.collect do |reactive_record_id, model|
               puts "saving rr_id: #{reactive_record_id} model.object_id: #{model.object_id} frozen? <#{model.frozen?}>"
-              if model and (model.frozen? or dont_save_list.include?(model))
+              if model and (model.frozen? or dont_save_list.include?(model) or model.changed.include?(model.class.primary_key))
+                # the above check for changed including the private key happens if you have an aggregate that includes its own id
                 puts "validating frozen model #{model.class.name} #{model} (reactive_record_id = #{reactive_record_id})"
                 valid = model.valid?
                 puts "has_errors before = #{has_errors}, validate= #{validate}, !valid= #{!valid}  (validate and !valid) #{validate and !valid}"

data/lib/reactive_record/server_data_cache.rb

@@ -187,9 +187,11 @@ module ReactiveRecord
                     self
                   end
                 rescue Exception => e
-                  binding.pry if cache_item.value and cache_item.value != []
-                  raise "ReactiveRecord exception caught when applying #{method} to db objects #{e}" if cache_item.value and cache_item.value != []
-                  representative
+                  if cache_item.value and cache_item.value != []
+                    raise "ReactiveRecord exception caught when applying #{method} to db object #{cache_item.value}: #{e}"
+                  else
+                    representative
+                  end
                 end
               else
                 representative
@@ -314,8 +316,8 @@ module ReactiveRecord
             target.send "#{method}=", aggregation.deserialize(value.first)
           elsif value.is_a? Array
             target.send "#{method}=", value.first unless method == "id" # we handle ids first so things sync nicely
-          elsif value.is_a? Hash and value[:id] and value[:id].first #and
-            association = target.class.reflect_on_association(method)
+          elsif value.is_a? Hash and value[:id] and value[:id].first and association = target.class.reflect_on_association(method)
+            # not sure if its necessary to check the id above... is it possible to for the method to be an association but not have an id?
             new_target = association.klass.find(value[:id].first)
             target.send "#{method}=", new_target
           elsif !(target.class < ActiveRecord::Base)

data/lib/reactive_record/version.rb

@@ -1,3 +1,3 @@
 module ReactiveRecord
-  VERSION = "0.7.27"
+  VERSION = "0.7.28"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: reactive-record
 version: !ruby/object:Gem::Version
-  version: 0.7.27
+  version: 0.7.28
 platform: ruby
 authors:
 - Mitch VanDuyn
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-16 00:00:00.000000000 Z
+date: 2015-10-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails