foobara 0.0.132 → 0.0.133

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d4c8a877056c159be9d156cde81dececc246a1dda21e99c39d567603c4b99aca
-  data.tar.gz: 71f5c66811f399c7311b6a13fcd9562b7b11f1e6b3eb34e6cd0298cb6e2e548a
+  metadata.gz: 267e71a64eccb434c34233c30b65e56d1c69109667c4ba806645324cf53d9581
+  data.tar.gz: b870b83c02edd0bb5e117a57f769c1adf6c73abbfae68660688d82cdc91b3b1a
 SHA512:
-  metadata.gz: 97210eac20ace35283b2fdffe8ac36d924c3ac19336a34b914357a7c0cc7ab0a3097a867d78032898cbf8dc21ef57c6c605100919f96cfbbad15a3955cde4d2f
-  data.tar.gz: fb717c26f27f5fb6be91d77ddce128de252198523d4cc535d315fbe1029981951242a845b37460cee4cc3b6b0862a6220463750499e400d4a8d8426c62e776d4
+  metadata.gz: f1226974494886a43bdcf51e00ec0061453d6307e5e031836824747f4b583775edf5b1529fad4280a1520c173e1d3a94253a5bf2885b996d5e7814d426eaac10
+  data.tar.gz: e44c33f5c39c9e32c01c5057ad1a747fa95e07bc47e7e5bdf57b896fdc78bfea18e9f2f09661dc6bfb47da325a452108942c99d7e9d789aae3e4b5c945e57aa3

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [0.0.133] - 2025-06-26
+
+- Add StateMachine.states_that_can_perform
+- Add TransformedCommand#raw_inputs
+- Fix several TransactionTable#find_by* bugs that incorrectly match/don't match in-memory changed records
+- Fix bug that looked for associations in generic :attributes types
+
 # [0.0.132] - 2025-06-19
 
 - Add/improve the description for primary key types in EntityToPrimaryKeyInputsTransformer

data/projects/command_connectors/src/transformed_command.rb

@@ -791,6 +791,10 @@ module Foobara
       end
     end
 
+    def raw_inputs
+      untransformed_inputs
+    end
+
     def method_missing(method_name, ...)
       if command.respond_to?(method_name)
         command.send(method_name, ...)

data/projects/detached_entity/src/concerns/associations.rb

@@ -253,11 +253,13 @@ module Foobara
                 contains_associations?(element_type, false)
               end
             elsif type.extends?(BuiltinTypes[:attributes])
-              type.element_types.values.any? do |element_type|
+
+              type.element_types&.values&.any? do |element_type|
                 if !remove_sensitive || !element_type.sensitive?
                   contains_associations?(element_type, false)
                 end
               end
+
             elsif type.extends?(BuiltinTypes[:associative_array])
               element_types = type.element_types
 

data/projects/entity/src/concerns/initialization.rb

@@ -57,7 +57,7 @@ module Foobara
 
             record = current_transaction_table.find_tracked(record_id)
 
-            if record
+            if record&.loaded?
               # :nocov:
               raise "Already loaded for #{attributes}. Bug maybe?"
               # :nocov:

data/projects/persistence/src/entity_attributes_crud_driver.rb

@@ -130,8 +130,20 @@ module Foobara
         end
 
         def find_by_attribute_containing(attribute_name, value)
+          found_type = entity_class.attributes_type.type_at_path("#{attribute_name}.#")
+
+          if value
+            value = restore_attributes(value, found_type)
+          end
+
           all.find do |found_attributes|
-            found_attributes[attribute_name]&.include?(value)
+            found_attributes[attribute_name].any? do |found_value|
+              if found_value
+                found_value = restore_attributes(found_value, found_type)
+              end
+
+              found_value == value
+            end
           end
         end
 
@@ -291,7 +303,10 @@ module Foobara
             if outcome.success?
               outcome.result
             else
+              # TODO: figure out how to test this code path
+              # :nocov:
               object
+              # :nocov:
             end
           end
         end

data/projects/persistence/src/entity_base/transaction_table.rb

@@ -260,6 +260,7 @@ module Foobara
           value = entity_class.attributes_type.element_types[attribute_name].element_type.process_value!(value)
 
           tracked_records.each do |record|
+            next unless record.loaded? || record.created?
             next if hard_deleted?(record)
 
             # TODO: what if there are multiple??
@@ -274,7 +275,22 @@ module Foobara
           )
 
           if found_attributes
-            entity_class.loaded(normalize_attributes(found_attributes))
+            found_attributes = normalize_attributes(found_attributes)
+            record_id = primary_key_for_attributes(found_attributes)
+
+            record = find_tracked(record_id)
+
+            if record
+              # was already considered among tracked records if loaded? is true so do not return it as
+              # it has changed and no longer matches
+              unless record.loaded?
+                loading(record) do
+                  record.successfully_loaded(found_attributes)
+                end
+              end
+            else
+              entity_class.loaded(found_attributes)
+            end
           end
         end
 
@@ -286,6 +302,7 @@ module Foobara
           end
 
           tracked_records.each do |record|
+            next unless record.loaded? || record.created?
             next if hard_deleted?(record)
 
             if entity_attributes_crud_driver_table.matches_attributes_filter?(record.attributes, attributes_filter)
@@ -293,17 +310,37 @@ module Foobara
             end
           end
 
-          found_attributes = entity_attributes_crud_driver_table.find_by(attributes_filter)
+          found_attributes_enumerator = entity_attributes_crud_driver_table.find_many_by(attributes_filter)
 
-          if found_attributes
+          found_attributes_enumerator.each do |found_attributes|
             found_attributes = normalize_attributes(found_attributes)
             record_id = primary_key_for_attributes(found_attributes)
 
             record = find_tracked(record_id)
-            return record if record
 
-            entity_class.loaded(found_attributes)
+            if record
+              if record.loaded?
+                # was already considered among tracked records if loaded? is true so do not return it as
+                # it has changed and no longer matches
+                # TODO: figure out how to test this code path
+                # :nocov:
+                record = nil
+                # :nocov:
+              else
+                loading(record) do
+                  record.successfully_loaded(found_attributes)
+                end
+              end
+            else
+              record = entity_class.loaded(found_attributes)
+            end
+
+            if record
+              return record
+            end
           end
+
+          nil
         end
 
         def find_many_by(attributes_filter)
@@ -353,9 +390,28 @@ module Foobara
 
               next if yielded_ids.include?(record_id)
 
-              record = find_tracked(record_id) || entity_class.loaded(found_attributes)
+              record = find_tracked(record_id)
+
+              if record
+                if record.loaded?
+                  # was already considered among tracked records if loaded? is true so do not return it as
+                  # it has changed and no longer matches
+                  # TODO: figure out how to test this code path
+                  # :nocov:
+                  record = nil
+                  # :nocov:
+                else
+                  loading(record) do
+                    record.successfully_loaded(found_attributes)
+                  end
+                end
+              else
+                record = entity_class.loaded(found_attributes)
+              end
 
-              yielder << record
+              if record
+                yielder << record
+              end
             end
           end
         end
@@ -374,6 +430,7 @@ module Foobara
 
           Enumerator.new do |yielder|
             tracked_records.each do |record|
+              next unless record.loaded? || record.created?
               next if hard_deleted?(record)
 
               record_values = record.read_attribute(attribute_name)
@@ -401,9 +458,28 @@ module Foobara
 
               next if yielded_ids.include?(record_id)
 
-              record = find_tracked(record_id) || entity_class.loaded(found_attributes)
+              record = find_tracked(record_id)
+
+              if record
+                # TODO: test this code path
+                # :nocov:
+                if record.loaded?
+                  # was already considered among tracked records if loaded? is true so do not return it as
+                  # it has changed and no longer matches
+                  record = nil
+                else
+                  loading(record) do
+                    record.successfully_loaded(found_attributes)
+                  end
+                end
+                # :nocov:
+              else
+                record = entity_class.loaded(found_attributes)
+              end
 
-              yielder << record
+              if record
+                yielder << record
+              end
             end
           end
         end
@@ -422,6 +498,7 @@ module Foobara
 
           Enumerator.new do |yielder|
             tracked_records.each do |record|
+              next unless record.loaded? || record.created?
               next if hard_deleted?(record)
 
               record_value = record.read_attribute(attribute_name)
@@ -445,9 +522,28 @@ module Foobara
 
               next if yielded_ids.include?(record_id)
 
-              record = find_tracked(record_id) || entity_class.loaded(found_attributes)
+              record = find_tracked(record_id)
+
+              if record
+                # TODO: test this code path
+                # :nocov:
+                if record.loaded?
+                  # was already considered among tracked records if loaded? is true so do not return it as
+                  # it has changed and no longer matches
+                  record = nil
+                else
+                  loading(record) do
+                    record.successfully_loaded(found_attributes)
+                  end
+                end
+                # :nocov:
+              else
+                record = entity_class.loaded(found_attributes)
+              end
 
-              yielder << record
+              if record
+                yielder << record
+              end
             end
           end
         end

data/projects/state_machine/src/state_machine.rb

@@ -2,6 +2,7 @@ module Foobara
   require_project_file("state_machine", "sugar")
   require_project_file("state_machine", "callbacks")
   require_project_file("state_machine", "validations")
+  require_project_file("state_machine", "transitions")
 
   # TODO: allow quick creation of a statemachine either through better options to #initialize or a
   # .for method.
@@ -10,6 +11,7 @@ module Foobara
     include Callbacks
     include Validations
     include TransitionLog
+    include Transitions
 
     class InvalidTransition < StandardError; end
 

data/projects/state_machine/src/transitions.rb

@@ -0,0 +1,22 @@
+module Foobara
+  class StateMachine
+    module Transitions
+      include Concern
+
+      module ClassMethods
+        def states_that_can_perform(transition)
+          states = []
+          transition = transition.to_sym
+
+          transition_map.each_pair do |from, transitions|
+            if transitions.key?(transition)
+              states << from
+            end
+          end
+
+          states
+        end
+      end
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: foobara
 version: !ruby/object:Gem::Version
-  version: 0.0.132
+  version: 0.0.133
 platform: ruby
 authors:
 - Miles Georgi
@@ -406,6 +406,7 @@ files:
 - projects/state_machine/src/state_machine.rb
 - projects/state_machine/src/sugar.rb
 - projects/state_machine/src/transition_log.rb
+- projects/state_machine/src/transitions.rb
 - projects/state_machine/src/validations.rb
 - projects/type_declarations/lib/foobara/type_declarations.rb
 - projects/type_declarations/src/attributes.rb