aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #3748 from samsonasu/has_many_custom_pk_new_recordJon Leighton2011-11-272-1/+5
|\ | | | | New records should load has_many relationships with custom primary keys
| * load has_many associations keyed off a custom primary key if that key is ↵Brian Samson2011-11-252-1/+5
| | | | | | | | present but the record is unsaved
* | Use any instead of lengthRahul P. Chaudhari2011-11-271-1/+1
| |
* | add the query to AR::Relation#explain outputXavier Noria2011-11-251-2/+3
| | | | | | | | | | | | | | Rationale: this is more readable if serveral queries are involved in one call. Also, it will be possible to let AR log EXPLAINs automatically in production mode, where queries are not even around.
* | Implement ArraySerializer and move old serialization API to a new namespace.José Valim2011-11-232-4/+4
|/ | | | | | | | | | | | | The following constants were renamed: ActiveModel::Serialization => ActiveModel::Serializable ActiveModel::Serializers::JSON => ActiveModel::Serializable::JSON ActiveModel::Serializers::Xml => ActiveModel::Serializable::XML The main motivation for such a change is that `ActiveModel::Serializers::JSON` was not actually a serializer, but a module that when included allows the target to be serializable to JSON. With such changes, we were able to clean up the namespace to add true serializers as the ArraySerializer.
* oops! I suck! :bomb:Aaron Patterson2011-11-201-1/+0
|
* removing deprecated methodsAaron Patterson2011-11-193-26/+0
|
* adding visitors to the respective adaptersAaron Patterson2011-11-192-0/+2
|
* pushing caching and visitors down to the connectionAaron Patterson2011-11-197-93/+119
|
* Perf fixJon Leighton2011-11-191-2/+6
| | | | | | | If we're deleting all records in an association, don't add a IN(..) clause to the query. Fixes #3672.
* Use `show index from`. We could fix `pk_and_sequence_for` method's ↵kennyj2011-11-192-21/+3
| | | | performance problem (GH #3678)
* Only used detailed schema introspection when doing a schema dump.Christos Zisopoulos2011-11-182-1/+10
| | | Fixes #3678
* Stop db:seed from performing a reload of the dbSam Ruby2011-11-181-2/+2
| | | | Fix regression introduced by 38d26b0cb56d82093889efa95992a35ba3bb9f29
* Merge pull request #3666 from oscardelben/databases_rake_cleanupVijay Dev2011-11-181-8/+9
|\ | | | | Cleanup of databases.rake psql env variables
| * Cleanup of databases.rake psql env variablesOscar Del Ben2011-11-171-8/+9
| |
* | Revert "Merge pull request #1163 from amatsuda/sexier_migration_31"Aaron Patterson2011-11-173-22/+13
| | | | | | | | | | | | | | | | | | | | This reverts commit 0e407a90413d8a19002b85508d811ccdf2190783, reversing changes made to 533a9f84b035756eedf9fdccf0c494dc9701ba72. Conflicts: activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb activerecord/test/cases/migration_test.rb
* | Revert "Merge pull request #3603 from vijaydev/change_table_without_block_arg"Aaron Patterson2011-11-171-12/+7
|/ | | | | | | | | This reverts commit 81fad6a270ec3cbbb88553c9f2e8200c34fd4d13, reversing changes made to 23101de283de13517e30c4c3d1ecc65525264886. Conflicts: activerecord/test/cases/migration_test.rb
* No need to `readlines` then `join`, just use `read` :heart:Arun Agrawal2011-11-171-1/+1
| | | same as 4c1a1933cbc5ab96efe340a3b31ac5fee12c99c8
* No need to `readlines` then `join`, just use `read` :heart:Aaron Patterson2011-11-161-1/+1
|
* Adding a deprecation warning for use of the schema_info table.Aaron Patterson2011-11-161-0/+2
|
* Join method uses empty string by default, so remove itAaron Patterson2011-11-161-2/+2
|
* dbfile isn't supported anymore, so removeAaron Patterson2011-11-161-3/+3
|
* Reduce schema format testsAaron Patterson2011-11-161-10/+14
|
* Move conditionals to separate tasks so they can be reused.Aaron Patterson2011-11-161-7/+11
|
* `ActiveRecord::Base#becomes` should retain the errors of the original object.James Adam2011-11-161-0/+1
| | | | This commit contains a simple failing test that demonstrates the behaviour we expect, and a fix. When using `becomes` to transform the type of an object, it should retain any error information that was present on the original instance.
* removing some useless conditionalsAaron Patterson2011-11-161-10/+8
|
* Merge pull request #2948 from atambo/masterAaron Patterson2011-11-161-38/+66
|\ | | | | schema_format :sql should behave like schema_format :ruby
| * schema_format :sql should behave like schema_format :rubyAlex Tambellini2011-11-041-38/+66
| | | | | | | | | | | | | | This commit adds a db:structure:load task that is run instead of db:schema:load when schema_format is set to :sql. This patch also removes the prefixing of the structure.sql files to mimic the use of a single schema.rb file. The patch originates from github issue #715.
* | Fixes issue #3483, regarding using a mixture of ranges and discrete values ↵Ryan Naughton2011-11-141-6/+8
| | | | | | | | in find conditions. Paired with Joey Schoblaska.
* | Speed up attribute invocation by checking if both name and calls are compilable.José Valim2011-11-142-2/+2
| |
* | Fix pull request #3609Noé Froidevaux2011-11-131-1/+1
| |
* | Prevent multiple SHOW TABLES calls when a table don't exists in database.Noé Froidevaux2011-11-111-3/+3
| |
* | Modify change_table to remove the need for the block argument.Vijay Dev2011-11-111-7/+12
| |
* | Checking the arity of the block passed to create_tableVijay Dev2011-11-101-1/+7
| | | | | | | | | | | | A recent change made to create_table does away with the need for the block argument. Checking the arity will prevent the mixing up of the two syntaxes.
* | exclude ORDER BY clause when querying Relation#exists?Akira Matsuda2011-11-091-1/+1
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-083-7/+6
|\ \ | | | | | | | | | | | | Conflicts: railties/guides/source/active_record_querying.textile
| * | document fix: remove_column takes multiple column_namesAkira Matsuda2011-11-071-2/+2
| | |
| * | self.up, self.down => up, downAkira Matsuda2011-11-072-5/+4
| | |
* | | adds trailing +s to the output of EXPLAIN for MySQLXavier Noria2011-11-071-3/+2
| | |
* | | Merge pull request #3521 from ↵Aaron Patterson2011-11-061-2/+1
|/ / | | | | | | | | | | nulogy/fix_postgres_adapter_to_handle_spaces_between_schemas Fix postgres adapter to handle spaces between schemas
* | Update ActiveRecord#attribute_present? to work as documentedJustin Mazzi2011-11-051-1/+2
| | | | | | | | | | | | | | "Returns true if the specified attribute has been set by the user or by a database load and is neither nil nor empty?" Fixes #1613
* | implements AR::Relation#explainXavier Noria2011-11-054-1/+153
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a first implementation, EXPLAIN is highly dependent on the database and I have made some compromises. On one hand, the method allows you to run the most common EXPLAIN and that's it. If you want EXPLAIN ANALYZE in PostgreSQL you need to do it by hand. On the other hand, I've tried to construct a string as close as possible to the ones built by the respective shells. The rationale is that IMO the user should feel at home with the output and recognize it at first sight. Per database. I don't know whether this implementation is going to work well. Let's see whether people like it.
* | Merge pull request #3525 from kennyj/fix_3440-1Jon Leighton2011-11-051-2/+11
|\ \ | | | | | | Fixed an issue id false option is ignored on mysql/mysql2 (fix #3440)
| * | Fixed an issue id false option is ignored on mysql/mysql2 (fix #3440)kennyj2011-11-061-2/+11
| |/
* | Use uniq instead of manually putting a DISTINCT in the queryJon Leighton2011-11-051-14/+2
| |
* | Add ActiveRecord::Relation#uniq for toggling DISTINCT in the SQL queryJon Leighton2011-11-053-8/+28
| |
* | AR changes to support creating ordered (asc, desc) indexesVlad Jebelev2011-11-048-16/+85
|/
* Allow the :class_name option for associations to take a symbol.Jon Leighton2011-11-041-1/+1
| | | | | This is to avoid confusing newbies, and to be consistent with the fact that other options like :foreign_key already allow a symbol or a string.
* Merge pull request #1163 from amatsuda/sexier_migration_31Aaron Patterson2011-11-043-13/+16
|\ | | | | Sexier migrations
| * No "t." for the migration DSL!Akira Matsuda2011-05-203-13/+16
| | | | | | | | | | Now you can omit |t| block parameter and all the t. from your migration code, that means, the syntax looks more Rails-3-ish, like the routes DSL and ActionMailer DSL. Also, this change won't break any of your existing migration files, since the traditional syntax is still available.