Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Return false for exists? with new records - fixes #6199. | Andrew White | 2012-05-10 | 1 | -2/+1 |
| | |||||
* | Use `take` instead of `first` to avoid unwanted implicit ordering (fixes #6147) | Marcelo Silveira | 2012-05-05 | 1 | -4/+4 |
| | |||||
* | clean up implementation of dynamic methods. use method compilation etc. | Jon Leighton | 2012-05-04 | 1 | -41/+0 |
| | |||||
* | Use arel nodes instead of raw sql | Marcelo Silveira | 2012-05-02 | 1 | -3/+3 |
| | |||||
* | Use Array#first instead of Array#[0] | Marcelo Silveira | 2012-05-02 | 1 | -5/+5 |
| | |||||
* | Introducing `take` as a replacement to the old behavior of `first` | Marcelo Silveira | 2012-05-02 | 1 | -0/+27 |
| | |||||
* | Document `last`, check for primary key on default order and use quoted table ↵ | Marcelo Silveira | 2012-05-02 | 1 | -2/+5 |
| | | | | and column names | ||||
* | Made `first` finder consistent among database engines by adding a | Marcelo Silveira | 2012-05-02 | 1 | -2/+18 |
| | | | | default order clause (fixes #5103) | ||||
* | return value of block is returned | Aaron Patterson | 2012-04-30 | 1 | -2/+5 |
| | |||||
* | only yield to finder block if something is found | Aaron Patterson | 2012-04-30 | 1 | -1/+1 |
| | |||||
* | extract deprecated code for #find, #first, #last, #all | Jon Leighton | 2012-04-13 | 1 | -105/+36 |
| | |||||
* | use a hash to store relation values | Jon Leighton | 2012-04-13 | 1 | -7/+7 |
| | |||||
* | mention that coercion only happens when the primary key is an integer and ↵ | Reg Braithwaite | 2012-04-12 | 1 | -2/+2 |
| | | | | use +to_i+ formatting | ||||
* | Documents that ActiveRecord's find by id uses to_i to coerce its arguments ↵ | Reg Braithwaite | 2012-04-12 | 1 | -1/+3 |
| | | | | to integers | ||||
* | Add Relation#find_by and Relation#find_by! | Jon Leighton | 2012-03-30 | 1 | -0/+19 |
| | |||||
* | Remove 'needless boolean casting'. | Jack Chen (chendo) | 2012-03-26 | 1 | -1/+1 |
| | | | | | "Predicates in Rails rely on standard Ruby semantics for boolean values and guarantee no singletons whatsoever." - @fxn | ||||
* | Remove IdentityMap | Carlos Antonio da Silva | 2012-03-13 | 1 | -10/+0 |
| | |||||
* | Add dynamic find_or_create_by_{attribute}! method. | Andrew White | 2012-03-12 | 1 | -1/+1 |
| | |||||
* | use bind values for join columns | Aaron Patterson | 2012-02-27 | 1 | -2/+2 |
| | |||||
* | prepared statements can be disabled | Aaron Patterson | 2012-02-21 | 1 | -1/+1 |
| | |||||
* | Revert "Merge pull request #4746 from jenslukowski/issue4718" | Aaron Patterson | 2012-01-31 | 1 | -8/+1 |
| | | | | | This reverts commit 5cad7a0dba652f3e942e58087bc88ba5c5390edc, reversing changes made to bb842e8d2111e50b21a14b8bd6d89371a4b9cd68. | ||||
* | fix for Issue 4718 | Jens Lukowski | 2012-01-29 | 1 | -1/+8 |
| | |||||
* | correctly handle order calls after a reorder | Matt Jones + Scott Walker | 2012-01-03 | 1 | -2/+2 |
| | |||||
* | allow reorder to affect eager loading correctly | Matt Jones + Scott Walker | 2011-12-28 | 1 | -1/+1 |
| | |||||
* | Fix #exists? for AR::Model | Jon Leighton | 2011-12-24 | 1 | -1/+1 |
| | |||||
* | exclude ORDER BY clause when querying Relation#exists? | Akira Matsuda | 2011-11-09 | 1 | -1/+1 |
| | |||||
* | Merge pull request #2485 from akaspick/exists_fix | Jon Leighton | 2011-09-07 | 1 | -1/+3 |
|\ | | | | | fix exists? to return false if passed nil (which may come from a missing | ||||
| * | fix exists? to return false if passed nil (which may come from a missing URL ↵ | Andrew Kaspick | 2011-08-10 | 1 | -1/+3 |
| | | | | | | | | param) | ||||
* | | Use LIMIT sql word in last when it's possible | Damien Mathieu | 2011-09-05 | 1 | -1/+5 |
| | | |||||
* | | use LIMIT SQL word in first - Closes #2783 | Damien Mathieu | 2011-09-05 | 1 | -1/+1 |
|/ | |||||
* | Make it the responsibility of the connection to hold onto an ARel visitor ↵ | Jon Leighton | 2011-08-08 | 1 | -3/+3 |
| | | | | for generating SQL. This improves the code architecture generally, and solves some problems with marshalling. Adapter authors please take note: you now need to define an Adapter.visitor_for method, but it degrades gracefully with a deprecation warning for now. | ||||
* | avoid Symbol#to_proc | Vijay Dev | 2011-07-20 | 1 | -1/+1 |
| | |||||
* | ActiveRecord: Fix eager loading so that giving a blank order clause ↵ | Elliot Winkler | 2011-07-18 | 1 | -1/+1 |
| | | | | generates valid SQL | ||||
* | Merge pull request #1823 from gnarg/master | Santiago Pastorino | 2011-07-10 | 1 | -2/+2 |
|\ | | | | | Log instrumentation name for exists? queries | ||||
| * | Log instrumentation name for exists? queries | Jon Guymon | 2011-06-22 | 1 | -2/+2 |
| | | |||||
* | | Merge pull request #1273 from jeremyf/feature-association-proxy-send | Santiago Pastorino | 2011-07-06 | 1 | -0/+1 |
|\ \ | |/ |/| | Addresses an inconsistency in the ActiveRecord::Base.method_missing handl | ||||
| * | Addresses an inconsistency in the ActiveRecord::Base.method_missing handling ↵ | Jeremy Friesen | 2011-05-24 | 1 | -0/+1 |
| | | | | | | | | of dynamic finder methods and the passing of the &block parameter for :find_by_attributes. | ||||
* | | Merge branch 'master' of git://github.com/lifo/docrails | Xavier Noria | 2011-05-25 | 1 | -1/+1 |
|\ \ | |/ |/| | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/core_ext/kernel/requires.rb | ||||
| * | Remove extra white spaces on ActiveRecord docs. | Sebastian Martinez | 2011-05-23 | 1 | -1/+1 |
| | | |||||
* | | Add support for passing mass assignment roles to dynamic finders. Closes #1170. | Andrew White | 2011-05-21 | 1 | -2/+2 |
| | | |||||
* | | Syntax cleanup: for in => each | Guillermo Iguaran | 2011-05-19 | 1 | -1/+1 |
|/ | |||||
* | Fix for #371 | Nick Howard | 2011-05-08 | 1 | -1/+6 |
| | | | | | | | | | | | | if a query contains a limit or an offset, ActiveRecord::FinderMethods#find_last had inconsistent behavior. If the records were loaded, it returned the last record in the cached list. If they were not, it reversed the order of the query and changed the limit to one. If the earlier limit was less than the total matching the query in the db, it would return a different record than if the records had been cached. This commit changes find_last so that it loads the records when getting the last record on a query containing a limit or an offset, which makes the behavior consistent. | ||||
* | deprecated the use of the guard_protected_attributes argument with ↵ | Josh Kalderimis | 2011-04-25 | 1 | -2/+2 |
| | | | | attributes= in AR in favor of assign_attributes(attrs, :without_protection => true) | ||||
* | stop using distinct on for the unique id queries. [#6450 state:resolved] | Aaron Patterson | 2011-04-21 | 1 | -0/+2 |
| | |||||
* | use index based substitution for bind parameters | Aaron Patterson | 2011-04-13 | 1 | -1/+1 |
| | |||||
* | stop using deprecated methods in arel | Aaron Patterson | 2011-04-11 | 1 | -2/+2 |
| | |||||
* | Use IM when trying to load records using ID. | Emilio Tagua | 2011-04-04 | 1 | -0/+9 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Change exists? so that it doesn't instantiate records [#6127 state:resolved] | Andrew White | 2011-03-29 | 1 | -4/+10 |
| | |||||
* | removes unnecessary selfs, and mentions that first! and last! take no ↵ | Xavier Noria | 2011-03-26 | 1 | -4/+6 |
| | | | | arguments in their API docs | ||||
* | No arguments for first! and last! | Pratik Naik | 2011-03-25 | 1 | -4/+4 |
| |