This project is archived and is in readonly mode.
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>
José Valim
Pratik