Skip to content
This repository has been archived by the owner on Aug 26, 2024. It is now read-only.

Adjust the arguments of find_templates() for Rails 4.2.5.2, 4.1.14.2 and 3.2.22.2 #51

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

minamijoyo
Copy link
Contributor

Rails 4.2.5.2, 4.1.14.2 and 3.2.22.2 have been released and these include a break change for find_templates().
Rendering templates causes an error such as ArgumentError: wrong number of arguments (5 for 4)
This patch adjusts the arguments of find_templates().

refs: http://weblog.rubyonrails.org/2016/2/29/Rails-4-2-5-2-4-1-14-2-3-2-22-2-have-been-released/

see also:
rails/rails@v4.2.5.1...v4.2.5.2
rails/rails@v4.1.14.1...v4.1.14.2
rails/rails@v3.2.22.1...v3.2.22.2

…and 3.2.22.2

Rails 4.2.5.2, 4.1.14.2 and 3.2.22.2 have been released and these include a break change for find_templates().
This patch adjusts the arguments of find_templates().

refs: http://weblog.rubyonrails.org/2016/2/29/Rails-4-2-5-2-4-1-14-2-3-2-22-2-have-been-released/
@minamijoyo minamijoyo changed the title Adjust the arguments of find_templates() for Rails 4.2.5.2, 4.1.14.2 …and 3.2.22.2 Adjust the arguments of find_templates() for Rails 4.2.5.2, 4.1.14.2 and 3.2.22.2 Mar 1, 2016
@@ -39,13 +39,13 @@ def replace_fallback_formats_chain(formats)
# Private: finds the right template on the filesystem,
# using fallback if needed
#
def find_templates(name, prefix, partial, details)
def find_templates(name, prefix, partial, details, outside_app_allowed = false)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants