Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | ||||
* | More cleanup on form helpers | Carlos Antonio da Silva | 2010-09-26 | 1 | -11/+12 |
| | |||||
* | Remove last tests with deprecated form_for and cleanup form_for helper | Carlos Antonio da Silva | 2010-09-26 | 1 | -4/+2 |
| | | | | | | | This cleans up the last bits of deprecation stuff from form_for helper. However there is still a bug when using :as => foo[], with index. The classes and ids are being generated using [], such as foo[]_edit. This bug already existed but it was not detected before. | ||||
* | Select tags with array options are deprecated, removing | Carlos Antonio da Silva | 2010-09-26 | 1 | -4/+0 |
| | |||||
* | Refactor form_for helper | Carlos Antonio da Silva | 2010-09-26 | 1 | -21/+11 |
| | |||||
* | There is no DeprecatedBlockHelpers in AV, so remove it | Carlos Antonio da Silva | 2010-09-26 | 1 | -1/+0 |
| |