Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Clean up the house before moving in the new furniture. | José Valim | 2010-10-10 | 9 | -305/+341 |
| | | | | This commit moves all the template rendering logic that was hanging around AV::Base to renderer objects. | ||||
* | Resolvers now consider timestamps. | José Valim | 2010-10-10 | 3 | -35/+62 |
| | | | | | | | | | | | Before this patch, every request in development caused the template to be compiled, regardless if it was updated in the filesystem or not. This patch now checks the timestamp and only compiles it again if any change was done. While this probably won't show any difference for current setups, but it will be useful for asset template handlers (like SASS), as compiling their templates is slower than ERb, Haml, etc. | ||||
* | Deprecate old template handler API. Remove old handlers. | José Valim | 2010-10-10 | 5 | -30/+35 |
| | |||||
* | Add more docs and tests to templates. | José Valim | 2010-10-09 | 1 | -41/+54 |
| | |||||
* | Fix example that became outdated after a code change. | Andrea Campi | 2010-10-09 | 1 | -1/+1 |
| | | | | | | [#5770 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | fix rendering a partial with an array as its :object [#5746 state:resolved] | Michael Koziarski | 2010-10-08 | 1 | -3/+7 |
| | | | | | | | | Signed-off-by: Michael Koziarski <michael@koziarski.com> Conflicts: actionpack/lib/action_view/render/partials.rb | ||||
* | Allow cache to be temporarily disabled through lookup_context. | José Valim | 2010-10-07 | 3 | -5/+20 |
| | |||||
* | render :template => 'foo/bar.json' now works as it should. | José Valim | 2010-10-07 | 3 | -11/+28 |
| | |||||
* | Make collection rendering faster. | José Valim | 2010-10-07 | 1 | -2/+2 |
| | |||||
* | Remove locals dependency from template. | José Valim | 2010-10-07 | 8 | -85/+129 |
| | | | | | | This means that templates does not need to store its source anymore, allowing us to reduce the ammount of memory taken by our Rails processes. Naively speaking, if your app/views contains 2MB of files, each of your processes (after being hit by a bunch of requests) will take 2MB less of memory after this commit. This is extremely important for the upcoming features. Since Rails will also render CSS and JS files, their source won't be stored as well allowing us to decrease the ammount of memory taken. | ||||
* | Get rid of ruby warnings in Resolvers. Move a few methods up to the abstract ↵ | José Valim | 2010-10-07 | 2 | -23/+26 |
| | | | | class. | ||||
* | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2010-10-06 | 1 | -2/+3 |
|\ | |||||
| * | Template::Error is also used if rendering fails. | John Firebaugh | 2010-10-04 | 1 | -2/+3 |
| | | |||||
* | | refactoring to use fewer intermediate variables | Aaron Patterson | 2010-10-04 | 1 | -11/+12 |
| | | |||||
* | | cleaning up more crazy! | Aaron Patterson | 2010-10-04 | 1 | -3/+3 |
| | | |||||
* | | deleting more crazy | Aaron Patterson | 2010-10-04 | 1 | -6/+4 |
| | | |||||
* | | dry up some crazy codes | Aaron Patterson | 2010-10-04 | 1 | -8/+6 |
| | | |||||
* | | two argument String#slice is faster than single argument, also avoid ↵ | Aaron Patterson | 2010-10-03 | 1 | -1/+1 |
| | | | | | | | | creating a Range object | ||||
* | | PERF: Hash[] + map is faster than this silly inject, and var[1..-1] is ↵ | Santiago Pastorino | 2010-10-03 | 1 | -3/+3 |
| | | | | | | | | faster than var.sub('@', '') | ||||
* | | Rename _assigns to view_assigns in AV::TC | David Chelimsky | 2010-10-03 | 1 | -9/+21 |
| | | | | | | | | | | | | | | | | | | - also add tests - also deprecate _assigns [#5751 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | | Fixing search_field to remove object attribute from options hash [#5730 ↵ | Aditya Sanghi | 2010-10-02 | 1 | -1/+1 |
| | | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | | Revert "Perf: refactor _assign method to avoid inject and defining unneeded ↵ | José Valim | 2010-10-02 | 1 | -1/+5 |
| | | | | | | | | | | | | | | | | local var." _assigns must return a hash. This reverts commit e66c1cee86aba1c81152f3d0872313e65cec85a9. | ||||
* | | Revert "removing crazy finalizer code until there is proof that we need it" | José Valim | 2010-09-30 | 1 | -0/+11 |
| | | | | | | | | This reverts commit a40e3c1a9604ab3737ad2465c8f6a6db0fe0cc78. | ||||
* | | tag value can be false, so nil? check is necessary | Aaron Patterson | 2010-09-29 | 1 | -1/+1 |
| | | |||||
* | | another case of extra nil? check | Neeraj Singh | 2010-09-30 | 1 | -1/+1 |
| | | |||||
* | | no need to check for nil? | Neeraj Singh | 2010-09-30 | 1 | -2/+2 |
| | | |||||
* | | removing crazy finalizer code until there is proof that we need it | Aaron Patterson | 2010-09-29 | 1 | -9/+0 |
| | | |||||
* | | removing useless code | Aaron Patterson | 2010-09-29 | 1 | -1/+0 |
|/ | |||||
* | Avoid (@_var ||= nil) pattern by using initialize methods and ensuring ↵ | José Valim | 2010-09-29 | 2 | -25/+28 |
| | | | | everyone calls super as expected. | ||||
* | Merge remote branch 'miloops/warnings' | José Valim | 2010-09-29 | 4 | -3/+11 |
|\ | |||||
| * | Initialize @path. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Ask if the instance variable is defined before asking for it, avoid *many* ↵ | Emilio Tagua | 2010-09-28 | 1 | -1/+5 |
| | | | | | | | | warnings. | ||||
| * | Initialize @sub_templates | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Don't define _test_case method if already defined. | Emilio Tagua | 2010-09-28 | 1 | -2/+4 |
| | | |||||
* | | convert inject to map + join | Aaron Patterson | 2010-09-28 | 1 | -5/+5 |
|/ | |||||
* | Merge remote branch 'miloops/warnings' | José Valim | 2010-09-27 | 4 | -5/+7 |
|\ | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/url_for.rb | ||||
| * | Remove method previous method if already defined. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | |||||
| * | Define @emitted_hidden_id if it doesn't exists and reuse it if it does. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | |||||
| * | Avoid uninitialized variable warning. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | |||||
| * | Don't shadow outer local variables. | Emilio Tagua | 2010-09-27 | 1 | -4/+4 |
| | | |||||
* | | Assume compute_asset_host returns reasonable values. | thedarkone | 2010-09-27 | 1 | -1/+1 |
| | | |||||
* | | Performance: refactor convert_options_to_data_attributes. | thedarkone | 2010-09-27 | 1 | -10/+14 |
| | | |||||
* | | Remove dead code. | thedarkone | 2010-09-27 | 1 | -2/+2 |
| | | |||||
* | | Clean up url_for. | thedarkone | 2010-09-27 | 1 | -3/+1 |
| | | |||||
* | | Do a single string interpolation. | thedarkone | 2010-09-27 | 1 | -1/+1 |
| | | |||||
* | | Simple .empty? test will do fine here (rails_asset_id returns nice strings). | thedarkone | 2010-09-27 | 1 | -1/+1 |
| | | |||||
* | | Make asset extension rewriting faster. | thedarkone | 2010-09-27 | 1 | -5/+11 |
| | | |||||
* | | Optimize relative_url_root rewriting code. | thedarkone | 2010-09-27 | 1 | -3/+5 |
| | | |||||
* | | Compile ActionController::Base.config's methods to avoid method_missing ↵ | thedarkone | 2010-09-27 | 1 | -2/+1 |
|/ | | | | overhead. | ||||
* | We don't need to check if the format exists to generate the url, it already ↵ | Carlos Antonio da Silva | 2010-09-26 | 1 | -8/+7 |
| | | | | does it internally |