mashirozx/mastodon

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

Open

  def unmerge_from_home(from_account, into_account)
    timeline_key        = key(:home, into_account.id)
    timeline_status_ids = redis.zrange(timeline_key, 0, -1)

    from_account.statuses.select('id, reblog_of_id').where(id: timeline_status_ids).reorder(nil).find_each do |status|

• Read up
  Read up

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

Open

      if ActiveRecord::Migrator.current_version < 20220310060641 # rubocop:disable Style/NumericLiterals
        ActiveRecord::Base.connection.add_index :users, ['reset_password_token'], name: 'index_users_on_reset_password_token', unique: true
      else
        ActiveRecord::Base.connection.add_index :users, ['reset_password_token'], name: 'index_users_on_reset_password_token', unique: true, where: 'reset_password_token IS NOT NULL', opclass: :text_pattern_ops
      end

• Read up
  Read up

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

Open

  def unmerge_from_list(from_account, list)
    timeline_key        = key(:list, list.id)
    timeline_status_ids = redis.zrange(timeline_key, 0, -1)

    from_account.statuses.select('id, reblog_of_id').where(id: timeline_status_ids).reorder(nil).find_each do |status|

• Read up
  Read up

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

Open

  def batch
    authorize :preview_card_provider, :review?

    @form = Trends::PreviewCardProviderBatch.new(trends_preview_card_provider_batch_params.merge(current_account: current_account, action: action_from_button))
    @form.save

• 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

        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_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

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_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

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 [sourceNumber, DECIMAL_UNITS.ONE, 0];

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 @account if @account&.local? || gone_from_origin? || !webfinger_update_due?

Avoid too many return statements within this method.

Open

      return !!should_filter

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