Merge pull request #302 from matestack/action-view-interoperability

action-view interoperability

Author: jonasjabari

.circleci/config.yml

@@ -51,7 +51,7 @@ jobs:
           name: run tests
           command: |
             mkdir /tmp/test-results
-            TEST_FILES="$(circleci tests glob "spec/usage/**/*_spec.rb" | circleci tests split --split-by=timings)"
+            TEST_FILES="$(circleci tests glob "spec/usage/**/*_spec.rb" "spec/matestack/**/*_spec.rb" | circleci tests split --split-by=timings)"
 
             bundle exec rspec --format progress \
                             --out /tmp/test-results/rspec.xml \

app/concepts/matestack/ui/core/component/dynamic.rb

@@ -4,6 +4,7 @@ class Dynamic < Trailblazer::Cell
     include ::Cell::Haml
     include Matestack::Ui::Core::ApplicationHelper
     include Matestack::Ui::Core::ToCell
+    include Matestack::Ui::Core::HasViewContext
 
     view_paths << "#{Matestack::Ui::Core::Engine.root}/app/concepts"
     view_paths << "#{::Rails.root}/app/matestack"

app/concepts/matestack/ui/core/page/page.rb

@@ -5,6 +5,7 @@ class Page < Trailblazer::Cell
     include ::Cell::Haml
     include Matestack::Ui::Core::ApplicationHelper
     include Matestack::Ui::Core::ToCell
+    include Matestack::Ui::Core::HasViewContext
 
     view_paths << "#{Matestack::Ui::Core::Engine.root}/app/concepts"
 

app/helpers/matestack/ui/core/application_helper.rb

@@ -2,11 +2,13 @@ module Matestack
   module Ui
     module Core
       module ApplicationHelper
+        include Matestack::Ui::Core::Render
 
         def render_page(page_class, only_page=false)
           page_class.new(nil, context: {
             params: params,
-            request: request
+            request: request,
+            view_context: view_context
           }, controller_instance: self).call(:show, nil, only_page)
         end
 

app/lib/matestack/ui/core/component_node.rb

@@ -21,7 +21,11 @@ def initialize(component_instance, included_config)
 
     def method_missing meth, *args, &block
       begin
-        @component_instance.send(meth, *args, &block)
+        if (result = @component_instance.send(meth, *args, &block)).kind_of? ActiveSupport::SafeBuffer
+          plain result
+        else
+          result
+        end
       rescue
         node_id = @node_start_id + 1
         @node_start_id = node_id

app/lib/matestack/ui/core/has_view_context.rb

@@ -0,0 +1,10 @@
+module Matestack::Ui::Core::HasViewContext
+  def initialize(model = nil, options = {})
+    @view_context = options[:context][:view_context]
+    super
+  end
+
+  def method_missing(*args, &block)
+    @view_context.send(*args, &block)
+  end
+end

app/lib/matestack/ui/core/page_node.rb

@@ -22,7 +22,11 @@ def initialize(page_instance, included_config, url_params)
 
     def method_missing meth, *args, &block
       begin
-        @page_instance.send(meth, *args, &block)
+        if (result = @page_instance.send(meth, *args, &block)).kind_of? ActiveSupport::SafeBuffer
+          plain result
+        else
+          result
+        end
       rescue
         node_id = @node_start_id + 1
         @node_start_id = node_id

app/lib/matestack/ui/core/render.rb

@@ -0,0 +1,89 @@
+module Matestack::Ui::Core::Render
+
+  # Matestack allows you to use `render` to render matestack pages.
+  #
+  #     render Pages::Member::Bookings::Index
+  #     render matestack: Pages::Member::Bookings::Index
+  #     render matestack: 'member/bookings/index'
+  #
+  def render(*args)
+    if (matestack_class = args.first).is_a?(Class) && (matestack_class < Matestack::Ui::Page)
+      responder_for matestack_class
+    elsif (options = args.first).kind_of?(Hash) && (matestack_arg = options[:matestack]).present?
+      if (matestack_path = matestack_arg).kind_of? String
+        matestack_path = "pages/#{matestack_path}" unless matestack_path.start_with?("pages/") or matestack_path.start_with?("components/")
+        matestack_class = matestack_path.split("/").collect { |str| str.camelcase }.join("::").constantize
+      elsif matestack_arg.is_a?(Class) && (matestack_arg < Matestack::Ui::Page)
+        matestack_class = matestack_arg
+      end
+      responder_for matestack_class
+    else
+      super
+    end
+  end
+
+  # Matestack allows implicit rendering. The matestack page class name is inferred from the
+  # controller path and action name.
+  #
+  #     class Clients::BookingsController < ApplicationController
+  #       def index
+  #         @bookings = Booking.all
+  #         # looks for Pages::Clients::Bookings::Index
+  #       end
+  #
+  #       def show
+  #         @booking = Booking.find params[:id]
+  #         # looks for Pages::Clients::Bookings::Show
+  #       end
+  #     end
+  #
+  # In this example, `clients/bookings#index` will render `Pages::Clients::Bookings::Index`,
+  # `clients/bookings#show` will render `Pages::Clients::Bookings::Show`.
+  #
+  # Custom action names translate also into page names.
+  #
+  #     class Clients::BookingsController < ApplicationController
+  #       def step1
+  #         # looks for Pages::Clients::Bookings::Step1
+  #       end
+  #     end
+  #
+  # In this example, the `clients/bookings#step1` action will render
+  # `Pages::Clients::Bookings::Step1`.
+  #
+  def default_render(*args)
+    if matestack_page_class = default_render_matestack_page_class
+      render matestack: matestack_page_class
+    else
+      super
+    end
+  end
+
+  def possible_default_render_matestack_page_paths
+    paths = []
+    paths << "pages/#{controller_path}/#{action_name}"
+    paths << "pages/#{controller_path}" if action_name == "index"
+    paths << "pages/#{controller_path.singularize}" if action_name == "show"
+    paths << "#{controller_path}/#{action_name}_page"
+    paths << "#{controller_path}_page" if action_name == "index"
+    paths << "#{controller_path.singularize}_page" if action_name == "show"
+    paths
+  end
+
+  def possible_default_render_matestack_page_class_names
+    possible_default_render_matestack_page_paths.collect { |page_path|
+      page_path.split("/").collect { |str| str.camelcase }.join("::")
+    }
+  end
+
+  def default_render_matestack_page_class
+    possible_default_render_matestack_page_class_names.each do |class_name|
+      begin
+        return matestack_class = class_name.constantize
+      rescue NameError
+      end
+    end
+    return nil
+  end
+
+end