Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove conditional adding a new method | Rafael Mendonça França | 2013-09-11 | 1 | -5/+5 |
| | | | | | | At the point we want to add the reflection we already know the reflection is of the AggregateReflection type so we can call a specific method | ||||
* | support anonymous classes on has_many associations | Aaron Patterson | 2013-09-03 | 1 | -0/+1 |
| | |||||
* | pk should not be required for hm:t associations | Aaron Patterson | 2013-08-29 | 1 | -1/+0 |
| | |||||
* | Fixing multi-word automatic inverse detection. | wangjohn | 2013-08-15 | 1 | -2/+2 |
| | | | | | Currently, ActiveRecord models with multiple words cannot have their inverse associations detected automatically. | ||||
* | Revert "all `scope_chain` methods allocate new arrays, no need to dup" | Aaron Patterson | 2013-07-31 | 1 | -1/+1 |
| | | | | This reverts commit 1468a4b89aa4bca99160bfa03572b2c0ab348db5. | ||||
* | all `scope_chain` methods allocate new arrays, no need to dup | Aaron Patterson | 2013-07-30 | 1 | -1/+1 |
| | |||||
* | oops! | Aaron Patterson | 2013-07-23 | 1 | -1/+1 |
| | |||||
* | eliminate conditional in JoinHelper | Aaron Patterson | 2013-07-23 | 1 | -1/+1 |
| | |||||
* | acually get the actual source reflection :cry::gun: | Aaron Patterson | 2013-07-23 | 1 | -7/+14 |
| | |||||
* | factory methods should not have side effects. | Aaron Patterson | 2013-07-23 | 1 | -3/+1 |
| | | | | Move model mutation to the methods that are called on the model. | ||||
* | extract mutation from generation method | Aaron Patterson | 2013-07-23 | 1 | -3/+5 |
| | |||||
* | habtm can only ever be an association reflection | Aaron Patterson | 2013-07-22 | 1 | -2/+4 |
| | |||||
* | use the predicate method | Aaron Patterson | 2013-07-22 | 1 | -1/+1 |
| | |||||
* | AR::Base does not need to know how to create reflection objects | Aaron Patterson | 2013-07-22 | 1 | -19/+19 |
| | |||||
* | use the superclass implementation | Aaron Patterson | 2013-07-18 | 1 | -1/+3 |
| | |||||
* | quoted table name is also cached | Aaron Patterson | 2013-06-13 | 1 | -1/+1 |
| | |||||
* | table name is cached on the class, so stop caching twice | Aaron Patterson | 2013-06-13 | 1 | -1/+1 |
| | |||||
* | calculate types on construction | Aaron Patterson | 2013-06-13 | 1 | -8/+4 |
| | |||||
* | only cache the primary key column in one place | Aaron Patterson | 2013-06-13 | 1 | -1/+1 |
| | |||||
* | these methods are never called, so remove them | Aaron Patterson | 2013-06-13 | 1 | -8/+0 |
| | |||||
* | This test does not test anything that happens in the real world. If you | Aaron Patterson | 2013-06-13 | 1 | -7/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | recreate the models without mucking with internal caches of the relation objects, then the test fails. For example: class Man < ActiveRecord::Base has_many :interests end class Interest < ActiveRecord::Base belongs_to :man end Then do this test: def test_validate_presence_of_parent_fails_without_inverse_of repair_validations(Interest) do Interest.validates_presence_of(:man) assert_no_difference ['Man.count', 'Interest.count'] do man = Man.create(:name => 'John', :interests_attributes => [{:topic=>'Cars'}, {:topic=>'Sports'}]) assert_not_predicate man.errors[:"interests.man"], :empty? end end end The test will fail. This is a bad test, so I am removing it. | ||||
* | oops. step away from the keyboard aaron. :cry: | Aaron Patterson | 2013-06-13 | 1 | -1/+1 |
| | |||||
* | fix caching of automatic inverse of. :bomb: | Aaron Patterson | 2013-06-13 | 1 | -1/+3 |
| | |||||
* | let the object stay in charge of internal cache invalidation | Aaron Patterson | 2013-06-13 | 1 | -7/+5 |
| | |||||
* | reduce automatic_inverse_of caching logic | Aaron Patterson | 2013-06-13 | 1 | -30/+16 |
| | |||||
* | active_record should always be set. Do or do not, there is no try | Aaron Patterson | 2013-06-13 | 1 | -1/+1 |
| | |||||
* | push ambiguous reflection warning down to reflection name calculation | Aaron Patterson | 2013-06-13 | 1 | -23/+28 |
| | |||||
* | clean up ivar assignment | Aaron Patterson | 2013-06-13 | 1 | -3/+2 |
| | |||||
* | Ambiguous reflections are on :through relationships are no longer supported. | Aaron Patterson | 2013-06-13 | 1 | -1/+27 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For example, you need to change this: class Author < ActiveRecord::Base has_many :posts has_many :taggings, :through => :posts end class Post < ActiveRecord::Base has_one :tagging has_many :taggings end class Tagging < ActiveRecord::Base end To this: class Author < ActiveRecord::Base has_many :posts has_many :taggings, :through => :posts, :source => :tagging end class Post < ActiveRecord::Base has_one :tagging has_many :taggings end class Tagging < ActiveRecord::Base end | ||||
* | sometimes singularize does not work, so we get a list of two strings. just ↵ | Aaron Patterson | 2013-06-12 | 1 | -1/+1 |
| | | | | uniq them | ||||
* | no need to cache hash lookups | Aaron Patterson | 2013-06-12 | 1 | -1/+1 |
| | |||||
* | split aggregates from association reflections to avoid is_a checks later | Aaron Patterson | 2013-06-12 | 1 | -5/+11 |
| | |||||
* | remove unnecessary is_a check | Aaron Patterson | 2013-06-12 | 1 | -2/+1 |
| | |||||
* | Getting rid of the +automatic_inverse_of: false+ option in associations in favor | wangjohn | 2013-06-08 | 1 | -7/+7 |
| | | | | | of using +inverse_of: false+ option. Changing the documentation and adding a CHANGELOG entry for the automatic inverse detection feature. | ||||
* | remove code duplication | Neeraj Singh | 2013-05-22 | 1 | -2/+3 |
| | |||||
* | Some editorial changes on the documentation. | Rafael Mendonça França | 2013-05-12 | 1 | -4/+4 |
| | | | | | | | | * Remove some autolinks * Fix the rendered result * Change sql to SQL [ci skip] | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-05-12 | 1 | -4/+4 |
|\ | | | | | | | | | Conflicts: activesupport/lib/active_support/callbacks.rb | ||||
| * | minor rdoc cleanup for reflection methods | Neeraj Singh | 2013-05-09 | 1 | -4/+4 |
| | | |||||
* | | Adding documentation to the automatic inverse_of finder. | wangjohn | 2013-05-11 | 1 | -1/+1 |
| | | |||||
* | | Created a method to automatically find inverse associations and cache | wangjohn | 2013-05-07 | 1 | -3/+98 |
|/ | | | | | | the results. Added tests to check to make sure that inverse associations are automatically found when has_many, has_one, or belongs_to associations are defined. | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2013-04-11 | 1 | -1/+6 |
|\ | | | | | | | | | Conflicts: guides/source/action_mailer_basics.md | ||||
| * | minor copy editing | Neeraj Singh | 2013-03-30 | 1 | -1/+6 |
| | | |||||
* | | Use a space after the comment sign when showing the result of commands | Carlos Antonio da Silva | 2013-04-04 | 1 | -4/+4 |
| | | | | | | | | | | | | http://guides.rubyonrails.org/api_documentation_guidelines.html#example-code [ci skip] | ||||
* | | expanded rdoc for chain and reflection_names | Neeraj Singh | 2013-04-04 | 1 | -2/+20 |
| | | |||||
* | | expanded rdoc on source_reflection method | Neeraj Singh | 2013-04-04 | 1 | -0/+10 |
|/ | |||||
* | Refactor to use each_key, remove extra spaces | Carlos Antonio da Silva | 2013-01-28 | 1 | -1/+0 |
| | |||||
* | Added STI support to init and building associations | Jason Rush | 2012-11-29 | 1 | -1/+1 |
| | | | | | | | | Allows you to do BaseClass.new(:type => "SubClass") as well as parent.children.build(:type => "SubClass") or parent.build_child to initialize an STI subclass. Ensures that the class name is a valid class and that it is in the ancestors of the super class that the association is expecting. | ||||
* | Another batch of hash syntax changes to comment, this time around, I tried ↵ | AvnerCohen | 2012-10-23 | 1 | -12/+12 |
| | | | | to keep 'output' messages untouched. | ||||
* | Remove mass_assignment_options from ActiveRecord | Guillermo Iguaran | 2012-09-16 | 1 | -2/+2 |
| | |||||
* | Refactor to remove some duplication | Jon Leighton | 2012-09-12 | 1 | -0/+4 |
| |