Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | fix association :extend option | Jon Leighton | 2012-07-13 | 6 | -16/+26 | |
| | | | | | ||||||
* | | | | | Remove duplicated unique index name. | kennyj | 2012-07-14 | 1 | -4/+7 | |
| | | | | | ||||||
* | | | | | Remove instance level quote_value method. This method is private and also ↵ | kennyj | 2012-07-14 | 2 | -10/+3 | |
| | | | | | | | | | | | | | | | | | | | | exists in class method. | |||||
* | | | | | Merge pull request #7045 from kennyj/remove_duplicated_code_20120714 | Rafael Mendonça França | 2012-07-13 | 1 | -11/+3 | |
|\ \ \ \ \ | | | | | | | | | | | | | Remove duplicated code in the AR::Store. | |||||
| * | | | | | Remove duplicated code in the AR::Store. | kennyj | 2012-07-14 | 1 | -11/+3 | |
| | | | | | | ||||||
* | | | | | | revises RUNNING_UNIT_TESTS to delegate stuff to the contributing guide | Xavier Noria | 2012-07-13 | 1 | -11/+5 | |
| | | | | | | ||||||
* | | | | | | move the deprecated options into active_record_deprecated_finders | Jon Leighton | 2012-07-13 | 2 | -5/+2 | |
| | | | | | | ||||||
* | | | | | | stop using class_attribute where methods/inheritance will suffice. | Jon Leighton | 2012-07-13 | 9 | -55/+57 | |
| | | | | | | ||||||
* | | | | | | Add link to relevant rails guide to Active Record unit test docs. | Katrina Owen | 2012-07-13 | 1 | -0/+3 | |
| | | | | | | ||||||
* | | | | | | Add note about needing mysql superuser for unit tests. | Katrina Owen | 2012-07-13 | 1 | -0/+2 | |
|/ / / / / | ||||||
* | | | | | Add documentation for ActiveRecord::Observer. | Katrina Owen | 2012-07-13 | 1 | -0/+6 | |
| | | | | | | | | | | | | | | | | | | | | The instructions about configuration covered only rails-specific usage. | |||||
* | | | | | extract deprecated association options to active_record_deprecated_finders | Jon Leighton | 2012-07-13 | 1 | -43/+0 | |
| | | | | | ||||||
* | | | | | Represent association scope options as AR::Relations insternally. | Jon Leighton | 2012-07-13 | 15 | -165/+191 | |
| | | | | | ||||||
* | | | | | fix assertion arguments order | Jon Leighton | 2012-07-13 | 1 | -2/+2 | |
| | | | | | ||||||
* | | | | | Allow associations to take a lambda which builds the scope | Jon Leighton | 2012-07-13 | 9 | -48/+69 | |
| | | | | | ||||||
* | | | | | support relations created with a table alias | Jon Leighton | 2012-07-13 | 2 | -1/+11 | |
| | | | | | ||||||
* | | | | | Add teardown method to AR::Mig::RenameTableTest | Robb Kidd | 2012-07-10 | 1 | -18/+13 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dry up reseting the renamed table after each test. Also made use of the AR::Base.connection object already available from AR::MigrationTest#connection. | |||||
* | | | | | Merge pull request #6874 from robbkidd/rename_sequences_too | Aaron Patterson | 2012-07-10 | 2 | -0/+20 | |
|\ \ \ \ \ | |_|/ / / |/| | | | | Rename default sequence when table is renamed? [AR:postgres] | |||||
| * | | | | Update psql adapter to rename a default pkey sequence when renaming a table. | Robb Kidd | 2012-06-27 | 2 | -0/+20 | |
| | | | | | ||||||
* | | | | | Add references statements to migration generator | Aleksey Magusev | 2012-07-08 | 1 | -4/+16 | |
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AddXXXToYYY/RemoveXXXFromYYY migrations are produced with references statements, for instance rails g migration AddReferencesToProducts user:references supplier:references{polymorphic} will generate the migration with: add_reference :products, :user, index: true add_reference :products, :supplier, polymorphic: true, index: true | |||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-07-07 | 3 | -6/+41 | |
|\ \ \ \ | ||||||
| * | | | | minor text change [ci skip] | Vijay Dev | 2012-07-07 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | Revert "Add nodoc to relation methods" | Vijay Dev | 2012-07-07 | 1 | -22/+24 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit c47a698d5d497340d4e349257522212173865838. Reason: Let's revert pending further discussions | |||||
| * | | | | Add nodoc to relation methods | Oscar Del Ben | 2012-07-06 | 1 | -24/+22 | |
| | | | | | ||||||
| * | | | | Add order docs | Oscar Del Ben | 2012-07-05 | 1 | -0/+10 | |
| | | | | | ||||||
| * | | | | Add group documentation | Oscar Del Ben | 2012-07-05 | 1 | -0/+12 | |
| | | | | | ||||||
| * | | | | Add documentation for includes | Oscar Del Ben | 2012-07-05 | 1 | -0/+11 | |
| | | | | | ||||||
| * | | | | remove :nodoc: of AR::Scoping#unscoped [ci skip] | Francesco Rodriguez | 2012-07-01 | 1 | -4/+4 | |
| | | | | | ||||||
| * | | | | fix AR::SchemaStatements#column_exists? example [ci skip] | Francesco Rodriguez | 2012-06-30 | 1 | -1/+1 | |
| | | | | | ||||||
| * | | | | update AR::SchemaStatements#column_exists? documentation [ci skip] | Francesco Rodriguez | 2012-06-30 | 1 | -2/+4 | |
| | | | | | ||||||
* | | | | | Refactor locked? method in query cache | Carlos Antonio da Silva | 2012-07-07 | 1 | -5/+2 | |
| | | | | | | | | | | | | | | | | | | | | Introduced in 75b340d1a4bcf2f1233fb65a15ff6b8059e2230e | |||||
* | | | | | Load all records in Relation#inspect | Jon Leighton | 2012-07-07 | 1 | -4/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A test was failing due to the way that Relation#inspect causes association proxies to ignore unsaved records added to the association. This is fixed by simply calling to_a and letting to_a figure out how to get the records (which, in the case of associations, takes into account new records). I think it is acceptable to do this rather than limiting the query at the database level: * It's what we've done in all released Rails versions up to this point * The goal of the limit is to not flood the console with output - this is the problem we're targeting, rather than the actual loading of the records from the database * You probably want to do something with those records later anyway, otherwise you wouldn't have built a relation for them. | |||||
* | | | | | Merge pull request #6838 from kennyj/added_registration_task | Carlos Antonio da Silva | 2012-07-07 | 2 | -7/+27 | |
|\ \ \ \ \ | | | | | | | | | | | | | Allow to register database tasks from different adapters | |||||
| * | | | | | Added a feature to add a database task. e.g. OracleDatabaseTasks | kennyj | 2012-06-24 | 2 | -7/+27 | |
| | | | | | | ||||||
* | | | | | | fixup changelog | Jon Leighton | 2012-07-07 | 1 | -1/+1 | |
| | | | | | | ||||||
* | | | | | | Relation#inspect handles doesn't perform a new query on an already-loaded ↵ | Jon Leighton | 2012-07-07 | 2 | -2/+16 | |
| | | | | | | | | | | | | | | | | | | | | | | | | relation | |||||
* | | | | | | Simplify Relation#inspect | Jon Leighton | 2012-07-07 | 1 | -12/+3 | |
| | | | | | | ||||||
* | | | | | | Merge pull request #6993 from morgoth/engine-table-name-prefix-generator-fix | José Valim | 2012-07-07 | 1 | -1/+1 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Engine table name prefix generator fix | |||||
| * | | | | | | Fixed generating namespaced table_name_prefix in engines | Wojciech Wnętrzak | 2012-07-06 | 1 | -1/+1 | |
| | | | | | | | ||||||
* | | | | | | | Merge pull request #6997 from kennyj/fix_stored_attributes | José Valim | 2012-07-07 | 4 | -2/+16 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | Added *instance_writer: false* to stored/serialized attributes. | |||||
| * | | | | | | | Added *instance_writer: false* for stored/serialized attributes. | kennyj | 2012-07-07 | 4 | -2/+16 | |
| |/ / / / / / | ||||||
* / / / / / / | Limit the number of records in Relation#inspect | Damien Mathieu | 2012-07-06 | 3 | -3/+22 | |
|/ / / / / / | | | | | | | | | | | | | | | | | | | While it's interesting to have the results array, it can make a console or a webpage freeze if there are a lot of them. So this limits the number of records displayed in #inspect to 10 and tells how much were effectively found. | |||||
* | | | | | | Merge pull request #6985 from sidonath/disable-query-cache-for-locks | Rafael Mendonça França | 2012-07-06 | 2 | -1/+17 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Disable query cache for lock queries | |||||
| * | | | | | | Disable query cache for lock queries | Damir Zekic | 2012-07-06 | 2 | -1/+17 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #867 | |||||
* | | | | | | | Update release notes with AR::Relation#inspect change [ci skip] | Carlos Antonio da Silva | 2012-07-06 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See 07314e64fd62fb8e6165c8c539420160da9437e9. Also fix some tabs in AR Changelog. | |||||
* | | | | | | | Show the records in Relation#inspect | Jon Leighton | 2012-07-06 | 3 | -6/+17 | |
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reason for removing the previous implementation of `#inspect` was that it hid from you that you were dealing with a `Relation` rather than an `Array`. But it is still useful to be able to see the records, particularly if you're writing something like the following in tests: assert_equal [foo], Post.where(:bar) If the assertion fails, you want to see what records were actually loaded. So this implementation makes it clear that you've got a `Relation`, but also shows your records. | |||||
* | | | | | | Merge pull request #6927 from parndt/patch-3 | Carlos Antonio da Silva | 2012-07-05 | 2 | -3/+3 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Make ArgumentError message more consistent in nested attributes | |||||
| * | | | | | | Made ArgumentError messages consistent. | Philip Arndt | 2012-07-06 | 2 | -3/+3 | |
| | | | | | | | ||||||
* | | | | | | | Add documentation for inheritance_column method | Vitor Balocco | 2012-07-05 | 1 | -1/+3 | |
| | | | | | | | ||||||
* | | | | | | | fix quoting for ActiveSupport::Duration instances | Francesco Rodriguez | 2012-07-04 | 3 | -1/+15 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes quoting for ActiveSupport::Duration instances: # before >> ActiveRecord::Base.connection.quote 30.minutes => "'--- 1800\n...\n'" # after >> ActiveRecord::Base.connection.quote 30.minutes => "1800" Also, adds a test for type casting ActiveSupport::Duration instances. Related to #1119. |