ee0a521a83
* Favoring delete_all on user relationships Prior to this commit, several of the user's "has_many" were marked as `depenedent: :destroy`. In the case of :destroy, ActiveRecord instantiates each object and then runs destroy. Whereas in the case of :delete, ActiveRecord issues a SQL command to delete the related files. It is often "safer" to use :destroy, as it guarantees that you'll instantiate the record and run it's callbacks. But sometimes you have to go with the speed of SQL. And while not directly related to #15424 it is representative of our callback ecosystem creating some unexpected computational loads. Related to #15442 and #15424 * Noting models that user no longer cascade destroys
45 lines
1.9 KiB
Ruby
45 lines
1.9 KiB
Ruby
# @note When we destroy the related user, it's using dependent:
|
|
# :delete for the relationship. That means no before/after
|
|
# destroy callbacks will be called on this object.
|
|
class ResponseTemplate < ApplicationRecord
|
|
resourcify
|
|
|
|
belongs_to :user, optional: true
|
|
|
|
TYPE_OF_TYPES = %w[personal_comment mod_comment abuse_report_email_reply email_reply tag_adjustment].freeze
|
|
USER_NIL_TYPE_OF_TYPES = %w[mod_comment abuse_report_email_reply email_reply tag_adjustment].freeze
|
|
CONTENT_TYPES = %w[plain_text html body_markdown].freeze
|
|
COMMENT_CONTENT_TYPE = %w[body_markdown].freeze
|
|
EMAIL_CONTENT_TYPES = %w[plain_text html].freeze
|
|
COMMENT_VALIDATION_MSG = "Comment templates must use Markdown as its content type.".freeze
|
|
EMAIL_VALIDATION_MSG = "Email templates must use plain text or HTML as its content type.".freeze
|
|
USER_NIL_TYPE_OF_MSG = "cannot have a user ID associated.".freeze
|
|
|
|
validates :type_of, :content_type, :content, :title, presence: true
|
|
validates :content, uniqueness: { scope: %i[user_id type_of content_type] }
|
|
validates :type_of, inclusion: { in: TYPE_OF_TYPES }
|
|
validates :content_type, inclusion: { in: CONTENT_TYPES }
|
|
validates :content_type,
|
|
inclusion: { in: COMMENT_CONTENT_TYPE, message: COMMENT_VALIDATION_MSG },
|
|
if: -> { type_of&.include?("comment") }
|
|
validates :content_type,
|
|
inclusion: { in: EMAIL_CONTENT_TYPES, message: EMAIL_VALIDATION_MSG },
|
|
if: -> { type_of&.include?("email") }
|
|
validate :user_nil_only_for_user_nil_types
|
|
validate :template_count
|
|
|
|
private
|
|
|
|
def user_nil_only_for_user_nil_types
|
|
return unless user_id.present? && USER_NIL_TYPE_OF_TYPES.include?(type_of)
|
|
|
|
errors.add(:type_of, USER_NIL_TYPE_OF_MSG)
|
|
end
|
|
|
|
def template_count
|
|
return unless user
|
|
return if user.trusted || user.response_templates.count <= 30
|
|
|
|
errors.add(:user, "Response template limit of 30 per user has been reached")
|
|
end
|
|
end
|