This project is archived and is in readonly mode.
![José Valim José Valim](https://entp-lh-avatar-production.s3.amazonaws.com/avatars/19965/large.jpg?AWSAccessKeyId=AKIAJ4QBZRZBVMOUBNZA&Expires=2014394578&Signature=JD%2FUdJNGQzBhXVMsly7d4EQMNr0%3D)
Added more support to ActionView::Base.erb_variable
Reported by José Valim | June 2nd, 2008 @ 12:26 PM
If we change ActionView::Base.erb_variable, cache_fragment doesn't work and some javascript functions also, because they are checking for the existence of '_erbout' instead of the variable set in erb_variable.
It's just a small patch, that fix javascript functions and send erb_variable to Erb.new().
I also added the line below in fragment cache tests setup to assure that they are working with any variable set.
ActionView::Base.erb_variable = '_buf'
Comments and changes to this ticket
-
Pratik June 2nd, 2008 @ 01:24 PM
- State changed from new to incomplete
Getting 25 actionpack failures/errors.
-
José Valim June 2nd, 2008 @ 01:47 PM
Oh, I forgot to add a teardown at the end of tests.
Attached another patch. Applying both should work! =)
-
Pratik June 2nd, 2008 @ 01:54 PM
- State changed from incomplete to open
- Assigned user set to Pratik
-
Pratik June 17th, 2008 @ 06:54 PM
- State changed from open to invalid
- Milestone cleared.
Not relevant anymore as we got rid of _erbout variable.
Thanks.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>