aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Remove wrong rack/utils dependency from ASSantiago Pastorino2012-05-121-3/+3
| | | | | | | | | | | | Closes #6274
* | | Merge pull request #6281 from acapilleri/mispelling_errors_in_render_text_testVijay Dev2012-05-122-3/+3
|\ \ \ | | | | | | | | mispelling errors in render_text_test.rb
| * | | mispelling errors in render_text_test.rb and sqlite_specific_schemaangelo giovanni capilleri2012-05-122-3/+3
| | | |
* | | | Merge pull request #6278 from GRoguelon/masterVijay Dev2012-05-122-2/+2
|\ \ \ \ | | | | | | | | | | Add commas missing in performance tests
| * | | | Add commas missing in performance testsGeoffrey Roguelon2012-05-122-2/+2
| | |/ / | |/| |
* | | | Merge pull request #6280 from nashby/private-tryJon Leighton2012-05-123-3/+17
|\ \ \ \ | |_|/ / |/| | | Object#try can't call private methods
| * | | Object#try can't call private methodsVasiliy Ermolovich2012-05-123-3/+17
| |/ /
* | | Merge pull request #6142 from spartan-developer/masterJosé Valim2012-05-124-12/+10
|\ \ \ | | | | | | | | refactored extracting :size => 'XxY' into an extract_size! method
| * | | refactored extracting :size => 'XxY' into an extract_size! methodNicholas Yianilos2012-05-034-12/+10
| | | |
* | | | Merge pull request #6279 from ↵Xavier Noria2012-05-121-1/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | acapilleri/mispelling_error_in_actionpach_changelog mispelling error in actionpach changelog
| * | | mispelling error in actionpach changelogangelo giovanni capilleri2012-05-121-1/+1
|/ / /
* | | Remove #=== quirkJon Leighton2012-05-113-17/+0
| | | | | | | | | | | | Makes it consistent with Relation. Can't see a use for this.
* | | Add test directory to TestTask's libsSantiago Pastorino2012-05-111-0/+1
| | |
* | | Remove useless load path modificationsSantiago Pastorino2012-05-1112-38/+4
| | |
* | | Lazy load I18nSantiago Pastorino2012-05-114-6/+10
| | |
* | | Move require to where it's neededSantiago Pastorino2012-05-116-3/+6
| | |
* | | Remove ruby/shimSantiago Pastorino2012-05-113-17/+1
| | | | | | | | | | | | require 'action_view' is now much faster
* | | this stuff can all be handled by RelationJon Leighton2012-05-111-38/+0
| | |
* | | extract deprecated codeJon Leighton2012-05-111-8/+1
| | |
* | | set_owner_attributes is covered by the scopingJon Leighton2012-05-111-4/+3
| | |
* | | CollectionProxy < RelationJon Leighton2012-05-1111-60/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This helps bring the interfaces of CollectionProxy and Relation closer together, and reduces the delegation backflips we need to perform. For example, first_or_create is defined thus: class ActiveRecord::Relation def first_or_create(...) first || create(...) end end If CollectionProxy < Relation, then post.comments.first_or_create will hit the association's #create method which will actually add the new record to the association, just as post.comments.create would. With the previous delegation, post.comments.first_or_create expands to post.comments.scoped.first_or_create, where post.comments.scoped has no knowledge of the association.
* | | Use respond_to?(:to_ary) rather than is_a?(Enumerable) to detect ↵Jon Leighton2012-05-114-3/+40
| | | | | | | | | | | | collection-thing.
* | | Merge pull request #6269 from edtsech/hash-except-testXavier Noria2012-05-111-0/+6
|\ \ \ | | | | | | | | Add test Hash#except can receive more than one argument.
| * | | Test Hash#except can receive more than one argument.Edward Tsech2012-05-111-0/+6
| | | |
* | | | Merge pull request #6268 from carlesso/masterJosé Valim2012-05-112-2/+2
|\ \ \ \ | |/ / / |/| | | In robots.txt User-agent should be all downcase except for the first 'U'
| * | | In robots.txt, User-agent should be all downcase except for the first 'U', ↵Enrico Carlesso2012-05-112-2/+2
|/ / / | | | | | | | | | according with http://en.wikipedia.org/wiki/Robots_exclusion_standard
* | | Copy literal route constraints to defaults - fixes #3571 and #6224.Andrew White2012-05-114-3/+69
| | |
* | | Fixes issue #6251David Padilla2012-05-101-1/+1
| | | | | | | | | | | | | | | | | | | | | Plugin generator crashes when using the --dummy-path option Code was assuming the application name in `config/application.rb` was module Dummy.
* | | Failing test for #6251Piotr Sarnacki2012-05-101-0/+7
| | |
* | | Return false for exists? with new records - fixes #6199.Andrew White2012-05-103-2/+3
| | |
* | | Don't ignore nil positional arguments for url helpers - fixes #6196.Andrew White2012-05-102-1/+36
| | |
* | | Refactor the handling of default_url_options in integration testsAndrew White2012-05-102-7/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit improves the handling of default_url_options in integration tests by making behave closer to how a real application operates. Specifically the following issues have been addressed: * Options specified in routes.rb are used (fixes #546) * Options specified in controllers are used * Request parameters are recalled correctly * Tests can override default_url_options directly
* | | remove unnecessary 'examples' noise [ci skip]Vijay Dev2012-05-112-28/+18
| | |
* | | Merge pull request #6253 from rafmagana/masterJosé Valim2012-05-101-1/+1
|\ \ \ | | | | | | | | remove duplicate usage of Rack::Utils.status_code in LogSubscriber#process_action
| * | | remove duplicate usage of Rack::Utils.status_code in ↵Rafael Magana2012-05-101-1/+1
| | | | | | | | | | | | | | | | LogSubscriber#process_action
* | | | Merge pull request #6252 from grosser/unless_exist_for_memory_storeJosé Valim2012-05-102-0/+8
|\ \ \ \ | |/ / / |/| | | add unless_exist option to memory store
| * | | add unless_exist option to memory storegrosser2012-05-102-0/+8
|/ / /
* | | Merge pull request #6244 from frodsan/fix_build_remove_keyJosé Valim2012-05-101-1/+1
|\ \ \ | | | | | | | | Remove key argument from run_callbacks to fix build
| * | | Remove key argument from run_callbacks to fix buildFrancesco Rodriguez2012-05-101-1/+1
|/ / /
* | | Merge pull request #6242 from frodsan/fix_abstract_controllerJosé Valim2012-05-101-1/+1
|\ \ \ | | | | | | | | Removing key argument from run_callbacks method in abstract_controller
| * | | removing key argument from run_callbacks - fix buildFrancesco Rodriguez2012-05-101-1/+1
|/ / /
* | | Merge pull request #6241 from frodsan/deprecate_key_callbacksJosé Valim2012-05-104-6/+7
|\ \ \ | | | | | | | | AS::Callbacks#run_callbacks remove key argument
| * | | AS::Callbacks#run_callbacks remove key argumentFrancesco Rodriguez2012-05-104-6/+7
|/ / /
* | | Merge pull request #4254 from alexeymuranov/my-nicify-fixtures-rbJeremy Kemper2012-05-092-81/+82
|\ \ \ | | | | | | | | Rename variables and remove unused code in fixtures.rb
| * | | Rename some variablesAlexey Muranov2012-05-092-78/+82
| | | | | | | | | | | | | | | | | | | | | | | | Rename some parameters and instance and local variables, mostly in fixtures.rb. Also remove an unused assignment to an instance variable. There are minor code changes.
| * | | Remove unused private method in fixtures.rbAlexey Muranov2012-05-091-3/+0
| | | | | | | | | | | | | | | | Remove the unused private method ActiveRecord::Fixtures#yaml_fixtures_key(path).
| * | | Remove unused parameter in ::instantiate_fixturesAlexey Muranov2012-05-091-5/+5
| | | | | | | | | | | | | | | | Second parameter of Fixtures::instantiate_fixtures was not used, so i removed it.
* | | | Allow to run `connection_adapters/quoting_test.rb` independentlyPiotr Sarnacki2012-05-091-1/+3
| | | |
* | | | Merge pull request #5362 from zenprogrammer/quoting_bugAaron Patterson2012-05-092-0/+12
|\ \ \ \ | | | | | | | | | | Fixed bug in ActiveRecord that caused classes to be quoted incorrectly
| * | | | Fixed bug in Quoting that caused classes to be quoted incorrectlyRyan Oblak2012-03-092-0/+12
| | | | |