File mail_handler.rb has 537 lines of code (exceeds 250 allowed). Consider refactoring.

Open

class MailHandler < ActionMailer::Base
  include ActionView::Helpers::SanitizeHelper
  include Redmine::I18n

  class UnauthorizedAction < StandardError; end

Class MailHandler has 35 methods (exceeds 20 allowed). Consider refactoring.

Open

class MailHandler < ActionMailer::Base
  include ActionView::Helpers::SanitizeHelper
  include Redmine::I18n

  class UnauthorizedAction < StandardError; end

Method receive has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.

Open

  def receive(email, options={})
    @email = email
    @handler_options = options
    sender_email = email.from.to_a.first.to_s.strip
    # Ignore emails received from the application emission address to avoid hell cycles

• Read up
  Read up

Method get_project_from_receiver_addresses has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

  def get_project_from_receiver_addresses
    local, domain = handler_options[:project_from_subaddress].to_s.split("@")
    return nil unless local && domain

    local = Regexp.escape(local)

• Read up
  Read up

Method receive has 46 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def receive(email, options={})
    @email = email
    @handler_options = options
    sender_email = email.from.to_a.first.to_s.strip
    # Ignore emails received from the application emission address to avoid hell cycles

Method new_user_from_attributes has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    def new_user_from_attributes(email_address, fullname=nil)
      user = User.new

      # Truncating the email address would result in an invalid format
      user.mail = email_address

• Read up
  Read up

Method receive_issue has a Cognitive Complexity of 13 (exceeds 5 allowed). Consider refactoring.

Open

  def receive_issue
    project = target_project

    # Never receive emails to projects where adding issues is not possible
    raise NotAllowedInProject, "not possible to add issues to project [#{project.name}]" unless project.allows_to?(:add_issues)

• Read up
  Read up

Method add_attachments has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

  def add_attachments(obj)
    if email.attachments && email.attachments.any?
      email.attachments.each do |attachment|
        next unless accept_attachment?(attachment)
        next unless attachment.body.decoded.size > 0

• Read up
  Read up

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

Open

  def plain_text_body
    return @plain_text_body unless @plain_text_body.nil?

    parse_order =
      if Setting.mail_handler_preferred_body_part == 'html'

• Read up
  Read up

Method receive_issue has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def receive_issue
    project = target_project

    # Never receive emails to projects where adding issues is not possible
    raise NotAllowedInProject, "not possible to add issues to project [#{project.name}]" unless project.allows_to?(:add_issues)

Method dispatch has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

  def dispatch
    headers = [email.in_reply_to, email.references].flatten.compact
    subject = email.subject.to_s
    if headers.detect {|h| h.to_s =~ MESSAGE_ID_RE}
      klass, object_id = $1, $2.to_i

Method get_keyword has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

  def get_keyword(attr, options={})
    @keywords ||= {}
    if @keywords.has_key?(attr)
      @keywords[attr]
    else

• Read up
  Read up

Method dispatch has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

  def dispatch
    headers = [email.in_reply_to, email.references].flatten.compact
    subject = email.subject.to_s
    if headers.detect {|h| h.to_s =~ MESSAGE_ID_RE}
      klass, object_id = $1, $2.to_i

• Read up
  Read up

Method cleanup_body has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

  def cleanup_body(body)
    delimiters = Setting.mail_handler_body_delimiters.to_s.split(/[\r\n]+/).reject(&:blank?)

    if Setting.mail_handler_enable_regex_delimiters?
      begin

• Read up
  Read up

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

Open

  def receive_issue_reply(issue_id, from_journal=nil)
    issue = Issue.find_by(:id => issue_id)
    if issue.nil?
      raise MissingContainer, "reply to nonexistant issue [##{issue_id}]"
    end

• Read up
  Read up

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

Open

  def receive_message_reply(message_id)
    message = Message.find_by(:id => message_id)&.root
    if message.nil?
      raise MissingContainer, "reply to nonexistant message [#{message_id}]"
    end

• Read up
  Read up

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

Open

  def extract_keyword!(text, attr, format=nil)
    keys = [attr.to_s.humanize]
    if attr.is_a?(Symbol)
      if user && user.language.present?
        keys << l("field_#{attr}", :default => '', :locale =>  user.language)

• Read up
  Read up

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

Open

  def create_user_from_email
    if from_addr = email.header['from'].try(:addrs).to_a.first
      addr = from_addr.address
      name = from_addr.display_name || from_addr.comments.to_a.first
      user = self.class.new_user_from_attributes(addr, name)

• Read up
  Read up

Avoid too many return statements within this method.

Open

        return false

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

Open

  def accept_attachment?(attachment)
    @excluded ||= Setting.mail_handler_excluded_filenames.to_s.split(',').map(&:strip).reject(&:blank?)
    @excluded.each do |pattern|
      if Setting.mail_handler_enable_regex_excluded_filenames?
        regexp = %r{\A#{pattern}\z}i

• Read up
  Read up

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

Open

  def receive_news_reply(news_id)
    news = News.find_by_id(news_id)
    if news.nil?
      raise MissingContainer, "reply to nonexistant news [#{news_id}]"
    end

• Read up
  Read up