metadata_presenter 2.10.0 → 2.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e03698bc80c7779212a57aaa9bbca97b5567025b950a6a6b7170fb635745e3e0
-  data.tar.gz: 7c3eca3a9da18aa6001791d2c5def0abacbf95b8b9baeb2be8920df804734bdd
+  metadata.gz: 6cca8a3428e516ba176e5a898a1d1a76cd57fa836041f0c3fc492a8cd394e88a
+  data.tar.gz: 9940b709ccce5e211444fe6b7cf84dfeed78f1c13591c9291b797a3eca4ac609
 SHA512:
-  metadata.gz: a6dedb4d4413eab6222933b8096bf45c6fef7db7d51853e1a46ecaf51f4b718362bbe00776f5adfc3b3687dcf10fb6660b289d9dc930768dadfa5dc1bfbda1b9
-  data.tar.gz: c822f36b904e46fd6075979dd3acca19ed1e91ddcf73a5adf93ca4ad261e2da5e5507866fe890641c116ecb4695e25b16281fd5aa54304b7328b50b5a4d55b34
+  metadata.gz: 1b9c3ae006ca76d0386db74a2fb39d106884dc0cd285457c05be6f3d3f75e2a30cc79511112d3881c532ba7e6e3c087e5e83e4116a708cf3cd81b4ea345de813
+  data.tar.gz: 659da027fb5c27df2c007dfd961bfce0148879aec3833f72c53f4d195080bde041b29c53260c397430e5425383a2c7a364e1ed8fd4d22bbabdb33417ecb896ed

data/app/models/metadata_presenter/branch_destinations.rb

@@ -0,0 +1,18 @@
+module MetadataPresenter
+  module BranchDestinations
+    # The frontend requires that expressions of type 'or' get there own line and
+    # arrow. 'and' expression types continue to be grouped together.
+    # Return the UUIDs of the destinations exiting a branch and allow duplicates
+    # if the expression type is an 'or'.
+    def exiting_destinations_from_branch(branch)
+      destination_uuids = branch.conditionals.map do |conditional|
+        if conditional.type == 'or'
+          conditional.expressions.map { |_| conditional.next }
+        else
+          conditional.next
+        end
+      end
+      destination_uuids.flatten.push(branch.default_next)
+    end
+  end
+end

data/app/models/metadata_presenter/column_number.rb

@@ -0,0 +1,25 @@
+module MetadataPresenter
+  class ColumnNumber
+    def initialize(uuid:, coordinates:, new_column:)
+      @uuid = uuid
+      @coordinates = coordinates
+      @new_column = new_column
+    end
+
+    def number
+      use_new_column? ? new_column : existing_column
+    end
+
+    private
+
+    attr_reader :uuid, :coordinates, :new_column
+
+    def use_new_column?
+      existing_column.nil? || new_column > existing_column
+    end
+
+    def existing_column
+      @existing_column ||= @coordinates.uuid_column(uuid)
+    end
+  end
+end

data/app/models/metadata_presenter/coordinates.rb

@@ -0,0 +1,59 @@
+module MetadataPresenter
+  class Coordinates
+    def initialize(flow)
+      @flow = flow
+      @positions = setup_positions
+    end
+
+    attr_reader :positions
+
+    def set_column(uuid, number)
+      positions[uuid][:column] = number
+    end
+
+    def set_row(uuid, number)
+      positions[uuid][:row] = number
+    end
+
+    def uuid_column(uuid)
+      positions[uuid][:column]
+    end
+
+    def uuid_row(uuid)
+      positions[uuid][:row]
+    end
+
+    def uuid_at_position(column, row)
+      positions.find do |uuid, position|
+        if position[:column] == column && position[:row] == row
+          return uuid
+        end
+      end
+    end
+
+    def position(uuid)
+      positions[uuid]
+    end
+
+    def occupied?(column, row, new_object_uuid)
+      positions.any? do |uuid, position|
+        position[:column] == column &&
+          position[:row] == row &&
+          new_object_uuid != uuid
+      end
+    end
+
+    def positions_in_column(column_number)
+      positions.select { |_, position| position[:column] == column_number }
+    end
+
+    private
+
+    attr_reader :flow
+    attr_writer :positions
+
+    def setup_positions
+      flow.keys.index_with { |_uuid| { row: nil, column: nil } }
+    end
+  end
+end

data/app/models/metadata_presenter/grid.rb

@@ -12,6 +12,7 @@ module MetadataPresenter
   end
 
   class Grid
+    include BranchDestinations
     attr_reader :start_from
 
     def initialize(service, start_from: nil, main_flow: [])
@@ -21,7 +22,7 @@ module MetadataPresenter
       @ordered = []
       @routes = []
       @traversed = []
-      @coordinates = setup_coordinates
+      @coordinates = MetadataPresenter::Coordinates.new(service.flow)
     end
 
     ROW_ZERO = 0
@@ -30,8 +31,8 @@ module MetadataPresenter
       return @ordered unless @ordered.empty?
 
       @ordered = make_grid
-      add_columns
-      add_rows
+      set_column_numbers
+      set_row_numbers
       add_by_coordinates
       insert_expression_spacers
       trim_pointers unless main_flow.empty?
@@ -62,10 +63,6 @@ module MetadataPresenter
     attr_reader :service, :main_flow
     attr_accessor :ordered, :traversed, :routes, :coordinates
 
-    def setup_coordinates
-      service.flow.keys.index_with { |_uuid| { row: nil, column: nil } }
-    end
-
     def route_from_start
       @route_from_start ||=
         MetadataPresenter::Route.new(
@@ -83,7 +80,12 @@ module MetadataPresenter
     end
 
     def max_potential_rows
-      @max_potential_rows ||= @routes.map(&:row).max + 1
+      @max_potential_rows ||= begin
+        destinations_count = service.branches.map do |branch|
+          exiting_destinations_from_branch(branch).count
+        end
+        destinations_count.sum
+      end
     end
 
     def max_potential_columns
@@ -121,29 +123,47 @@ module MetadataPresenter
       end
     end
 
-    def add_columns
+    def set_column_numbers
       @routes.each do |route|
-        route.flow_uuids.each.with_index(route.column) do |uuid, column|
-          column_number = @coordinates[uuid][:column]
-          if column_number.nil? || column > column_number
-            @coordinates[uuid][:column] = column
-          end
+        route.flow_uuids.each.with_index(route.column) do |uuid, new_column|
+          column_number = MetadataPresenter::ColumnNumber.new(
+            uuid: uuid,
+            new_column: new_column,
+            coordinates: @coordinates
+          ).number
+          @coordinates.set_column(uuid, column_number)
         end
       end
     end
 
-    def add_rows
+    def set_row_numbers
       @routes.each do |route|
-        next if @traversed.include?(route.traverse_from)
+        next if @traversed.include?(route.traverse_from) && appears_later_in_flow?(route)
 
+        current_row = route.row
         route.flow_uuids.each do |uuid|
-          @coordinates[uuid][:row] = route.row if @coordinates[uuid][:row].nil?
+          row_number = MetadataPresenter::RowNumber.new(
+            uuid: uuid,
+            route: route,
+            current_row: current_row,
+            coordinates: @coordinates,
+            service: service
+          ).number
+          @coordinates.set_row(uuid, row_number)
+
           update_route_rows(route, uuid)
           @traversed.push(uuid) unless @traversed.include?(uuid)
+          current_row = row_number
         end
       end
     end
 
+    # New routes can be linked to later. We need to also traverse these to see
+    # if anything should be moved to a different row.
+    def appears_later_in_flow?(route)
+      @coordinates.uuid_column(route.traverse_from) > route.column
+    end
+
     # Each Route object has a starting row. Each Route object has no knowledge
     # of other potential routes and pages/branches that may or may not exist in
     # them. The starting row may need to change dependent upon what has been
@@ -161,7 +181,8 @@ module MetadataPresenter
     end
 
     def add_by_coordinates
-      @coordinates.each do |uuid, position|
+      service.flow.each_key do |uuid|
+        position = coordinates.position(uuid)
         next if detached?(position)
 
         @ordered[position[:column]][position[:row]] = get_flow_object(uuid)
@@ -219,37 +240,23 @@ module MetadataPresenter
     # the one the branch is located in.
     def insert_expression_spacers
       service.branches.each do |branch|
-        position = @coordinates[branch.uuid]
-        next if position[:row].nil? || position[:column].nil? # detached branch
+        next if coordinates.uuid_column(branch.uuid).nil?
 
-        next_column = position[:column] + 1
-        uuids = []
-        exiting_destinations_from_branch(branch).each.with_index(position[:row]) do |uuid, row|
-          if uuids.include?(uuid)
+        previous_uuid = ''
+        next_column = coordinates.uuid_column(branch.uuid) + 1
+        exiting_destinations_from_branch(branch).each_with_index do |uuid, row|
+          if uuid == previous_uuid
             @ordered[next_column].insert(row, MetadataPresenter::Spacer.new)
           end
-
-          uuids.push(uuid) unless uuids.include?(uuid)
-        end
-      end
-    end
-
-    # The frontend requires that expressions of type 'or' get there own line and
-    # arrow. 'and' expression types continue to be grouped together.
-    # Return the UUIDs of the destinations exiting a branch and allow duplicates
-    # if the expression type is an 'or'.
-    def exiting_destinations_from_branch(branch)
-      destination_uuids = branch.conditionals.map do |conditional|
-        if conditional.type == 'or'
-          conditional.expressions.map(&:next)
-        else
-          conditional.next
+          previous_uuid = uuid
         end
       end
-      destination_uuids.flatten
     end
 
     # Any destinations exiting the branch that have not already been traversed.
+    # This removes any branch destinations that already exist on other rows. If
+    # that is the case then the arrow will flow towards whatever row that object
+    # is located.
     def routes_exiting_branch(branch)
       branch.all_destination_uuids.reject { |uuid| @traversed.include?(uuid) }
     end

data/app/models/metadata_presenter/row_number.rb

@@ -0,0 +1,71 @@
+module MetadataPresenter
+  class RowNumber
+    include BranchDestinations
+
+    def initialize(uuid:, route:, current_row:, coordinates:, service:)
+      @uuid = uuid
+      @route = route
+      @current_row = current_row
+      @coordinates = coordinates
+      @service = service
+    end
+
+    ROW_ZERO = 0
+
+    def number
+      return route.row if first_row? && existing_row.nil?
+
+      return ROW_ZERO if place_on_row_zero?
+
+      if object_above.branch? && uuid != object_above.uuid
+        coordinates.uuid_row(object_above.uuid) + number_of_destinations
+      else
+        existing_row.nil? ? current_row : [current_row, existing_row].max
+      end
+    end
+
+    private
+
+    attr_reader :uuid, :route, :current_row, :coordinates, :service
+
+    def existing_row
+      @existing_row ||= coordinates.uuid_row(uuid)
+    end
+
+    def first_row?
+      @first_row ||= route.row.zero?
+    end
+
+    def object_above
+      @object_above ||=
+        service.flow_object(coordinates.uuid_at_position(uuid_column, row_above))
+    end
+
+    # Takes into account the 'or' type of conditionals which requires an
+    # additional spacer
+    def number_of_destinations
+      exiting_destinations_from_branch(object_above).count
+    end
+
+    def uuid_column
+      @uuid_column ||= coordinates.uuid_column(uuid)
+    end
+
+    def row_above
+      @row_above ||= route.row - 1
+    end
+
+    # If an object has already been positioned on row 0 then leave it there.
+    # If the object is a checkanswers or confirmation type then always place it
+    # on row 0.
+    def place_on_row_zero?
+      cya_or_confirmation_page? || coordinates.uuid_row(uuid) == ROW_ZERO
+    end
+
+    def cya_or_confirmation_page?
+      %w[page.checkanswers page.confirmation].include?(
+        service.find_page_by_uuid(uuid)&.type
+      )
+    end
+  end
+end