aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* 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>
* Add namespace for test_unit generatorsPiotr Sarnacki2010-09-254-3/+9
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Allow per Resource format settingsJacques Crocker2010-09-255-17/+25
| | | | | | | | | | Previously, ActiveResource was using the connection level formatter for get requests. This made it impossible to use custom formatters per resource. Additionally this commit makes the Connection request methods more consistent. It always returns a Response. The base will then decode it each the response using its format setting. Merging this commit will allow users to add custom formatters on a per Resource basis. This enables handling pagination responses from the server side, a very common use case that was previously impossible without monkeypatching XmlFormat. Signed-off-by: José Valim <jose.valim@gmail.com>
* Allow ActiveResource to work with non-generated ids [#5660 state:resolved]Jacques Crocker2010-09-255-5/+80
| | | | | | This commit updates new? so that it knows whether or not the record was actually new or not, and doesn't rely solely on the presence of id. This enables the ability to set a custom primary_key that is not autogenerated by the server. Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix 'warning: method redefined' [#5551 state:resolved]Krekoten' Marjan2010-09-251-4/+4
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix logging when cache key contains % sign [#5570 state:resolved]Krekoten' Marjan2010-09-252-1/+17
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Remove duplicate helper methodKrekoten' Marjan2010-09-251-3/+0
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Update abort message (ht: tilsammans).José Valim2010-09-251-1/+1
|
* Don't act destructively on ActiveModel::Name#human options hash. [#5366 ↵John Firebaugh2010-09-253-2/+13
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't duplicate Rack::Response functionality. [#5320 state:resolved]John Firebaugh2010-09-251-18/+4
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add ActionDispatch::TestResponse tests.John Firebaugh2010-09-251-0/+21
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Rails.application is set anyway, we don't need to set it manually, this was ↵Piotr Sarnacki2010-09-252-3/+1
| | | | | | purpose of different behavior in tests and in application Signed-off-by: José Valim <jose.valim@gmail.com>
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-09 04:17:47 +0000