aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/CHANGELOG.md
Commit message (Collapse)AuthorAgeFilesLines
* Mute psql output when running rake db:schema:loadGodfrey Chan2013-04-301-0/+4
|
* Fix #8856 Ensure has_one association=(associate) triggers save.Chris Thompson2013-04-301-0/+6
| | | | | | | | | | | | | | | | | | | | | | | activerecord/lib/active_record/associations.rb states: # [association=(associate)] # Assigns the associate object, extracts the primary key, sets it as the foreign key, # and saves the associate object. Since commit 42dd5d9f2976677a4bf22347f2dde1a8135dfbb4 to fix #7191, this is no longer the case if the associate has changed, but is the same object. For example: # Pirate has_one :ship pirate = Pirate.create!(catchphrase: "A Pirate") ship = pirate.build_ship(name: 'old name') ship.save! ship.name = 'new name' pirate.ship = ship That last line should trigger a save. Although we are not changing the association, the associate (ship) has changed.
* Abort a rake task when missing db/structure.sql like `db:schema:load` task.kennyj2013-05-011-1/+3
|
* rails/master is now 4.1.0.betaRafael Mendonça França2013-04-291-2043/+2
|
* Minor AR changelog review [ci skip]Carlos Antonio da Silva2013-04-241-6/+8
|
* Merge branch 'active-record-standalone-improvements'Piotr Sarnacki2013-04-241-0/+2
|\
| * Move CHANGELOG entry for db.rake improvements to 4.0.0Piotr Sarnacki2013-04-241-3/+1
| |
| * Don't require having Rails.application to run database tasksPiotr Sarnacki2013-04-241-0/+4
| |
* | Added testcase for #10067 and a CHANGELOG entry about this change.kennyj2013-04-251-0/+7
| |
* | Revert "Revert "`belongs_to :touch` behavior now touches old association ↵Andrew White2013-04-241-0/+20
|/ | | | | | | | | when transitioning to new association" until a proper fix is found for #10197" This reverts commit 7389df139a35436f00876c96d20e81ba23c93f0a. Conflicts: activerecord/test/cases/timestamp_test.rb
* Revert "`belongs_to :touch` behavior now touches old association when ↵David Heinemeier Hansson2013-04-231-20/+0
| | | | transitioning to new association" until a proper fix is found for #10197
* Update counter cache when pushing into associationMatthew Robertson2013-04-211-0/+9
| | | | | | | | | | | | | | | | This commit fixes a regression bug in which counter_cache columns were not being updated correctly when newly created records were being pushed into an assocation. EG: # this was fine @post.comment.create! # this was fine @comment = Comment.first @post.comments << @comment # this would not update counters @post.comments << Comment.create!
* Rewrites a CHANGELOG entry.Xavier Noria2013-04-201-16/+6
| | | | | | | | | | | | | | | | | The entry is basically copy & paste of the commit message, but the CHANGELOG has a different purpose than Git history, it just communicates what is new: * No need to explain why did the bug happen (unless it is truly relevant). * No need to explain how was the bug fixed. * Whether the user gives new names to columns does not really matter, use of select to cherry-pick a column for example also presented that behaviour. Non-selected attributes are the key, either because they were not included in the selected list, or because they were but with a different alias. * In the case of an attribute alias, what you really want to depict is that respond_to? returns false for the original attribute name.
* Typo Fix in AR CHANGELOG [ci skip]Prathamesh Sonpatki2013-04-201-2/+2
|
* fix respond_to? for non selected columnNeeraj Singh2013-04-191-0/+23
| | | | | | | | | | | | | | | | | | | | | | | fixes #4208 If a query selects only a few columns and gives custom names to those columns then respond_to? was returning true for the non selected columns. However calling those non selected columns raises exception. post = Post.select("'title' as post_title").first In the above case when `post.body` is invoked then an exception is raised since `body` attribute is not selected. Howevere `respond_to?` did not behave correctly. pos.respond_to?(:body) #=> true Reason was that Active Record calls `super` to pass the call to Active Model and all the columns are defined on Active Model. Fix is to actually check if the data returned from the db contains the data for column in question.
* Revert "Merge pull request #10183 from jholton/fix_association_auto_save"Jon Leighton2013-04-191-7/+0
| | | | | | | This reverts commit e8727d37fc49d5bf9976c3cb5c46badb92cf4ced, reversing changes made to d098e1c24bc145e0cc14532348436e14dc46d375. Reason: it broke the mysql build
* destroys association records before saving/inserting new association recordsJohnny Holton2013-04-191-0/+7
| | | | | | | | | | | | | | | | | | | | | fixes bug introduced by #3329 These are the conditions necessary to reproduce the bug: - For an association, autosave => true. - An association record is being destroyed - A new association record is being created. - There is a unique index one of the association's fields. - The record being created has the same value as the record being destroyed on the indexed field. Before, the deletion of records was postponed until after all insertions/saves. Therefore the new record with the identical value in the indexed field caused a non-unique value error to be thrown at the database level. With this fix, the deletions happen first, before the insertions/saves. Therefore the record with the duplicate value is gone from the database before the new record is created, thereby avoiding the non-uniuqe value error.
* Merge pull request #10217 from mirasrael/make-migrator-run-transactional-4.0Rafael Mendonça França2013-04-191-0/+4
|\ | | | | | | | | | | | | Support transactions in Migrator.run Conflicts: activerecord/CHANGELOG.md
| * Support transactions in Migrator.runbondarev2013-04-181-0/+4
| |
* | Fix #7619. 0x prefix must be added when assigning hexadecimal string into ↵kennyj2013-04-171-0/+4
| | | | | | | | bit column in Postgresql, because solving ambiguity.
* | Fixed typos in AR CHANGELOGPrathamesh Sonpatki2013-04-161-4/+4
|/
* use unified and clean formatting in CHANGELOGS. [ci skip]Yves Senn2013-04-121-9/+9
|
* Switched to new naming conventionsNoemj2013-04-111-3/+3
| | | | [ci skip]
* Merge pull request #10152 from Noemj/statement_cacheRafael Mendonça França2013-04-101-0/+17
|\ | | | | | | | | | | | | Statement cache Conflicts: activerecord/CHANGELOG.md
| * Added statement cacheNoemj2013-04-101-0/+17
| |
* | While merging relations preserve context for joinsJared Armstrong and Neeraj Singh2013-04-101-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #3002. Also see #5494. ``` class Comment < ActiveRecord::Base belongs_to :post end class Author < ActiveRecord::Base has_many :posts end class Post < ActiveRecord::Base belongs_to :author has_many :comments end ``` `Comment.joins(:post).merge(Post.joins(:author).merge(Author.where(:name => "Joe Blogs"))).all` would fail with `ActiveRecord::ConfigurationError: Association named 'author' was not found on Comment`. It is failing because `all` is being called on relation which looks like this after all the merging: `{:joins=>[:post, :author], :where=>[#<Arel::Nodes::Equality: ....}`. In this relation all the context that `Post` was joined with `Author` is lost and hence the error that `author` was not found on `Comment`. Ths solution is to build JoinAssociation when two relations with join information are being merged. And later while building the arel use the previously built `JoinAssociation` record in `JoinDependency#graft` to build the right from clause. Thanks to Jared Armstrong (https://github.com/armstrjare) for most of the work. I ported it to make it compatible with new code base.
* | Minor AR changelog improvements [ci skip]Carlos Antonio da Silva2013-04-071-4/+3
| |
* | Deprecate ActiveRecord#Base.default_scopes?Agis-2013-04-061-0/+5
| | | | | | | | See #10107.
* | Merge pull request #10098 from subwindow/postgres_dump_bigint_defaultRafael Mendonça França2013-04-051-0/+5
|\ \ | | | | | | | | | | | | | | | | | | Correctly parse bigint defaults in PostgreSQL Conflicts: activerecord/CHANGELOG.md
| * | Correctly parse bigint defaults in PostgreSQLErik Peterson2013-04-051-0/+5
| | |
* | | Move changelog to the top [ci skip]Carlos Antonio da Silva2013-04-051-17/+17
| | |
* | | Merge pull request #9996 from mikz/masterJon Leighton2013-04-051-0/+17
|\ \ \ | |/ / |/| | Association with inverse_of does not set the parent in association building block
| * | fix inverse_of association in block of new childMichal Cichra2013-04-011-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes inconsistency when building children of association which has inverse_of set properly. When creating new association object with a block: parent.association.build do |child| child.parent.equal?(parent) # false end So the block the `child.parent` did not point to the same object. But when the object is created it points to same instance: child = parent.association.build child.parent.equal?(parent) # true
* | | has_many through obeys order on through associationNeeraj Singh2013-04-041-0/+5
| | | | | | | | | | | | fixes #10016
* | | Move changelog entry to the top and fix conflict [ci skip]Carlos Antonio da Silva2013-04-031-22/+20
| | |
* | | Merge pull request #9141 from adamgamble/issue-9091David Heinemeier Hansson2013-04-031-0/+22
|\ \ \ | | | | | | | | belongs_to :touch should touch old record when transitioning.
| * | | Modifies belong_to touch callback to touch old associations also #9091Adam Gamble2013-03-141-0/+22
| | | |
* | | | Add CHANGELOG entry about extracting and deprecating ↵kennyj2013-04-031-0/+5
| | | | | | | | | | | | | | | | Firebird/Sqlserver/Oracle database tasks.
* | | | Minor reviews in AR changelog [ci skip]Carlos Antonio da Silva2013-03-301-10/+10
| |/ / |/| |
* | | Add support for FULLTEXT and SPATIAL indexes using the :type flag for MySQL.Ken Mazaika2013-03-271-1/+9
| | |
* | | Make connection_handler overridable per threadSam Saffron2013-03-271-1/+7
| | |
* | | Merge pull request #9928 from vipulnsward/fix_rename_auto_increment_mysqlRafael Mendonça França2013-03-261-1/+6
|\ \ \ | | | | | | | | respect auto_increment in rename_column for mysql
| * | | respect auto_increment in rename_column for mysqlVipul A M2013-03-271-1/+6
| | | |
* | | | Adds support for concurrent indexing in PostgreSQL adapterDan McClain2013-03-251-0/+12
|/ / / | | | | | | | | | | | | | | | | | | | | | Adds support for algorithm option in MySQL indexes Moves USING and algorithm options upstream The syntax is still specific to the Adapter, so the actual executed string happens in the corresponding adapter
* | | Make Postgres point type correspond to ruby array with two floats insideMartin Schürrer2013-03-251-0/+4
| | |
* | | Custom index type support with :using.doabit2013-03-241-0/+7
| | |
* | | Review some CHANGELOG entries [ci skip]Rafael Mendonça França2013-03-231-7/+8
| | |
* | | Spelling 'PostgreSQL' correctlyPrathamesh Sonpatki2013-03-231-7/+7
| | |
* | | Fixed typos in activerecord CHANGELOGPrathamesh Sonpatki2013-03-231-3/+3
| | |
* | | When we pass id to update_attributes it will try to set new id for that recordDmitry Vorotilin2013-03-221-0/+7
| | |