Abstracted Recordable concern

Blogs, streams, and updates are all recordable, and there's no need to repeat code in the models for this.

4 files changed, 14 insertions, 9 deletions

@@ -1,12 +1,10 @@
 class Blog < ApplicationRecord
-  has_many :records, as: :recordable, inverse_of: :recordable
+  include Recordable
 
   validates :title, presence: true
   validates :body, presence: true, if: :published
   validates :slug, presence: true, format: /\A[-a-z0-9]+\z/, if: :published
 
-  accepts_nested_attributes_for :records, allow_destroy: true
-
   before_validation :set_draft_title
   before_save :set_published_at
 
@@ -0,0 +1,9 @@
+module Recordable
+  extend ActiveSupport::Concern
+
+  included do
+    has_many :records, as: :recordable, inverse_of: :recordable
+
+    accepts_nested_attributes_for :records, allow_destroy: true
+  end
+end
@@ -1,12 +1,11 @@
 class Stream < ApplicationRecord
-  has_many :records, as: :recordable, inverse_of: :recordable
+  include Recordable
+
   has_many :updates
 
   validates :title, presence: true
   validates :slug, presence: true, format: /\A[-a-z0-9]+\z/
 
-  accepts_nested_attributes_for :records, allow_destroy: true
-
   def path
     "/thinks/#{slug}"
   end
@@ -1,11 +1,10 @@
 class Update < ApplicationRecord
-  has_many :records, as: :recordable, inverse_of: :recordable
+  include Recordable
+
   belongs_to :stream
 
   validates :stream, :body, presence: true
 
-  accepts_nested_attributes_for :records, allow_destroy: true
-
   def path
     "/thinks/#{stream.slug}\#update-#{id}"
   end