mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
2dd81877a4
`has_many_inversing` adds records to a has_many association. It does so with destroyed records, too. So if a child was destroyed with a `touch: true` association on the parent *and* the parent was not loaded, it tried to load the parent to touch it. While loading the parent it added the child record to the parent's has_many association. The logic doing this always set the child's parent id – even if it was correct/the same already. But since the child is destroyed, it resulted in a `FrozenError`. This commit prevents doing the unnecessary setting of the identical id and therefore fixes this error. Fixes #40943 by not doing an unneeded attribute set. Closes #40969. [Markus Doits + Rafael Mendonça França]
33 lines
1.1 KiB
Ruby
33 lines
1.1 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
class Book < ActiveRecord::Base
|
|
belongs_to :author
|
|
|
|
has_many :citations, foreign_key: "book1_id", inverse_of: :book
|
|
has_many :references, -> { distinct }, through: :citations, source: :reference_of
|
|
|
|
has_many :subscriptions
|
|
has_many :subscribers, through: :subscriptions
|
|
|
|
enum status: [:proposed, :written, :published]
|
|
enum last_read: { unread: 0, reading: 2, read: 3, forgotten: nil }
|
|
enum nullable_status: [:single, :married]
|
|
enum language: [:english, :spanish, :french], _prefix: :in
|
|
enum author_visibility: [:visible, :invisible], _prefix: true
|
|
enum illustrator_visibility: [:visible, :invisible], _prefix: true
|
|
enum font_size: [:small, :medium, :large], _prefix: :with, _suffix: true
|
|
enum difficulty: [:easy, :medium, :hard], _suffix: :to_read
|
|
enum cover: { hard: "hard", soft: "soft" }
|
|
enum boolean_status: { enabled: true, disabled: false }
|
|
|
|
def published!
|
|
super
|
|
"do publish work..."
|
|
end
|
|
end
|
|
|
|
class PublishedBook < ActiveRecord::Base
|
|
self.table_name = "books"
|
|
|
|
validates_uniqueness_of :isbn
|
|
end
|