<commit_before>class FollowService < BaseService
  # Follow a remote user, notify remote user about the follow
  # @param [Account] source_account From which to follow
  # @param [String] uri User URI to follow in the form of username@domain
  def call(source_account, uri)
    target_account = follow_remote_account_service.call(uri)

    return nil if target_account.nil? || target_account.id == source_account.id

    follow = source_account.follow!(target_account)

    if target_account.local?
      NotificationMailer.follow(target_account, source_account).deliver_later unless target_account.blocking?(source_account)
    else
      subscribe_service.call(target_account)
      NotificationWorker.perform_async(follow.stream_entry.id, target_account.id)
    end

    merge_into_timeline(target_account, source_account)
    HubPingWorker.perform_async(source_account.id)
    follow
  end

  private

  def merge_into_timeline(from_account, into_account)
    timeline_key = FeedManager.instance.key(:home, into_account.id)

    from_account.statuses.find_each do |status|
      redis.zadd(timeline_key, status.id, status.id)
    end

    FeedManager.instance.trim(:home, into_account.id)
    FeedManager.instance.broadcast(into_account.id, type: 'merge')
  end

  def redis
    $redis
  end

  def follow_remote_account_service
    @follow_remote_account_service ||= FollowRemoteAccountService.new
  end

  def subscribe_service
    @subscribe_service ||= SubscribeService.new
  end
end
<commit_msg>Fix #83 - if user cannot be found (or is self) throw error, don't return empty 200 This prevents the undefined profile from being opened in the first place on such an error<commit_after>class FollowService < BaseService
  # Follow a remote user, notify remote user about the follow
  # @param [Account] source_account From which to follow
  # @param [String] uri User URI to follow in the form of username@domain
  def call(source_account, uri)
    target_account = follow_remote_account_service.call(uri)

    raise ActiveRecord::RecordNotFound if target_account.nil? || target_account.id == source_account.id

    follow = source_account.follow!(target_account)

    if target_account.local?
      NotificationMailer.follow(target_account, source_account).deliver_later unless target_account.blocking?(source_account)
    else
      subscribe_service.call(target_account)
      NotificationWorker.perform_async(follow.stream_entry.id, target_account.id)
    end

    merge_into_timeline(target_account, source_account)
    HubPingWorker.perform_async(source_account.id)
    follow
  end

  private

  def merge_into_timeline(from_account, into_account)
    timeline_key = FeedManager.instance.key(:home, into_account.id)

    from_account.statuses.find_each do |status|
      redis.zadd(timeline_key, status.id, status.id)
    end

    FeedManager.instance.trim(:home, into_account.id)
    FeedManager.instance.broadcast(into_account.id, type: 'merge')
  end

  def redis
    $redis
  end

  def follow_remote_account_service
    @follow_remote_account_service ||= FollowRemoteAccountService.new
  end

  def subscribe_service
    @subscribe_service ||= SubscribeService.new
  end
end

{'fraction_non_alphanumeric': 0.07270448135380758, 'fraction_numerical': 0.0015669069257286117, 'mean_word_length': 4.560975609756097, 'pattern_counts': {'":': 0, '<': 5, '<?xml version=': 0, '>': 3, 'https://': 0, 'lorem ipsum': 0, 'www.': 0, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 0, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '3926066', 'n_tokens_mistral': 973, 'n_tokens_neox': 929, 'n_words': 224}