Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | Don't shadow outer local variables. | Emilio Tagua | 2010-09-27 | 2 | -5/+5 | |
| | | ||||||
| * | Remove duplicated test. | Emilio Tagua | 2010-09-27 | 1 | -8/+0 | |
| | | ||||||
| * | Avoid uninitialized variable warning, reuse @integration_session. | Emilio Tagua | 2010-09-27 | 1 | -8/+14 | |
| | | ||||||
| * | Ask is @controller is defined to avoid warning. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 | |
| | | ||||||
| * | Initialize @_routes if it doesn't exists. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 | |
| | | ||||||
| * | Remove warning "too many arguments for format string" when interpolating ↵ | Emilio Tagua | 2010-09-27 | 1 | -1/+1 | |
| | | | | | | | | with empty hash. | |||||
| * | Initialize @_etag. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 | |
| | | ||||||
| * | Initialize @as before plural method is called. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 | |
| | | ||||||
| * | Remove old method before redefining it. | Emilio Tagua | 2010-09-27 | 1 | -0/+2 | |
| | | ||||||
| * | Remove warning "URI.unescape is obsolete" from actionpack. | Emilio Tagua | 2010-09-27 | 5 | -14/+34 | |
| | | ||||||
| * | Fix indentation. | Emilio Tagua | 2010-09-27 | 1 | -19/+18 | |
| | | ||||||
| * | Use parentheses when using assert_match followed by a regexp to avoid warnings. | Emilio Tagua | 2010-09-27 | 17 | -58/+58 | |
| | | ||||||
* | | Call it compile_methods! and do the same on AM. | José Valim | 2010-09-27 | 1 | -2/+2 | |
| | | ||||||
* | | Assume compute_asset_host returns reasonable values. | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | ||||||
* | | Convert unless/else into if/else. | thedarkone | 2010-09-27 | 1 | -4/+4 | |
| | | ||||||
* | | 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 | |
| | | ||||||
* | | Hash#empty? is faster than Enumerable#any? when used on a Hash. | thedarkone | 2010-09-27 | 1 | -1/+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 | |
| | | ||||||
* | | options[:action] is very likely to be nil. | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | ||||||
* | | Cache 2 of Request's commonly called methods. | thedarkone | 2010-09-27 | 1 | -2/+2 | |
| | | ||||||
* | | Cache url_options on a per-request basis. | thedarkone | 2010-09-27 | 1 | -9/+11 | |
| | | ||||||
* | | No need to create a separate lambda for each call. | thedarkone | 2010-09-27 | 1 | -15/+14 | |
| | | ||||||
* | | 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 | 2 | -2/+11 | |
|/ | | | | overhead. | |||||
* | Fix the precedence issue here | Santiago Pastorino | 2010-09-26 | 1 | -1/+1 | |
| | ||||||
* | Not need to do this double ternary | Santiago Pastorino | 2010-09-26 | 1 | -1/+1 | |
| | ||||||
* | port is appended twice to HTTP_HOST when host already has the port | Santiago Pastorino | 2010-09-26 | 1 | -3/+3 | |
| | ||||||
* | Bump up some deps | Santiago Pastorino | 2010-09-26 | 1 | -2/+2 | |
| | ||||||
* | Partial revert of #d650b71 'Remove deprecated stuff in ActionController' | Carlos Antonio da Silva | 2010-09-26 | 4 | -0/+11 | |
| | | | | | This brings back the deprecated modules from ActionController, because they didn't have any deprecation warning. | |||||
* | 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 | 2 | -67/+51 | |
| | | | | | | | 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 | 2 | -10/+0 | |
| | ||||||
* | Refactor form_for helper | Carlos Antonio da Silva | 2010-09-26 | 1 | -21/+11 | |
| | ||||||
* | Review most of the form_for deprecated tests, missing tests with index like [] | Carlos Antonio da Silva | 2010-09-26 | 2 | -405/+322 | |
| | ||||||
* | Remove deprecated stuff in ActionController | Carlos Antonio da Silva | 2010-09-26 | 25 | -50/+39 | |
| | | | | | | This removes all deprecated classes in ActionController related to Routing, Abstract Request/Response and Integration/IntegrationTest. All tests and docs were changed to ActionDispatch instead of ActionController. | |||||
* | There is no need to open AC::Base three times to setup tests. | Carlos Antonio da Silva | 2010-09-26 | 1 | -27/+20 | |
| | ||||||
* | There is no DeprecatedBlockHelpers in AV, so remove it | Carlos Antonio da Silva | 2010-09-26 | 1 | -1/+0 | |
| | ||||||
* | Fix logging when cache key contains % sign [#5570 state:resolved] | Krekoten' Marjan | 2010-09-25 | 2 | -1/+17 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Remove duplicate helper method | Krekoten' Marjan | 2010-09-25 | 1 | -3/+0 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Don't duplicate Rack::Response functionality. [#5320 state:resolved] | John Firebaugh | 2010-09-25 | 1 | -18/+4 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Add ActionDispatch::TestResponse tests. | John Firebaugh | 2010-09-25 | 1 | -0/+21 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | These tests shouldn't depend on local time to pass or fail, we can use utc here. | Emilio Tagua | 2010-09-24 | 2 | -2/+2 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | There is no need to use instance_eval since the proc is created in the same ↵ | José Valim | 2010-09-24 | 1 | -1/+1 | |
| | | | | context it is eval'ed. | |||||
* | Make redirect_to accept blocks [#5643 state:resolved] | Nando Vieira | 2010-09-24 | 2 | -1/+43 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Changed documentation for auto_link so that the truncate example works ↵ | Kevin Hughes | 2010-09-24 | 1 | -1/+1 | |
| | | | | | | [#5694 state:resolved] Signed-off-by: Xavier Noria <fxn@hashref.com> |