File post_creator.rb has 410 lines of code (exceeds 250 allowed). Consider refactoring.

Open

require_dependency 'rate_limiter'
require_dependency 'topic_creator'
require_dependency 'post_jobs_enqueuer'
require_dependency 'distributed_mutex'
require_dependency 'has_errors'

Class PostCreator has 37 methods (exceeds 20 allowed). Consider refactoring.

Open

class PostCreator
  include HasErrors

  attr_reader :opts

Method valid? has a Cognitive Complexity of 20 (exceeds 5 allowed). Consider refactoring.

Open

  def valid?
    @topic = nil
    @post = nil

    if @user.suspended? && !skip_validations?

• Read up
  Read up

Method create has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

  def create
    if valid?
      transaction do
        build_post_stats
        create_topic

• Read up
  Read up

Method valid? has 56 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def valid?
    @topic = nil
    @post = nil

    if @user.suspended? && !skip_validations?

Method create has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def create
    if valid?
      transaction do
        build_post_stats
        create_topic

Method update_user_counts has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

  def update_user_counts
    return if @opts[:import_mode]

    @user.create_user_stat if @user.user_stat.nil?

• Read up
  Read up

Method auto_close has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def auto_close
    topic = @post.topic
    is_private_message = topic.private_message?
    topic_posts_count = @post.topic.posts_count

Method initialize has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

  def initialize(user, opts)
    # TODO: we should reload user in case it is tainted, should take in a user_id as opposed to user
    # If we don't do this we introduce a rather risky dependency
    @user = user
    @opts = opts || {}

• Read up
  Read up

Method update_topic_stats has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

  def update_topic_stats
    attrs = { updated_at: Time.now }

    if @post.post_type != Post.types[:whisper]
      attrs[:last_posted_at] = @post.created_at

• Read up
  Read up

Avoid too many return statements within this method.

Open

        return false

Avoid too many return statements within this method.

Open

    return true if skip_validations?

Avoid too many return statements within this method.

Open

      return false

Method set_reply_info has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def self.set_reply_info(post)
    return unless post.reply_to_post_number.present?

    reply_info = Post.where(topic_id: post.topic_id, post_number: post.reply_to_post_number)
      .select(:user_id, :post_type)

• Read up
  Read up

Method setup_post has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def setup_post
    @opts[:raw] = TextCleaner.normalize_whitespaces(@opts[:raw] || '').gsub(/\s+\z/, "")

    post = Post.new(raw: @opts[:raw],
                    topic_id: @topic.try(:id),

• Read up
  Read up

Method ensure_in_allowed_users has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def ensure_in_allowed_users
    return unless @topic.private_message? && @topic.id

    unless @topic.topic_allowed_users.where(user_id: @user.id).exists?
      unless @topic.topic_allowed_groups.where('group_id IN (

• Read up
  Read up

Method update_topic_auto_close has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

  def update_topic_auto_close
    return if @opts[:import_mode]

    if @topic.closed?
      @topic.delete_topic_timer(TopicTimer.types[:close])

• Read up
  Read up