Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | | | | | Merge pull request #5305 from arunagw/warning_fix | Vijay Dev | 2012-03-07 | 1 | -1/+1 | |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | warning fix : mismatched indentations at 'end' with 'case' | |||||
| * | | | | | warning fix : mismatched indentations at 'end' with 'case' | Arun Agrawal | 2012-03-06 | 1 | -1/+1 | |
| | |_|/ / | |/| | | | ||||||
* | | | | | Clean up module docs [ci skip] | Vijay Dev | 2012-03-07 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | Removed some useless docstrings and no-doc'ed some. | |||||
* | | | | | Revert table_name= needs connection now, so just stub table_name | kennyj | 2012-03-07 | 1 | -1/+1 | |
|/ / / / | ||||||
* | | | | Fix #5069 - Protect foreign key from mass assignment throught association ↵ | Jean Boussier | 2012-03-05 | 1 | -1/+2 | |
| | | | | | | | | | | | | | | | | builder | |||||
* | | | | Allow translations of activerecord.errors.messages.record_invalid to be ↵ | Christopher Dell | 2012-03-05 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | looked up in errors.messages.record_invalid | |||||
* | | | | Merge pull request #5268 from carlosantoniodasilva/ar-new-record-warning | Jon Leighton | 2012-03-04 | 2 | -1/+1 | |
|\ \ \ \ | | | | | | | | | | | Fix ActiveRecord warning in tests with Marshal | |||||
| * | | | | Initialize @stale_state to nil in association | Carlos Antonio da Silva | 2012-03-04 | 2 | -1/+1 | |
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This apparently fix the warning related to @new_record variable not being initialized in AR's test suit, when an association is built and the object is marshalled/loaded. See these tests in AR's base_test.rb: test_marshalling_with_associations test_marshalling_new_record_round_trip_with_associations Closes #3720. | |||||
* | | | | tidy up formatting | Jon Leighton | 2012-03-04 | 1 | -10/+11 | |
| | | | | ||||||
* | | | | Merge pull request #5267 from kennyj/fix_4674 | Jon Leighton | 2012-03-04 | 1 | -5/+14 | |
|\ \ \ \ | | | | | | | | | | | Fix GH #4674. Reset column information and sequence name when setting table_name. | |||||
| * | | | | Don't clear sequence name when we explicitly assign it. | kennyj | 2012-03-04 | 1 | -2/+4 | |
| | | | | | ||||||
| * | | | | Fix GH #4674. Reset column information and sequence name when setting ↵ | kennyj | 2012-03-04 | 1 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | table_name. | |||||
* | | | | | Fix GH #3163. Should quote database on mysql/mysql2. | kennyj | 2012-03-05 | 1 | -1/+1 | |
|/ / / / | ||||||
* / / / | Remove unusused variables | Jon Leighton | 2012-03-04 | 3 | -14/+9 | |
|/ / / | ||||||
* | | | Merge pull request #5255 from carlosantoniodasilva/active-record-review | Piotr Sarnacki | 2012-03-03 | 8 | -68/+77 | |
|\ \ \ | | | | | | | | | Refactor and cleanup in some ActiveRecord modules | |||||
| * | | | Refactor and cleanup in some ActiveRecord modules | Carlos Antonio da Silva | 2012-03-03 | 8 | -68/+77 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Avoid double hash lookups in AR::Reflection when reflecting associations/aggregations * Minor cleanups: use elsif, do..end, if..else instead of unless..else * Simplify DynamicMatchers#respond_to? * Use "where" instead of scoped with conditions hash * Extract `scoped_by` method pattern regexp to constant * Extract noisy class_eval from method_missing in dynamic matchers * Extract readonly check, avoid calling column#to_s twice in persistence * Refactor predicate builder, remove some variables | |||||
* | | | | Merge pull request #4984 from raghunadhd/change_orderhash | Vijay Dev | 2012-03-03 | 2 | -2/+2 | |
|\ \ \ \ | |_|_|/ |/| | | | Refactored code | |||||
| * | | | Refactored code | Raghunadh | 2012-02-10 | 2 | -2/+2 | |
| | | | | ||||||
* | | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-03-03 | 1 | -0/+3 | |
|\ \ \ \ | |_|/ / |/| | | | ||||||
| * | | | Add documentation for IndexDefinition. | Waseem Ahmad | 2012-03-02 | 1 | -0/+3 | |
| | | | | ||||||
* | | | | only log an error if there is a logger. fixes #5226 | Aaron Patterson | 2012-03-02 | 2 | -2/+2 | |
| | | | | ||||||
* | | | | revert setting NOT NULL constraints in add_timestamps | Xavier Noria | 2012-03-01 | 1 | -2/+2 | |
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 3dbedd2 added NOT NULL constraints both to table creation and modification. For creation the new default makes sense, but the generic situation for changing a table is that there exist records. Those records have no creation or modification timestamps, and in the general case you don't even know them, so when updating a table these constraints are not going to work. See a bug report for this use case in #3334. | |||||
* | | | Merge pull request #5212 from MarkMT/master | Xavier Noria | 2012-02-28 | 1 | -8/+19 | |
|\ \ \ | | | | | | | | | update_only documentation for accepts_nested_attributes_for | |||||
| * | | | Modified comments explaining what the update_only option does. | Mark | 2012-02-28 | 1 | -8/+19 | |
| | | | | ||||||
* | | | | Merge pull request #5196 from lest/patch-3 | José Valim | 2012-02-28 | 1 | -0/+1 | |
|\ \ \ \ | |_|/ / |/| | | | force datetime attributes to be changed, fix GH #3965 | |||||
| * | | | force datetime attributes to be changed, fix GH #3965 | Sergey Nartimov | 2012-02-28 | 1 | -0/+1 | |
| | | | | ||||||
* | | | | Merge pull request #5197 from lest/patch-4 | José Valim | 2012-02-27 | 1 | -6/+0 | |
|\ \ \ \ | |_|/ / |/| | | | remove unnecessary code | |||||
| * | | | remove unnecessary code | Sergey Nartimov | 2012-02-28 | 1 | -6/+0 | |
| |/ / | | | | | | | | | | | | | it was added in 36129f21b86db4bd69e932e586129e246c2a5ca8 but isn't useful anymore as corresponding tests pass without it | |||||
* | | | Revert "only mutate the scope object in the `bind` method" | Aaron Patterson | 2012-02-27 | 1 | -1/+1 | |
| | | | | | | | | | | | | This reverts commit 1b9e19cd22f2b5d5e7b82e042f92340822c0f966. | |||||
* | | | only mutate the scope object in the `bind` method | Aaron Patterson | 2012-02-27 | 1 | -1/+1 | |
| | | | ||||||
* | | | bind value creation refactoring | Aaron Patterson | 2012-02-27 | 1 | -4/+10 | |
|/ / | ||||||
* | | removing dead code | Aaron Patterson | 2012-02-27 | 1 | -3/+2 | |
| | | ||||||
* | | use bind values for join columns | Aaron Patterson | 2012-02-27 | 6 | -12/+36 | |
| | | ||||||
* | | Merge pull request #5168 from kennyj/fix_5152 | Aaron Patterson | 2012-02-27 | 1 | -1/+1 | |
|\ \ | | | | | | | [PostgreSQL] Remove NULLS FIRST/LAST. Closes #5152 | |||||
| * | | Remove NULLS FIRST/LAST. closes #5152 | kennyj | 2012-02-25 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #5173 from kennyj/fix_3931-2 | Aaron Patterson | 2012-02-27 | 1 | -9/+20 | |
|\ \ \ | | | | | | | | | Fix type_to_sql with text and limit on mysql/mysql2. Fix GH #3931 (Try again). | |||||
| * | | | Fix type_to_sql with text and limit on mysql/mysql2. Fix GH #3931. | kennyj | 2012-02-26 | 1 | -9/+20 | |
| |/ / | ||||||
* | | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-02-25 | 2 | -3/+4 | |
|\ \ \ | |/ / |/| | | | | | | | | Conflicts: actionmailer/CHANGELOG.md | |||||
| * | | Fixed typo in composed_of example with Money#<=>, was comparing amount ↵ | Noah Hendrix | 2012-02-24 | 1 | -1/+1 | |
| | | | | | | | | | | | | itself instead of other_money.amount | |||||
| * | | Revert "Remove meaningless code from the examples" of AutosaveAssociation | Dimitar Dimitrov | 2012-02-22 | 1 | -0/+3 | |
| | | | | | | | | | Reverts a part of 91148936f770dc8bbbb33d46a09528f1a32d8a71. Should probably be squashed with it when merging back in rails/rails. | |||||
| * | | Minor fixes to ActiveRecord::AutosaveAssociation's docs | Dimitar Dimitrov | 2012-02-22 | 1 | -5/+3 | |
| | | | | | | | | | | | | * Fix typos * Remove meaningless code from the examples | |||||
* | | | use regular ruby for fewer method calls. we do not need :gift:s! | Aaron Patterson | 2012-02-24 | 1 | -1/+4 | |
| | | | ||||||
* | | | removes verify_active_connections! | Xavier Noria | 2012-02-24 | 2 | -16/+1 | |
| | | | | | | | | | | | | | | | | | | | | | The method verify_active_connections! was used in the old days (up to 2.1 I think) by the dispatcher to verify the connections, but nowadays we do that in a different way and this method is obsolete. | |||||
* | | | prepared statements can be disabled | Aaron Patterson | 2012-02-21 | 8 | -30/+56 | |
| | | | ||||||
* | | | more ruby 2.0 respond_to? changes | Aaron Patterson | 2012-02-21 | 1 | -1/+1 | |
| | | | ||||||
* | | | tag bind params with a bind param object | Aaron Patterson | 2012-02-20 | 3 | -3/+3 | |
| | | | ||||||
* | | | removed commented line. 3434 tests, 10531 assertions, 0 failures, 0 errors, ↵ | William Lawson | 2012-02-20 | 1 | -1/+0 | |
| | | | | | | | | | | | | 31 skips | |||||
* | | | Restored ability to identify ID and Sequence from tables relying on a ↵ | William Lawson | 2012-02-20 | 1 | -16/+37 | |
|/ / | | | | | | | nonmatching sequence default value for PK. | |||||
* | | Correctly print names of non-timestamped migrations with db:migrate:status | Peter Mitchell | 2012-02-17 | 1 | -2/+3 | |
| | | ||||||
* | | use Process.pid rather than $$ | Aaron Patterson | 2012-02-16 | 3 | -7/+7 | |
| | |