Skip to content

fix overwriting lambda with evaluated value - #55

Open
aserafin wants to merge 1 commit into
zachinglis:masterfrom
aserafin:fix_overwriting_lambda_in_name
Open

aserafin wants to merge 1 commit into
zachinglis:masterfrom
aserafin:fix_overwriting_lambda_in_name

Conversation

@aserafin

Copy link
Copy Markdown

This line

name = name.call(instance) if name.is_a?(Proc) 

overwrites procedure passed as name parameter to add_crumb call so it is invoked only once and any subsequent calls to controller using this method will return evaluated value.

@andrew

andrew commented Dec 17, 2013

Copy link
Copy Markdown
Contributor

Would you be able to add a regression test for this as well, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants