Skip to content

Instantly share code, notes, and snippets.

@richseviora
Created April 26, 2017 22:15
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save richseviora/1d01d4457fb7738c513de3be7430d291 to your computer and use it in GitHub Desktop.
Save richseviora/1d01d4457fb7738c513de3be7430d291 to your computer and use it in GitHub Desktop.
Patching ActionView::Digestor.tree to address cross-format dependency resolution.
module ActionView
class Digestor
class << self
def tree(name, finder, partial = false, seen = {})
logical_name = name.gsub(%r|/_|, "/")
options = {}
options[:formats] = [finder.rendered_format] if finder.rendered_format
options[:format].push(:html, :json) if finder.rendered_format == :js
if template = finder.disable_cache { finder.find_all(logical_name, [], partial, [], options).first }
finder.rendered_format ||= template.formats.first
if node = seen[template.identifier] # handle cycles in the tree
node
else
node = seen[template.identifier] = Node.create(name, logical_name, template, partial)
deps = DependencyTracker.find_dependencies(name, template, finder.view_paths)
deps.uniq { |n| n.gsub(%r|/_|, "/") }.each do |dep_file|
node.children << tree(dep_file, finder, true, seen)
end
node
end
else
logger.error " '#{name}' file doesn't exist, so no dependencies"
logger.error " Couldn't find template for digesting: #{name}"
seen[name] ||= Missing.new(name, logical_name, nil)
end
end
end
end
end
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment