Use controller_path instead of controller_name for template path resolving. Makes partials in namespaces work.

Author: tomash

lib/liquid-rails/file_system.rb

@@ -4,11 +4,10 @@ module Liquid
   module Rails
     class FileSystem < ::Liquid::LocalFileSystem
       def read_template_file(template_path, context)
-        controller_name = context.registers[:controller].controller_name
-        template_path   = "#{controller_name}/#{template_path}" unless template_path.include?('/')
-
+        controller_path = context.registers[:controller].controller_path
+        template_path   = "#{controller_path}/#{template_path}" unless template_path.include?('/')
         super
       end
     end
   end
-end
+end

spec/dummy/app/controllers/foospace/bar_controller.rb

@@ -0,0 +1,5 @@
+class Foospace::BarController < ApplicationController
+  def index_partial
+    render layout: false
+  end
+end

spec/dummy/app/views/foospace/bar/_partial.liquid

@@ -0,0 +1 @@
+Bar Partial

spec/dummy/app/views/foospace/bar/index_partial.liquid

@@ -0,0 +1,3 @@
+Foospace::BarController
+
+{% include 'partial' %}

spec/dummy/config/routes.rb

@@ -7,4 +7,6 @@
 
   get  '/index_partial',                to: 'home#index_partial'
   get  '/index_partial_with_full_path', to: 'home#index_partial_with_full_path'
-end
+
+  get '/foospace/bar/index_partial',    to: 'foospace/bar#index_partial'
+end

spec/lib/liquid-rails/template_handler_spec.rb

@@ -35,6 +35,12 @@
 
       expect(page.body).to eq "Application Layout\nLiquid on Rails\n\nHome Partial\nShared Partial"
     end
+
+    it 'respects namespace of original template for partials path' do
+      visit '/foospace/bar/index_partial'
+      expect(page.body.strip).to eq("Foospace::BarController\n\nBar Partial")
+    end
+
   end
 
   context 'render with filter' do
@@ -50,4 +56,4 @@
       expect(page.body).to eq("Application Layout\nLiquid on Rails\nThis is a long section of text")
     end
   end
-end
+end