activefacts-api 1.9.10 → 1.9.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: b2ae2f52c3b4809d9032158fab55e93634e6bb6f
-  data.tar.gz: 9da905e8331bf238bc7d4aebd1dc556181cc24b8
+SHA256:
+  metadata.gz: 8c3ae722ceb4f264e163be7c166507f6ab3ab328cea1513ff0599ea0c54a3270
+  data.tar.gz: '096701429372fe651503a58c4631c0f6090f21493555a3c2705ec2cbd71c1f04'
 SHA512:
-  metadata.gz: 8c5ab2949f5a5767f9e3eaa1951bbbe0de21bcbd2bb5a3469f127f17cae9ddbbfe0f5d184bcb6fcd6bd0d8a68cb80b683bb7abacf4e1dac3d0d2f33a77d3e0b5
-  data.tar.gz: d83db68561b25300a0ef71b753c0e983db8aacf35b6a7f0c48f5880772150801a4f7fcfc7e4ef1f90b4cbc361f3c9434c97cfa64bf72c621401961ce12c5c459
+  metadata.gz: cbb093b3faaa2c8f31e5c9f33c6afd076e3d72e803beb57272eb6769e6a44af9a98c909f9643c5d08da3f9fe0e0815ed76b90cecd9de524a6453497485d79346
+  data.tar.gz: 8f349dca05234d1b24c2408eb81ee5fbc3d0a3458021c099374cf4da6118c349d7dbc32c5e82a86c462da06acb878f6de8ab1f43a8de58cdcced608ec8b79979

data/activefacts-api.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   spec.version       = ActiveFacts::API::VERSION
   spec.authors       = ["Clifford Heath"]
   spec.email         = ["clifford.heath@gmail.com"]
-  spec.date = "2015-10-02"
+  spec.date          = "2022-06-15"
 
   spec.summary = "A fact-based data model DSL and API"
   spec.description   = %q{
@@ -35,7 +35,7 @@ over the fact population.
   spec.add_runtime_dependency 'tracing', ["~> 2", ">= 2.0.4"]
 
   spec.add_development_dependency "bundler", ">= 1.10"
-  spec.add_development_dependency "rake", "~> 10.0"
+  spec.add_development_dependency "rake", ">= 10"
   spec.add_development_dependency "rspec", "~> 3.3"
 end
 

data/lib/activefacts/api/constellation.rb

@@ -65,7 +65,7 @@ module ActiveFacts
       # With no parameters, return the collection of all instances of that object_type.
       # With parameters, assert an instance of the object_type identified by the values passed as args.
       def method_missing(m, *args, &b)
-        klass = @vocabulary.const_get(m)
+        klass = @vocabulary.const_get(m) rescue nil   # Catch missing lower-case names (wrong constant) and use super
         if invalid_object_type klass
           super
         else
@@ -75,7 +75,8 @@ module ActiveFacts
       end
 
       def define_class_accessor m, klass
-        singleton_class.send(:define_method, m) do |*args|
+        singleton_class.
+          send(:define_method, m) do |*args|
             if args.size == 0
               # Return the collection of all instances of this class in the constellation:
               instances[klass]

data/lib/activefacts/api/entity.rb

@@ -32,9 +32,9 @@ module ActiveFacts
       end
 
       def initialize_existential_roles(klass, arg_hash)
-        # If overrides_identification_of, assign those attributes too (recursively)
-        if o = klass.overrides_identification_of
-          initialize_existential_roles(o, arg_hash)
+        # Assign the identifying attributes of all superclasses first
+        klass.supertypes_transitive.each do |supertype|
+          initialize_existential_roles(supertype, arg_hash)
         end
 
         irns = klass.identifying_role_names
@@ -159,9 +159,9 @@ module ActiveFacts
       # identified by this entity.  Save the current key and
       # class for each such instance.
       # This function is transitive!
-      def analyse_impacts role
-        impacts = []
-        impacted_roles = []
+      def collect_instance_index_updates role
+        instance_index_updates = []
+        propagation_roles = []
 
         # Consider the object itself and all its supertypes
         ([self.class]+self.class.supertypes_transitive).map do |supertype|
@@ -169,7 +169,7 @@ module ActiveFacts
 
           old_key = identifying_role_values(supertype)
           # puts "Need to reindex #{self.class} as #{supertype} from #{old_key.inspect}"
-          impacts << [constellation.instances[supertype], self, old_key]
+          instance_index_updates << [constellation.instances[supertype], self, old_key]
 
           supertype.
           all_role.
@@ -178,29 +178,29 @@ module ActiveFacts
             next unless counterpart = propagation_role.counterpart  # And the role is not unary
             if counterpart.is_identifying                         # This object identifies another
               # puts "Changing #{propagation_role.inspect} affects #{counterpart.inspect}"
-              impacted_roles << propagation_role
+              propagation_roles << propagation_role
             else
               next if counterpart.unique                          # But a one-to-many
               next unless value = send(propagation_role.getter)   # A value is set
               role_values = value.send(counterpart.getter)        # This is the index we have to change
               # puts "Changing #{role.inspect} of a #{self.class} requires updating #{propagation_role.counterpart.inspect}"
-              impacts << [role_values, self, old_key]
+              instance_index_updates << [role_values, self, old_key]
             end
           end
         end
 
-        impacted_roles.each do |role|
+        propagation_roles.each do |role|
           affected_instances = Array(send(role.getter))
           # puts "considering #{affected_instances.size} #{role.object_type.name} instances that include #{role.inspect}: #{affected_instances.map(&:identifying_role_values).inspect}"
           affected_instances.each do |counterpart|
-            impacts.concat(counterpart.analyse_impacts(role.counterpart))
+            instance_index_updates.concat(counterpart.collect_instance_index_updates(role.counterpart))
           end
         end
-        impacts
+        instance_index_updates
       end
 
-      def apply_impacts impacts
-        impacts.each do |index, entity, old_key|
+      def apply_instance_index_updates instance_index_updates
+        instance_index_updates.each do |index, entity, old_key|
           new_key = entity.identifying_role_values(index.object_type)
           # puts "Reindexing #{klass} from #{old_key.inspect} to #{new_key.inspect}"
 
@@ -390,6 +390,7 @@ module ActiveFacts
             unless role.is_identifying && role.object_type == self
               value =
                 if v == nil
+                  raise "nil value passed for mandatory #{role.name} of #{role.object_type.name}" if role.mandatory
                   nil
                 elsif role.unary?
                   (v && true)   # Preserve nil and false
@@ -465,7 +466,7 @@ module ActiveFacts
 
         def other.new_instance constellation, *args
           instance = allocate
-          instance.instance_variable_set(@@constellation_variable_name ||= "@constellation", constellation)
+          instance.instance_variable_set(constellation_variable_name, constellation)
           instance.send(:initialize, *args)
           instance
         end

data/lib/activefacts/api/guid.rb

@@ -1,11 +1,13 @@
 require 'delegate'
 require 'securerandom'
 
-def SecureRandom.format_uuid hex32
-  hex32.sub(
-    @@format_pattern ||= /(........)(....)(....)(....)(............)/,
-    @@format_string ||= '\1-\2-\3-\4-\5'
-  )
+module SecureRandom
+  def self.format_uuid hex32
+    hex32.sub(
+      @@format_pattern ||= /(........)(....)(....)(....)(............)/,
+      @@format_string ||= '\1-\2-\3-\4-\5'
+    )
+  end
 end
 
 unless defined? SecureRandom.uuid

data/lib/activefacts/api/instance.rb

@@ -26,6 +26,10 @@ module ActiveFacts
         end
       end
 
+      def constellation_variable_name
+        @@constellation_variable_name ||= "@constellation"
+      end
+
       def is_a? klass
         super || self.class.supertypes_transitive.include?(klass)
       end
@@ -77,14 +81,14 @@ module ActiveFacts
         # The counterpart roles get cleared automatically.
         klasses = [self.class]+self.class.supertypes_transitive
 
-        irvrvs = {}  # identifying_role_values by RoleValues
+        key_by_type = {}
         self.class.all_role_transitive.each do |_, role|
           next unless role.counterpart and
             role.unique and
             !role.counterpart.unique and
             counterpart = send(role.getter)
           role_values = counterpart.send(role.counterpart.getter)
-          irvrvs[role_values] = role_values.index_values(self)
+          key_by_type[role.object_type] ||= identifying_role_values(role.object_type)
         end
 
         # Nullify the counterpart role of objects we identify first, before damaging our identifying_role_values:
@@ -124,7 +128,7 @@ module ActiveFacts
                 counterpart_instance.send(counterpart.setter, nil, false)
               else
                 rv = counterpart_instance.send(role.counterpart.getter)
-                rv.delete_instance(self, irvrvs[rv])
+                rv.delete_instance(self, key_by_type[role.object_type])
 
                 if (rv.empty? && !counterpart_instance.class.is_entity_type)
                   counterpart_instance.retract if counterpart_instance.plays_no_role
@@ -158,6 +162,10 @@ module ActiveFacts
       module ClassMethods #:nodoc:
         include ObjectType
         # Add Instance class methods here
+
+        def constellation_variable_name
+          @@constellation_variable_name ||= "@constellation"
+        end
       end
     end
   end

data/lib/activefacts/api/numeric.rb

@@ -62,7 +62,9 @@ class Decimal < SimpleDelegator #:nodoc:
   include ActiveFacts::API::SimpleDelegation
 
   def delegate_new(v)
-    if v.is_a?(BigDecimal) || v.is_a?(Bignum)
+    if RUBY_VERSION >= "3"
+      BigDecimal(v)
+    elsif v.is_a?(BigDecimal) || v.is_a?(Integer)
       BigDecimal.new(v.to_s)
     else
       BigDecimal.new(v)

data/lib/activefacts/api/object_type.rb

@@ -260,16 +260,16 @@ module ActiveFacts
 
           if role.is_identifying and (options&CHECKED_IDENTIFYING_ROLE) == 0
             check_identification_change_legality(role, value)
-            impacts = analyse_impacts(role)
+            instance_index_updates = collect_instance_index_updates(role)
           end
 
           instance_variable_set(role.variable, value)
 
-          if impacts
+          if instance_index_updates
             @constellation.when_admitted do
               # REVISIT: Consider whether we want to provide a way to find all instances
               # playing/not playing this boolean role, analogous to true.all_thing_as_role_name...
-              apply_impacts(impacts)    # Propagate dependent key changes
+              apply_instance_index_updates(instance_index_updates)    # Propagate dependent key changes
             end
           end
 
@@ -314,7 +314,7 @@ module ActiveFacts
             check_identification_change_legality(role, value)
 
             # puts "Starting to analyse impact of changing 1-1 #{role.inspect} to #{value.inspect}"
-            impacts = analyse_impacts(role)
+            instance_index_updates = collect_instance_index_updates(role)
           end
 
           instance_variable_set(role_var, value)
@@ -328,7 +328,7 @@ module ActiveFacts
             # Assign self to the new counterpart
             value.send(role.counterpart.setter, self, options) if value && (options&SKIP_MUTUAL_PROPAGATION) == 0
 
-            apply_impacts(impacts) if impacts   # Propagate dependent key changes
+            apply_instance_index_updates(instance_index_updates) if instance_index_updates   # Propagate dependent key changes
           end
 
           unless @constellation.loggers.empty? or options != 0
@@ -363,12 +363,12 @@ module ActiveFacts
             check_identification_change_legality(role, value)
 
             # puts "Starting to analyse impact of changing 1-N #{role.inspect} to #{value.inspect}"
-            impacts = analyse_impacts(role)
+            instance_index_updates = collect_instance_index_updates(role)
           end
 
           if old and (options&SKIP_MUTUAL_PROPAGATION) == 0
             old_role_values = old.send(getter = role.counterpart.getter)
-            old_key = old_role_values.index_values(self)
+            old_key = identifying_role_values(role.object_type)
           end
 
           instance_variable_set(role_var, value)
@@ -388,7 +388,7 @@ module ActiveFacts
               rv.add_instance(self, identifying_role_values(role.object_type))
             end
 
-            apply_impacts(impacts) if impacts   # Propagate dependent key changes
+            apply_instance_index_updates(instance_index_updates) if instance_index_updates   # Propagate dependent key changes
           end
 
           unless @constellation.loggers.empty? or options != 0

data/lib/activefacts/api/role_values.rb

@@ -11,6 +11,7 @@ module ActiveFacts
       attr_accessor :role
       attr_accessor :sort
       attr_accessor :index_roles
+
       def object_type
         @role.object_type
       end
@@ -21,7 +22,11 @@ module ActiveFacts
         @sort = API::sorted
         @excluded_role = excluded_role
         @a = @sort ? RBTree.new : []
-        (@index_roles = role.object_type.identifying_roles.dup).delete_at(@excluded_role) if @excluded_role
+        if @excluded_role
+          @index_roles = role.object_type.identifying_roles.dup
+          @index_roles.delete_at(@excluded_role)
+          @index_roles.freeze
+        end
       end
 
       def +(a)
@@ -68,17 +73,19 @@ module ActiveFacts
         KeyArray.new(a)
       end
 
+      # Return the full key for the object according to the object_type of this role
       def index_values object
-        if @index_roles
-          @index_roles.map{|r|
+        if @excluded_role
+          @index_roles.map do |r|
             role_value = object.send(r.name)
             role_value.identifying_role_values((c = r.counterpart) ? c.object_type : role_value.class)
-          }
+          end
         else
-          object.identifying_role_values
+          object.identifying_role_values(role.object_type)
         end
       end
 
+      # The key must include the excluded role, if any
       def add_instance(value, key)
         if @sort
           # Exclude the excluded role, if any:
@@ -89,9 +96,12 @@ module ActiveFacts
         end
       end
 
+      # The key must include the excluded role, if any
       def delete_instance(value, key)
         if @sort
-          # Exclude the excluded role, if any:
+          if key.size != role.object_type.identifying_roles.size
+            raise "Internal error: incorrectly-sized key #{key.inspect} to delete_instance"
+          end
           (key = key.dup).delete_at(@excluded_role) if @excluded_role
           deleted = @a.delete(form_key(key))
         else
@@ -134,16 +144,16 @@ module ActiveFacts
       def_single_delegator :@a, :empty?
       def_single_delegator :@a, :include?
       def_single_delegator :@a, :inject, 2
-      def_single_delegator :@a, :select, -1, 1
-      def_single_delegator :@a, :reject, -1, 1
+      def_single_delegator :@a, :select, -2, 1
+      def_single_delegator :@a, :reject, -2, 1
       def_single_delegator :@a, :size
-      def_single_delegator :@a, :sort_by, -1, 1
+      def_single_delegator :@a, :sort_by, -2, 1
       def_single_delegator :@a, :to_a
       def_single_delegator :@a, :-
       # These delegators allow a negative arity in RSpec because the tests test it (to make sure the code doesn't pass too many args)
-      def_single_delegator :@a, :each, *([-1, 1] + Array(defined?(::RSpec) ? -2 : nil))
+      def_single_delegator :@a, :each, *([-2, 1] + Array(defined?(::RSpec) ? -2 : nil))
       def_single_delegator :@a, :detect, 1, *([-1, 1] + Array(defined?(::RSpec) ? -2 : nil))
-      def_single_delegator :@a, :map, -1, 1
+      def_single_delegator :@a, :map, -2, 1
     end
   end
 end

data/lib/activefacts/api/standard_types.rb

@@ -17,7 +17,7 @@ module ActiveFacts
     module ValueClass #:nodoc:
       def value_type *args, &block #:nodoc:
         # The inclusion of instance methods triggers ClassMethods to be included in the class too
-        include ActiveFacts::API::Value
+        include ::ActiveFacts::API::Value
         value_type(*args, &block)
       end
     end

data/lib/activefacts/api/support.rb

@@ -11,12 +11,6 @@ unless Object.const_defined?("Infinity")
   Infinity = 1.0/0.0
 end
 
-class Symbol #:nodoc:
-  def to_proc
-    Proc.new{|*args| args.shift.__send__(self, *args)}
-  end
-end
-
 class String #:nodoc:
   # This may be overridden by a version from ActiveSupport. For our purposes, either will work.
   def camelcase(first_letter = :upper)

data/lib/activefacts/api/value.rb

@@ -155,7 +155,7 @@ module ActiveFacts
           class << klass
             def new_instance constellation, *args
               instance = allocate
-              instance.instance_variable_set(@@constellation_variable_name ||= "@constellation", constellation)
+              instance.instance_variable_set(constellation_variable_name, constellation)
               instance.send(:initialize, *args)
               instance
             end

data/lib/activefacts/api/version.rb

@@ -1,5 +1,5 @@
 module ActiveFacts
   module API
-    VERSION = "1.9.10"
+    VERSION = "1.9.13"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activefacts-api
 version: !ruby/object:Gem::Version
-  version: 1.9.10
+  version: 1.9.13
 platform: ruby
 authors:
 - Clifford Heath
-autorequire: 
+autorequire:
 bindir: exe
 cert_chain: []
-date: 2015-10-02 00:00:00.000000000 Z
+date: 2022-06-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rbtree-pure
@@ -68,16 +68,16 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '10'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -139,7 +139,7 @@ homepage: https://github.com/cjheath/activefacts-api
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -154,9 +154,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5
-signing_key: 
+rubygems_version: 3.2.22
+signing_key:
 specification_version: 4
 summary: A fact-based data model DSL and API
 test_files: []