aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #5973 from academia-edu/masterAaron Patterson2012-04-251-1/+9
|\ | | | | Always quote hstore keys and values
| * Always quote hstore keys and valuesRyan Fitzgerald2012-04-241-1/+9
| | | | | | | | | | | | | | escape_hstore uses quotation marks around keys and values only if it seems necessary. However, it currently breaks in the presence of some non-ASCII characters. Instead of trying to guess exactly which characters are safe, it seems better to always use quotes.
* | Merge remote-tracking branch 'kennyj/fix_5563'Jon Leighton2012-04-251-0/+11
|\ \ | | | | | | | | | | | | Conflicts: activerecord/test/cases/associations/belongs_to_associations_test.rb
| * | Fix #5563. Should reflect the most recent change to either of association / id.kennyj2012-04-131-0/+11
| | |
* | | add missing testJon Leighton2012-04-251-0/+12
| | |
* | | extract deprecated codeJon Leighton2012-04-252-7/+2
| | |
* | | Extract deprecated codeJon Leighton2012-04-251-0/+5
| | |
* | | fix interpolation for hash mergingJon Leighton2012-04-251-4/+13
| | |
* | | allow merging a single where valueJon Leighton2012-04-251-0/+6
| | |
* | | giving a hash to default scope should not be deprecated (well, not for this ↵Jon Leighton2012-04-251-2/+2
| |/ |/| | | | | reason)
* | Fix build. It seems that the Mocha's behavior were changed.kennyj2012-04-252-3/+9
| |
* | be sure to currectly fetch PK name from MySQL even if the PK has some custom ↵Akira Matsuda2012-04-191-0/+30
| | | | | | | | option
* | opening a connection will block if the pool is fullAaron Patterson2012-04-151-0/+24
| |
* | Automatically create indexes for references/belongs_to statements in migrations.Joshua Wood2012-04-142-0/+100
| |
* | Merge pull request #5837 from yahonda/new_sequence_name_is_too_longJon Leighton2012-04-141-4/+4
|\ \ | | | | | | Fix ORA-00972 error at test_rename_table_with_prefix_and_suffix
| * | Fix ORA-00972 error at test_rename_table_with_prefix_and_suffixYasuo Honda2012-04-131-4/+4
| | |
* | | fix testsJon Leighton2012-04-137-81/+74
| | |
* | | Merge pull request #5832 from kennyj/fix_5267Jon Leighton2012-04-131-2/+6
|\ \ \ | | | | | | | | Fix a wrong return value from reset_sequence_name method.
| * | | Fix wrong return value from reset_sequence_name method.kennyj2012-04-141-2/+6
| | | |
* | | | now we can just manipulate the values hash in #only and #exceptJon Leighton2012-04-131-0/+13
| | | |
* | | | use a hash to store relation valuesJon Leighton2012-04-131-7/+7
|/ / /
* | | remove apply_finder_options call from AssociationScopeJon Leighton2012-04-131-0/+5
| | |
* | | Make Relation#extending work like other value methodsJon Leighton2012-04-131-8/+9
| | |
* | | Add Relation#merge!Jon Leighton2012-04-131-0/+5
| | |
* | | assert valid keysJon Leighton2012-04-131-0/+4
| | |
* | | Allow Relation#merge to take a hashJon Leighton2012-04-131-2/+20
| | |
* | | we have no need for the ASSOCIATION_METHODS constantJon Leighton2012-04-131-8/+1
| | |
* | | we don't need to test that constant assignment worksJon Leighton2012-04-131-15/+0
|/ /
* / Add bang versions of relation query methods.Jon Leighton2012-04-121-0/+52
|/ | | | | The main reason for this is that I want to separate the code that does the mutating from the code that does the cloning.
* Merge pull request #5720 from kennyj/should_use_klass_methodJon Leighton2012-04-111-0/+1
|\ | | | | Get a properly aliased_table_name, when we use a polymorphic association.
| * Get a properly aliased_table_name, when we use a polymorphic association.kennyj2012-04-041-0/+1
| |
* | Override AR::Relation methods in NullRelation.Juanjo Bazán2012-04-111-0/+33
| | | | | | So a NullRelation (Relation#none) is chainable with database methods.
* | Fix delete_all when chained with joins.Rafael Mendonça França2012-04-101-1/+19
| | | | | | | | Closes #5202 and #919
* | Adds test to check that circular preloading does not modify Model.unscoped ↵Benedikt Deicke2012-04-033-4/+8
| | | | | | | | (as described in #5667)
* | Removes caching from ActiveRecord::Core::ClassMethods#relationBenedikt Deicke2012-04-031-0/+4
|/ | | | | | | | | | | The #relation method gets called in four places and the return value was instantly cloned in three of them. The only place that did not clone was ActiveRecord::Scoping::Default::ClassMethods#unscoped. This introduced a bug described in #5667 and should really clone the relation, too. This means all four places would clone the relation, so it doesn't make a lot of sense caching it in the first place. The four places with calls to relations are: activerecord/lib/active_record/scoping/default.rb:110:in `block in build_default_scope'" activerecord/lib/active_record/scoping/default.rb:42:in `unscoped'" activerecord/lib/active_record/scoping/named.rb:38:in `scoped'" activerecord/lib/active_record/scoping/named.rb:52:in `scope_attributes'"
* Fix #5667. Preloading should ignore scoping.Jon Leighton2012-03-301-0/+11
|
* fix testJon Leighton2012-03-301-1/+2
|
* whitespace :scissors:Jon Leighton2012-03-302-5/+4
|
* Merge pull request #5334 from courtland/masterJon Leighton2012-03-304-1/+25
|\ | | | | Fix deleting from a HABTM join table upon destroying an object of a model with optimistic locking enabled.
| * Tests for removing a HABTM association when optimistic locking is enabled.Nick Rogers2012-03-074-1/+25
| |
* | Add Relation#find_by and Relation#find_by!Jon Leighton2012-03-302-0/+48
| |
* | Merge pull request #2945 from Casecommons/nested_attributes_moduleJon Leighton2012-03-302-0/+14
|\ \ | | | | | | Nested attribute setters can be overridden.
| * | Nested attribute setters can be overridden.Jonathan Mukai & Peter Jaros2012-03-282-0/+14
| | | | | | | | | | | | Overriding implementation can call super.
* | | Deprecate Column#type_cast_codeCarlos Antonio da Silva2012-03-291-4/+0
| | |
* | | Add ActiveRecord::Base#slice to slice method callsGuillermo Iguaran2012-03-291-0/+12
|/ /
* | attributes are cached by string keys, so to_s to support symbols. fixes #5549Aaron Patterson2012-03-272-1/+16
| |
* | removed unused variablesArun Agrawal2012-03-271-2/+2
| |
* | ActiveRecord::Coders::YAMLColumn#dump should raise an errorGuten2012-03-261-0/+7
| | | | | | | | | | | | | | | | closes #2737 Conflicts: activerecord/lib/active_record/coders/yaml_column.rb
* | properly namespace the fixture exceptionAaron Patterson2012-03-221-1/+1
| |
* | Deprecate eager-evaluated scopes.Jon Leighton2012-03-2117-71/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't use this: scope :red, where(color: 'red') default_scope where(color: 'red') Use this: scope :red, -> { where(color: 'red') } default_scope { where(color: 'red') } The former has numerous issues. It is a common newbie gotcha to do the following: scope :recent, where(published_at: Time.now - 2.weeks) Or a more subtle variant: scope :recent, -> { where(published_at: Time.now - 2.weeks) } scope :recent_red, recent.where(color: 'red') Eager scopes are also very complex to implement within Active Record, and there are still bugs. For example, the following does not do what you expect: scope :remove_conditions, except(:where) where(...).remove_conditions # => still has conditions
generated by cgit v1.2.3 (git 2.25.1) at 2025-04-06 00:07:40 +0000