mashirozx/mastodon

Similar blocks of code found in 4 locations. Consider refactoring.

Open

  def batch
    authorize :preview_card, :review?

    @form = Trends::PreviewCardBatch.new(trends_preview_card_batch_params.merge(current_account: current_account, action: action_from_button))
    @form.save

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

      if ActiveRecord::Migrator.current_version < 20220307083603 # rubocop:disable Style/NumericLiterals
        ActiveRecord::Base.connection.add_index :conversations, ['uri'], name: 'index_conversations_on_uri', unique: true
      else
        ActiveRecord::Base.connection.add_index :conversations, ['uri'], name: 'index_conversations_on_uri', unique: true, where: 'uri IS NOT NULL', opclass: :text_pattern_ops
      end

• Read up
  Read up

Similar blocks of code found in 4 locations. Consider refactoring.

Open

      if ActiveRecord::Migrator.current_version < 20220310060706 # rubocop:disable Style/NumericLiterals
        ActiveRecord::Base.connection.add_index :statuses, ['uri'], name: 'index_statuses_on_uri', unique: true
      else
        ActiveRecord::Base.connection.add_index :statuses, ['uri'], name: 'index_statuses_on_uri', unique: true, where: 'uri IS NOT NULL', opclass: :text_pattern_ops
      end

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  def follows
    return @follows if defined?(@follows)

    scope = Follow.where(target_account: @account)
    scope = scope.where.not(account_id: current_account.excluded_from_timeline_account_ids) if user_signed_in?

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        scope.find_each do |target_account|
          begin
            UnfollowService.new.call(target_account, account)
          rescue => e
            progress.log pastel.red("Error processing #{target_account.id}: #{e}")

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        scope.find_each do |target_account|
          begin
            UnfollowService.new.call(account, target_account)
          rescue => e
            progress.log pastel.red("Error processing #{target_account.id}: #{e}")

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  def perform_data_query
    sql = <<-SQL.squish
      SELECT axis.*, (
        WITH new_users AS (
          SELECT users.id

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  def unpush_from_list(list, status, update: false)
    return false unless remove_from_feed(:list, list.id, status, aggregate_reblogs: true)

    redis.publish("timeline:list:#{list.id}", Oj.dump(event: :delete, payload: status.id.to_s)) unless update
    true

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  def perform_data_query
    sql = <<-SQL.squish
      SELECT axis.*, (
        WITH new_reports AS (
          SELECT reports.id

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  def perform_data_query
    sql = <<-SQL.squish
      SELECT axis.*, (
        WITH resolved_reports AS (
          SELECT reports.id

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

  def unpush_from_home(account, status, update: false)
    return false unless remove_from_feed(:home, account.id, status, aggregate_reblogs: true)

    redis.publish("timeline:#{account.id}", Oj.dump(event: :delete, payload: status.id.to_s)) unless update
    true

• Read up
  Read up

Avoid too many return statements within this function.

Open

  return [sourceNumber, DECIMAL_UNITS.ONE, 0];

Avoid too many return statements within this function.

Open

    return (
      <HotKeys handlers={handlers}>
        <div className={classNames('status__wrapper', `status__wrapper-${status.get('visibility')}`, { 'status__wrapper-reply': !!status.get('in_reply_to_id'), unread, focusable: !this.props.muted })} tabIndex={this.props.muted ? null : 0} data-featured={featured ? 'true' : null} aria-label={textForScreenReader(intl, status, rebloggedByText)} ref={this.handleRef}>
          {prepend}

Avoid too many return statements within this function.

Open

      return state;

Avoid too many return statements within this function.

Open

        return;

Avoid too many return statements within this method.

Open

    return unless supported_context?(@owner) && confirmed_owner?

Avoid too many return statements within this method.

Open

      return

Avoid too many return statements within this method.

Open

      return !!should_filter

Avoid too many return statements within this method.

Open

    return @account if @account&.local? || gone_from_origin? || !webfinger_update_due?

Similar blocks of code found in 6 locations. Consider refactoring.

Open

  static propTypes = {
    onRetry: PropTypes.func.isRequired,
    onClose: PropTypes.func.isRequired,
    intl: PropTypes.object.isRequired,
  }

• Read up
  Read up