Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Fix: attribute_for_inspect truncate upto (51 => 50) characters. | namusyaka | 2013-07-09 | 1 | -1/+1 | |
|/ | ||||||
* | Extract AR::AttributeMethods#attribute_for_inspect's test code out from ↵ | Takehiro Adachi | 2013-05-22 | 1 | -0/+8 | |
| | | | | | | | base_test.rb The method itself got extracted out from ActiveRecored::Base in commit ceb33f84933639d3b61aac62e5e71fd087ab65ed, but the test code never did. | |||||
* | test refactor | Aaron Patterson | 2013-05-16 | 1 | -5/+2 | |
| | ||||||
* | Add more data to AR::UnknownAttributeError | Bogdan Gusiev | 2013-05-16 | 1 | -0/+12 | |
| | | | | | | | | | begin Topic.new("hello" => "world") rescue ActiveRecord::UnknownAttributeError => e e.record # => #<Topic ... > e.attribute # => "hello" end | |||||
* | read_attribute_before_type_cast should accept symbol | Neeraj Singh | 2013-05-11 | 1 | -0/+1 | |
| | ||||||
* | fixes a test, and explains why AR::AttributeMethods checks ↵ | Xavier Noria | 2013-04-28 | 1 | -5/+2 | |
| | | | | defined?(@attributes) in some places | |||||
* | Removed space and unused class | Anupam Choudhury | 2013-04-06 | 1 | -1/+1 | |
| | ||||||
* | fix typos in AR. lots of them. | Vipul A M | 2013-03-19 | 1 | -2/+2 | |
| | ||||||
* | Remove not used require and some useless test comments | Carlos Antonio da Silva | 2012-11-17 | 1 | -12/+3 | |
| | ||||||
* | Fix typo in module name and make #in_time_zone private | Carlos Antonio da Silva | 2012-11-17 | 1 | -1/+1 | |
| | ||||||
* | Extract #in_time_zone helper method duplication to a module | Carlos Antonio da Silva | 2012-11-17 | 1 | -12/+2 | |
| | ||||||
* | Simplify helper method in attribute methods test | Carlos Antonio da Silva | 2012-11-17 | 1 | -3/+1 | |
| | ||||||
* | AR::AttributeMethods#[] raises AM::AttributeMissingError for missing attributes. | Francesco Rodriguez | 2012-10-28 | 1 | -0/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following behaviour: class Person < ActiveRecord::Base belongs_to :company end # Before: person = Person.select('id').first person[:name] # => nil person.name # => ActiveModel::MissingAttributeError: missing_attribute: name person[:company_id] # => nil person.company # => nil # After: person = Person.select('id').first person[:name] # => ActiveModel::MissingAttributeError: missing_attribute: name person.name # => ActiveModel::MissingAttributeError: missing_attribute: name person[:company_id] # => ActiveModel::MissingAttributeError: missing_attribute: company_id person.company # => ActiveModel::MissingAttributeError: missing_attribute: company_id Fixes #5433. | |||||
* | Revert "Key the attributes hash with symbols" | Jon Leighton | 2012-10-12 | 1 | -2/+2 | |
| | | | | | | | | | | | | This reverts commit 86c3dfbd47cb96af02daaa655963292b1a1b110e. Conflicts: activerecord/lib/active_record/attribute_methods/read.rb Reason: whilst this increased performance, it also presents a DoS risk via memory exhaustion if users were allowing user input to dictate the arguments of read/write_attribute. I will investigate alternative ways to cut down on string allocations here. | |||||
* | test cleanup, remove ruby_type because it's no longer needed | Yves Senn | 2012-09-03 | 1 | -1/+1 | |
| | | | | | All tests with a custom inheritance_column use the `Vegtable` model. The field ruby_type on the Company models is no longer needed | |||||
* | Key the attributes hash with symbols | Jon Leighton | 2012-08-31 | 1 | -2/+2 | |
| | | | | | | | | This is a performance/GC optimisation. In theory, this could be optimised by the implementation (last time I checked, this would have no effect on JRuby). But in practise, this make attribute access faster. | |||||
* | Move the serialized attribute tests to their own file | Rafael Mendonça França | 2012-08-26 | 1 | -11/+0 | |
| | ||||||
* | Move test to the correct file | Rafael Mendonça França | 2012-08-26 | 1 | -0/+6 | |
| | ||||||
* | Refactor missing parameter validation based on position | Carlos Antonio da Silva | 2012-08-18 | 1 | -1/+0 | |
| | ||||||
* | load active_support/core_ext/object/inclusion in active_support/rails | Xavier Noria | 2012-08-02 | 1 | -1/+0 | |
| | ||||||
* | Deprecate ActiveRecord::Base.scoped. | Jon Leighton | 2012-07-27 | 1 | -4/+4 | |
| | | | | | | | It doesn't serve much purpose now that ActiveRecord::Base.all returns a Relation. The code is moved to active_record_deprecated_finders. | |||||
* | Fix class_eval without __FILE__ and __LINE__. | kennyj | 2012-07-18 | 1 | -1/+1 | |
| | ||||||
* | Move newly added multiparameter tests to base_test file | Carlos Antonio da Silva | 2012-06-28 | 1 | -35/+1 | |
| | | | | | All other multiparameter assignment tests are in the same file, so it makes sense to have all of them here. | |||||
* | Skip test_setting_time_attribute because Oracle database | Yasuo Honda | 2012-06-28 | 1 | -0/+2 | |
| | | | | does not have TIME data type. | |||||
* | Fixes #6825, adds tests covering cases and error possibilities, also changes ↵ | Mauricio Linhares | 2012-06-27 | 1 | -0/+33 | |
| | | | | SQLite3 driver to correctly generate a time column instead of datetime | |||||
* | Remove unused assignments from activerecord tests | Mark Rushakoff | 2012-04-29 | 1 | -2/+2 | |
| | ||||||
* | %s/find(:\(first\|last\|all\), \([^()]*\))/scoped(\2).\1/gcI amongst other ↵ | Jon Leighton | 2012-04-27 | 1 | -4/+4 | |
| | | | | things | |||||
* | remove calls to find(:first), find(:last) and find(:all) | Jon Leighton | 2012-04-26 | 1 | -1/+1 | |
| | ||||||
* | attribute_present? should return false for empty strings | Jacob Green | 2012-03-06 | 1 | -0/+3 | |
| | ||||||
* | Fix warning: assigned but unused variable | kennyj | 2012-03-01 | 1 | -3/+0 | |
| | ||||||
* | force datetime attributes to be changed, fix GH #3965 | Sergey Nartimov | 2012-02-28 | 1 | -0/+13 | |
| | ||||||
* | move id_before_type_cast to PrimaryKey module | Sergey Nartimov | 2012-02-16 | 1 | -0/+8 | |
| | ||||||
* | Revert PR#5001 "removed unuseful require for reply as we are not using ↵ | Vishnu Atrai | 2012-02-12 | 1 | -0/+1 | |
| | | | | | | require model" This reverts commit 7381596ea32aff47222af89e4bc66000f4597acb. | |||||
* | removed unuseful require for reply as we are not using require model | Karunakar (Ruby) | 2012-02-11 | 1 | -1/+0 | |
| | ||||||
* | fixing the list of cached columns | Aaron Patterson | 2012-02-09 | 1 | -1/+3 | |
| | ||||||
* | only exclude serialized columns from cacheable columns | Aaron Patterson | 2012-02-09 | 1 | -1/+1 | |
| | ||||||
* | Test that #[] and #[]= keep working when #read_attribute and ↵ | Tom Stuart | 2012-01-11 | 1 | -0/+33 | |
| | | | | #write_attribute are overridden | |||||
* | Test ActiveRecord::Base#[]= as well as #write_attribute | Tom Stuart | 2012-01-11 | 1 | -1/+7 | |
| | ||||||
* | Fix #4046. | Jon Leighton | 2011-12-23 | 1 | -0/+20 | |
| | ||||||
* | remove deprecated underscore versions of attribute methods | Sergey Nartimov | 2011-12-21 | 1 | -8/+0 | |
| | ||||||
* | handle not only strings in date type cast | Sergey Nartimov | 2011-12-19 | 1 | -0/+8 | |
| | ||||||
* | Stop the build asploding on 1.8.7 | Jon Leighton | 2011-12-14 | 1 | -0/+4 | |
| | ||||||
* | Fix #3837. | Jon Leighton | 2011-12-03 | 1 | -0/+12 | |
| | | | | | We also need to time zone convert time zone aware attributes when accessed via read_attribute, not only when via direct access. | |||||
* | Get rid of the underscore versions of attribute methods! | Jon Leighton | 2011-12-01 | 1 | -0/+5 | |
| | | | | This makes me happy! | |||||
* | don't alter global state in test | Jon Leighton | 2011-11-30 | 1 | -2/+3 | |
| | ||||||
* | If the table behind has no primary key, do not ask again and just return nil. | Julius de Bruijn | 2011-11-30 | 1 | -0/+5 | |
| | ||||||
* | omg computer science! | Jon Leighton | 2011-11-30 | 1 | -5/+1 | |
| | | | | | | | Implement a mini state machine for serialized attributes. This means we do not have to deserialize the values upon initialization, which means that if we never actually access the attribute, we never have to deserialize it. | |||||
* | Use inheritance to avoid special-case code for the 'id' method | Jon Leighton | 2011-11-30 | 1 | -1/+1 | |
| | ||||||
* | Update ActiveRecord#attribute_present? to work as documented | Justin Mazzi | 2011-11-05 | 1 | -0/+18 | |
| | | | | | | | "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 | |||||
* | Stop trying to be clever about when to define attribute methods. | Jon Leighton | 2011-09-14 | 1 | -24/+0 | |
| | | | | | | | | | There is no meaningful performance penalty in defining attribute methods, since it only happens once. There is also no reason *not* to define them, since they get thrown in an included module, so they will not 'overwrite' anything. In fact, this is desirable, since it allows us to call super. |