aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
Commit message (Collapse)AuthorAgeFilesLines
* association methods are now generated in modulesJosh Susser2011-11-159-30/+43
| | | | | | | | | | Instead of generating association methods directly in the model class, they are generated in an anonymous module which is then included in the model class. There is one such module for each association. The only subtlety is that the generated_attributes_methods module (from ActiveModel) must be forced to be included before association methods are created so that attribute methods will not shadow association methods.
* Speed up attribute invocation by checking if both name and calls are compilable.José Valim2011-11-142-2/+2
|
* Merge pull request #3371 from yahonda/fix_ora_00932_errorAaron Patterson2011-11-141-1/+1
|\ | | | | Fix ORA-00932 error when trying to insert 0 to DATE type columns.
| * Fix ORA-00932 error when trying to insert 0 to DATE type columns.Yasuo Honda2011-10-191-1/+1
| |
* | Sync CHANGELOGs from 3-1-stableJon Leighton2011-11-141-0/+21
| |
* | 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
| |
* | change_table bulk test case should check if the connection supports bulk alterVijay Dev2011-11-111-13/+15
| |
* | Modify change_table to remove the need for the block argument.Vijay Dev2011-11-112-7/+64
| |
* | Checking the arity of the block passed to create_tableVijay Dev2011-11-102-1/+22
| | | | | | | | | | | | 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-092-1/+6
| |
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-11-084-9/+8
|\ \ | | | | | | | | | | | | 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-073-7/+6
| | |
* | | adds trailing +s to the output of EXPLAIN for MySQLXavier Noria2011-11-071-3/+2
| | |
* | | revises EXPLAIN tests for SQLite3Xavier Noria2011-11-061-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The output in Travis is a bit different. The SQLite documentation (http://www.sqlite.org/eqp.html) warns output may change dramatically between releases. I do not want to mock the result set because I want a real EXPLAIN to happen. I prefer a test that may fail in future releases than a test that may give false positives in future releases.
* | | Merge pull request #3521 from ↵Aaron Patterson2011-11-062-2/+5
|/ / | | | | | | | | | | 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-052-1/+20
| | | | | | | | | | | | | | "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-058-1/+226
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-053-2/+20
|\ \ | | | | | | 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-063-2/+20
| | |
* | | 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-057-9/+67
| | |
* | | AR changes to support creating ordered (asc, desc) indexesVlad Jebelev2011-11-0411-20/+111
|/ /
* | Fix test as one more has_many addedArun Agrawal2011-11-041-1/+1
| |
* | Synchronize the gemspecs since CHANGELOG has been renamed to CHANGELOG.mdFranck Verrot2011-11-041-1/+1
| |
* | Convert CHANGELOGs to Markdown format.Jon Leighton2011-11-042-6602/+6564
| | | | | | | | | | | | | | | | | | | | | | | | | | Reasons: * Markdown reads well as plain text, but can also be formatted. * It will make it easier for people to read on the web as Github formats the Markdown nicely. * It will encourage a level of consistency when people are writing CHANGELOG entries. The script used to perform the conversion is at https://gist.github.com/1339263
* | Allow the :class_name option for associations to take a symbol.Jon Leighton2011-11-044-2/+15
| | | | | | | | | | 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-044-13/+36
|\ \ | | | | | | Sexier migrations
| * | Tests for new create_table DSLAkira Matsuda2011-05-201-0/+20
| | |
| * | 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.
* | | Merge pull request #3507 from jmazzi/issue-3503Jeremy Kemper2011-11-032-2/+21
| | | | | | | | | | | | Preserve SELECT columns on the COUNT for finder_sql when possible
* | | Fix #3271.Jon Leighton2011-11-033-1/+14
| | | | | | | | | | | | | | | | | | | | | | | | Building the conditions of a nested through association could potentially modify the conditions of the through and/or source association. This is a Bad Thing.
* | | Memoize through associationJon Leighton2011-11-031-8/+11
| | |
* | | Remove all revelant through records.Jon Leighton2011-11-033-9/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | If a record is removed from a has_many :through, all of the join records relating to that record should also be removed from the through association's target. (Previously the records were removed in the database, but only one was removed from the in-memory target array.)
* | | Fix adding multiple instances of the same record to a has_many :through.Jon Leighton2011-11-033-21/+57
| | | | | | | | | | | | Fixes #3425.
* | | Fix #3247.Jon Leighton2011-11-033-2/+13
| | | | | | | | | | | | | | | Fixes creating records in a through association with a polymorphic source type.
* | | assert_match takes a regexp and a string in that orderVijay Dev2011-10-311-1/+1
| | |
* | | dump_schema_information: explicitly order inserts into schema_migrations tableLaust Rud Jacobsen2011-10-312-1/+10
| | | | | | | | | | | | | | | | | | | | | This change reduces churn in the db/development_structure.sql file when using :sql as active_record.schema_format, and makes comparing diffs much easier. Test ensures the output SQL-statements are lexically ordered by version.
* | | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-10-301-0/+3
|\ \ \
| * | | Added a note that optimistic locking also needs a hidden field to function ↵Joost Baaij2011-10-251-0/+3
| | | | | | | | | | | | | | | | across web workers.
* | | | clear statement cache when tables are alteredAaron Patterson2011-10-271-0/+6
| | | |
* | | | Added failing test case for changing schema in migration not clearing the ↵Joe Van Dyk2011-10-271-0/+12
| | | | | | | | | | | | | | | | prepared statement cache
* | | | More checks for instance writers.Arun Agrawal2011-10-271-1/+2
| | | |
* | | | Adding more checks for instance_writer falseArun Agrawal2011-10-271-1/+2
| | | |
* | | | Test fix Allow instances to disable record_timestampsArun Agrawal2011-10-271-1/+1
| | | | | | | | | | | | | | | | Removed here 0d0176c4ffe5c58fd1002efbd8f7bd45a8872e33
* | | | Allow instances to disable record_timestampsPratik Naik2011-10-262-1/+11
| | | |
* | | | Merge branch 'master' of github.com:rails/railsDavid Heinemeier Hansson2011-10-2520-53/+206
|\ \ \ \
| * | | | safeguard against configs missing environment or the database keyVijay Dev2011-10-261-1/+1
| | | | |
| * | | | minor revision to some new code in databases.rakeXavier Noria2011-10-241-2/+3
| | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-28 19:54:52 +0000