aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Prevent shadowing outer local variable.Emilio Tagua2010-09-271-3/+3
| | |
* | | Call it compile_methods! and do the same on AM.José Valim2010-09-274-6/+16
| | |
* | | adding a test for slug behaviorAaron Patterson2010-09-271-0/+4
| | |
* | | Merge remote branch 'thedarkone/performance'José Valim2010-09-2716-76/+179
|\ \ \
| * | | Fix Namind#model_name.thedarkone2010-09-271-2/+4
| | | |
| * | | Assume compute_asset_host returns reasonable values.thedarkone2010-09-271-1/+1
| | | |
| * | | Convert unless/else into if/else.thedarkone2010-09-271-4/+4
| | | |
| * | | Performance: refactor convert_options_to_data_attributes.thedarkone2010-09-271-10/+14
| | | |
| * | | Remove dead code.thedarkone2010-09-271-2/+2
| | | |
| * | | Clean up url_for.thedarkone2010-09-271-3/+1
| | | |
| * | | Hash#empty? is faster than Enumerable#any? when used on a Hash.thedarkone2010-09-271-1/+1
| | | |
| * | | Do a single string interpolation.thedarkone2010-09-271-1/+1
| | | |
| * | | Simple .empty? test will do fine here (rails_asset_id returns nice strings).thedarkone2010-09-271-1/+1
| | | |
| * | | options[:action] is very likely to be nil.thedarkone2010-09-271-1/+1
| | | |
| * | | Make assert_valid_keys slightly faster.thedarkone2010-09-272-5/+7
| | | |
| * | | No need for an extra wrapper array.thedarkone2010-09-271-1/+1
| | | |
| * | | Cache 2 of Request's commonly called methods.thedarkone2010-09-271-2/+2
| | | |
| * | | Cache url_options on a per-request basis.thedarkone2010-09-271-9/+11
| | | |
| * | | No need to create a separate lambda for each call.thedarkone2010-09-271-15/+14
| | | |
| * | | Make asset extension rewriting faster.thedarkone2010-09-271-5/+11
| | | |
| * | | Optimize relative_url_root rewriting code.thedarkone2010-09-271-3/+5
| | | |
| * | | Use native attr_* macros for performance reasons.thedarkone2010-09-271-6/+13
| | | |
| * | | Creating singleton class for every object whenever the instance-level ↵thedarkone2010-09-271-1/+21
| | | | | | | | | | | | | | | | accessor is used quite is expensive.
| * | | Try to use Hash's native #[] for speed.thedarkone2010-09-272-2/+8
| | | |
| * | | Make InheritableOptions's constructor more flexible.thedarkone2010-09-272-3/+7
| | | |
| * | | Compile ActionController::Base.config's methods to avoid method_missing ↵thedarkone2010-09-275-4/+55
| |/ / | | | | | | | | | overhead.
* | / Three performance improvements:Neeraj Singh2010-09-271-10/+9
| |/ |/| | | | | | | | | | | | | * for simple cases like User.last and User.order('name desc').last no need to perform Array#join operation. * Instead of performing String#blank? do Array#empty? * no need to create variable relation
* | depending on arel 2.0.0Aaron Patterson2010-09-271-1/+1
|/
* third parameter is not usedAaron Patterson2010-09-261-1/+1
|
* db:structure:dump should list current Rails.env adapter in errors, not ↵Nic Benders2010-09-261-1/+1
| | | | | | | | always the test adapter [#5710 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Fix remove_index issue when provided :name is a symbolTim Connor2010-09-262-2/+2
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Fix the precedence issue hereSantiago Pastorino2010-09-261-1/+1
|
* Not need to do this double ternarySantiago Pastorino2010-09-261-1/+1
|
* port is appended twice to HTTP_HOST when host already has the portSantiago Pastorino2010-09-261-3/+3
|
* Bump up some depsSantiago Pastorino2010-09-264-5/+5
|
* Partial revert of #d650b71 'Remove deprecated stuff in ActionController'Carlos Antonio da Silva2010-09-264-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 Silva2010-09-261-8/+7
| | | | does it internally
* More cleanup on form helpersCarlos Antonio da Silva2010-09-261-11/+12
|
* Remove last tests with deprecated form_for and cleanup form_for helperCarlos Antonio da Silva2010-09-262-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, removingCarlos Antonio da Silva2010-09-262-10/+0
|
* Refactor form_for helperCarlos Antonio da Silva2010-09-261-21/+11
|
* Review most of the form_for deprecated tests, missing tests with index like []Carlos Antonio da Silva2010-09-262-405/+322
|
* Remove deprecated stuff in ActionControllerCarlos Antonio da Silva2010-09-2629-58/+47
| | | | | | 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 Silva2010-09-261-27/+20
|
* There is no DeprecatedBlockHelpers in AV, so remove itCarlos Antonio da Silva2010-09-261-1/+0
|
* Fix active_resource assertions.José Valim2010-09-251-2/+2
|
* Fix app and actions generators testsPiotr Sarnacki2010-09-252-0/+9
| | | | | | | Normally Rails.application is an instance, but for those tests Rails.application needs to be class. Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor a few methods connected with namespacing in ↵Piotr Sarnacki2010-09-251-8/+9
| | | | | | Rails::Generators::NamedBase Signed-off-by: José Valim <jose.valim@gmail.com>
* Add namespacing to mailer generatorPiotr Sarnacki2010-09-253-0/+56
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add namespacing for observer generatorPiotr Sarnacki2010-09-253-0/+26
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>