aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fixes some ActionMailer testsSantiago Pastorino2010-08-143-4/+4
|
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-14337-2124/+2122
| | | | 's/[ \t]*$//' -i {} \;)
* Makes AR use AMo to_key implementationSantiago Pastorino2010-08-132-7/+1
| | | | [#5249]
* This method is actually not used, it's implemented on the concrete adaptersSantiago Pastorino2010-08-131-8/+0
| | | | [#5331 state:committed]
* converting to a symbol is not necessaryAaron Patterson2010-08-131-1/+1
|
* Removing most of the symbol to proc usage in Active RecordPrem Sichanugrist2010-08-1315-28/+28
| | | | This will hopefully make Active Record run a bit more faster.
* removing a lolinjectAaron Patterson2010-08-131-2/+2
|
* Moves local_request? to require.local?Santiago Pastorino2010-08-134-13/+15
| | | | [#5361 state:committed]
* avoid direct use of arel constantsAaron Patterson2010-08-131-2/+4
|
* removing references to arel constantsAaron Patterson2010-08-131-1/+1
|
* avoiding symbol to proc againAaron Patterson2010-08-131-1/+1
|
* do not use arel constants directlyAaron Patterson2010-08-131-1/+1
|
* symbol to proc is slow, we should avoid itAaron Patterson2010-08-131-1/+1
|
* It's snowing!Jeremy Kemper2010-08-123-3/+3
|
* be more precise re :validate and :autosaveXavier Noria2010-08-122-6/+5
|
* no callbacks should be created for empty array [#5289 state:resolved]Subba Rao Pasupuleti2010-08-122-3/+20
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* tidy up validations length code [#5297 state:resolved]Subba Rao Pasupuleti2010-08-121-6/+2
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add missing ActiveModel::Validations requireGreg Campbell2010-08-122-1/+1
| | | | | | | | | | | [#5311 state: resolved] ActiveModel::Validations uses Hash#except, but does not require it from ActiveSupport. (This wasn't showing up in the tests, because it was required in the helper, and was also required in ActiveModel::Serialization). Signed-off-by: José Valim <jose.valim@gmail.com>
* Ensure we can nest include calls [#5285 state:resolved]Neeraj Singh2010-08-127-3/+34
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Merge remote branch 'docrails/master'Xavier Noria2010-08-1216-102/+110
|\
| * revises some autosave docs, style and contentXavier Noria2010-08-122-59/+23
| |
| * applied guidelines to "# =>"Paco Guzman2010-08-1211-51/+51
| |
| * commit review: applies guidelines to "# =>"Xavier Noria2010-08-122-4/+4
| |
| * updating documentation for named_scope and default_scopeNeeraj Singh2010-08-122-0/+19
| |
| * ActiveModel::Base doesn't exist, include_root_in_json is a class_attribute ↵Santiago Pastorino2010-08-101-3/+3
| | | | | | | | for the classes which includes this module
| * Fixed problem with markup that caused an extra symbol in a function name ↵Brandon Tilley2010-08-091-1/+1
| | | | | | | | when rendered.
| * adding more documentation for autosave optionNeeraj Singh2010-08-092-14/+39
| |
* | Make update_attribute behave as in Rails 2.3 and document the behavior ↵José Valim2010-08-124-73/+77
| | | | | | | | intrinsic to its implementation.
* | fixed indentation in test casesMark Turner2010-08-113-4/+3
| | | | | | | | Signed-off-by: wycats <wycats@gmail.com>
* | Replace snowman with utf8=✓wycats2010-08-113-3/+3
| |
* | Raising exception if fixture file can't be foundPaul Hieromnimon2010-08-112-0/+15
| |
* | subtracting blank strings is slightly faster than blank?Aaron Patterson2010-08-111-3/+1
| |
* | AS guide: some revisionsXavier Noria2010-08-121-33/+17
| |
* | no need to assign if we are gonna returnXavier Noria2010-08-121-2/+2
| |
* | avoid multiple hash lookupsAaron Patterson2010-08-111-2/+6
| |
* | dry up the hash dup and avoid sending nil valuesAaron Patterson2010-08-112-11/+15
| |
* | avoiding tap saves us timeAaron Patterson2010-08-111-2/+6
| |
* | unless Array#empty? is faster than if Array#present?Aaron Patterson2010-08-111-1/+1
| |
* | stop using private methodsAaron Patterson2010-08-111-2/+2
| |
* | Ensure @config is not a reserved instance variable in controllers. [#5342 ↵José Valim2010-08-111-2/+2
| | | | | | | | state:resolved]
* | layout_for works again with objects as specified in the documentation and ↵José Valim & Carlos Antonio da Silva2010-08-114-4/+15
| | | | | | | | Rails 2.3 [#5357 state:resolved]
* | Missed one spot for --skip-active-record, which means that new Gemfile isn't ↵Nick Sieger2010-08-111-1/+1
| | | | | | | | | | | | set up right Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | added support for more printersGonçalo Silva2010-08-101-5/+9
| |
* | Revert "require_dependency should require using the normal mechanism if ↵José Valim2010-08-101-15/+7
| | | | | | | | | | | | | | possible to avoid double-requires" This was causing double requires since 991cd59a225b90ab1ba3 was reverted. This reverts commit 8bf79739b4219eb1d6464e6eb4853e92e81d7621.
* | fixes a typo reported by rymaiXavier Noria2010-08-101-1/+1
| |
* | AS guide: removes some duplication, and makes a second pass on method delegationXavier Noria2010-08-101-144/+21
| |
* | adds the AS guide to the guides indexXavier Noria2010-08-103-0/+6
| |
* | AS guide: documents Module#redefine_methodXavier Noria2010-08-101-0/+21
| |
* | AS guide: documents Module#(instance_)method_namesXavier Noria2010-08-101-0/+12
| |
* | Added test case to verify that transaction callbacks are correctly ↵Tobias Lütke2010-08-091-0/+41
| | | | | | | | propagated to class observers