Move Attributes Interpretation and delegation to Procedure

Schedules are a type of banch procedure with atts, thus extracting this to the parent class Procecure made sense Related to mantor#58
jfrioux · Dec 6, 2020 · 94e4662 · 94e4662
1 parent a117335
commit 94e4662
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/app/models/floristry/leaf_procedure.rb b/app/models/floristry/leaf_procedure.rb
@@ -5,16 +5,12 @@ class LeafProcedure < Procedure
     include LeafProcedureMixin
 
     attr_accessor :payload
-    attr_reader :att, :atts
-
-    delegate :att, :atts, :raw_atts, to: :@atts
 
     def initialize(id, name, atts, payload, era)
 
       super
 
       @payload = payload
-      @atts = Floristry::AttributesInterpreter.new(atts)
     end
   end
 end

diff --git a/app/models/floristry/procedure.rb b/app/models/floristry/procedure.rb
@@ -4,7 +4,8 @@ class Procedure
     include Floristry::CommonMixin
     include ProcedureMixin
 
-    attr_reader :id, :name, :payload, :era
+    attr_reader :id, :name, :payload, :era, :att, :atts
+    delegate :att, :atts, :raw_atts, to: :@atts
 
     def initialize(id, name, atts, payload, era)
 
@@ -19,6 +20,7 @@ def initialize(id, name, atts, payload, era)
       end
 
       @name = name
+      @atts = Floristry::AttributesInterpreter.new(atts)
       @payload = payload
       @era = era