Merge branch 'frozen-string-enable-app-models-even-more' into 'master'

Enable more frozen string in app/models/**/*.rb

See merge request gitlab-org/gitlab-ce!21019

app/models/concerns/access_requestable.rb

+# frozen_string_literal: true
+
 # == AccessRequestable concern
 #
 # Contains functionality related to objects that can receive request for access.

app/models/concerns/artifact_migratable.rb

+# frozen_string_literal: true
+
 # Adapter class to unify the interface between mounted uploaders and the
 # Ci::Artifact model
 # Meant to be prepended so the interface can stay the same

app/models/concerns/atomic_internal_id.rb

+# frozen_string_literal: true
+
 # Include atomic internal id generation scheme for a model
 #
 # This allows us to atomically generate internal ids that are

app/models/concerns/avatarable.rb

+# frozen_string_literal: true
+
 module Avatarable
   extend ActiveSupport::Concern
 
@@ -57,7 +59,8 @@ module Avatarable
       only_path = false
     end
 
-    url_base = ""
+    url_base = []
+
     if use_asset_host
       url_base << asset_host unless only_path
     else
@@ -65,7 +68,7 @@ module Avatarable
       url_base << gitlab_config.relative_url_root
     end
 
-    url_base + avatar.local_url + query_params
+    url_base.join + avatar.local_url + query_params
   end
 
   # Path that is persisted in the tracking Upload model. Used to fetch the

app/models/concerns/awardable.rb

+# frozen_string_literal: true
+
 module Awardable
   extend ActiveSupport::Concern
 

app/models/concerns/batch_destroy_dependent_associations.rb

+# frozen_string_literal: true
+
 # Provides a way to work around Rails issue where dependent objects are all
 # loaded into memory before destroyed: https://github.com/rails/rails/issues/22510.
 #

app/models/concerns/blob_like.rb

+# frozen_string_literal: true
+
 module BlobLike
   extend ActiveSupport::Concern
   include Linguist::BlobHelper

app/models/concerns/blocks_json_serialization.rb

+# frozen_string_literal: true
+
 # Overrides `as_json` and `to_json` to raise an exception when called in order
 # to prevent accidentally exposing attributes
 #

app/models/concerns/bulk_member_access_load.rb

+# frozen_string_literal: true
+
 # Returns and caches in thread max member access for a resource
 #
 module BulkMemberAccessLoad

app/models/concerns/cache_markdown_field.rb

+# frozen_string_literal: true
+
 # This module takes care of updating cache columns for Markdown-containing
 # fields. Use like this in the body of your class:
 #

app/models/concerns/cacheable_attributes.rb

+# frozen_string_literal: true
+
 module CacheableAttributes
   extend ActiveSupport::Concern
 

app/models/concerns/case_sensitivity.rb

+# frozen_string_literal: true
+
 # Concern for querying columns with specific case sensitivity handling.
 module CaseSensitivity
   extend ActiveSupport::Concern

app/models/concerns/chronic_duration_attribute.rb

+# frozen_string_literal: true
+
 module ChronicDurationAttribute
   extend ActiveSupport::Concern
 

app/models/concerns/created_at_filterable.rb

+# frozen_string_literal: true
+
 module CreatedAtFilterable
   extend ActiveSupport::Concern
 

app/models/concerns/deployment_platform.rb

+# frozen_string_literal: true
+
 module DeploymentPlatform
   # EE would override this and utilize environment argument
   # rubocop:disable Gitlab/ModuleWithInstanceVariables

app/models/concerns/diff_file.rb

+# frozen_string_literal: true
+
 module DiffFile
   extend ActiveSupport::Concern
 

app/models/concerns/discussion_on_diff.rb

+# frozen_string_literal: true
+
 # Contains functionality shared between `DiffDiscussion` and `LegacyDiffDiscussion`.
 module DiscussionOnDiff
   extend ActiveSupport::Concern

app/models/concerns/each_batch.rb

+# frozen_string_literal: true
+
 module EachBatch
   extend ActiveSupport::Concern
 

app/models/concerns/editable.rb

+# frozen_string_literal: true
+
 module Editable
   extend ActiveSupport::Concern
 

app/models/concerns/enum_with_nil.rb

+# frozen_string_literal: true
+
 module EnumWithNil
   extend ActiveSupport::Concern
 

app/models/concerns/expirable.rb

+# frozen_string_literal: true
+
 module Expirable
   extend ActiveSupport::Concern
 

app/models/concerns/fast_destroy_all.rb

+# frozen_string_literal: true
+
 ##
 # This module is for replacing `dependent: :destroy` and `before_destroy` hooks.
 #

app/models/concerns/faster_cache_keys.rb

+# frozen_string_literal: true
+
 module FasterCacheKeys
   # A faster version of Rails' "cache_key" method.
   #

app/models/concerns/feature_gate.rb

+# frozen_string_literal: true
+
 module FeatureGate
   def flipper_id
     return nil if new_record?

app/models/concerns/ghost_user.rb

+# frozen_string_literal: true
+
 module GhostUser
   extend ActiveSupport::Concern
 

app/models/concerns/group_descendant.rb

+# frozen_string_literal: true
+
 module GroupDescendant
   # Returns the hierarchy of a project or group in the from of a hash upto a
   # given top.

app/models/concerns/has_status.rb

+# frozen_string_literal: true
+
 module HasStatus
   extend ActiveSupport::Concern
 

app/models/concerns/has_variable.rb

+# frozen_string_literal: true
+
 module HasVariable
   extend ActiveSupport::Concern
 

app/models/concerns/ignorable_column.rb

+# frozen_string_literal: true
+
 # Module that can be included into a model to make it easier to ignore database
 # columns.
 #

app/models/concerns/iid_routes.rb

+# frozen_string_literal: true
+
 module IidRoutes
   ##
   # This automagically enforces all related routes to use `iid` instead of `id`

app/models/concerns/importable.rb

+# frozen_string_literal: true
+
 module Importable
   extend ActiveSupport::Concern
 

app/models/concerns/issuable.rb

+# frozen_string_literal: true
+
 # == Issuable concern
 #
 # Contains common functionality shared between Issues and MergeRequests

app/models/concerns/loaded_in_group_list.rb

+# frozen_string_literal: true
+
 module LoadedInGroupList
   extend ActiveSupport::Concern
 

app/models/concerns/manual_inverse_association.rb

+# frozen_string_literal: true
+
 module ManualInverseAssociation
   extend ActiveSupport::Concern
 

app/models/concerns/mentionable.rb

+# frozen_string_literal: true
+
 # == Mentionable concern
 #
 # Contains functionality related to objects that can mention Users, Issues, MergeRequests, Commits or Snippets by

app/models/concerns/mentionable/reference_regexes.rb

+# frozen_string_literal: true
+
 module Mentionable
   module ReferenceRegexes
     def self.reference_pattern(link_patterns, issue_pattern)

app/models/concerns/milestoneish.rb

+# frozen_string_literal: true
+
 module Milestoneish
   def closed_items_count(user)
     memoize_per_user(user, :closed_items_count) do

app/models/concerns/note_on_diff.rb

+# frozen_string_literal: true
+
 # Contains functionality shared between `DiffNote` and `LegacyDiffNote`.
 module NoteOnDiff
   extend ActiveSupport::Concern

app/models/concerns/noteable.rb

+# frozen_string_literal: true
+
 module Noteable
   # Names of all implementers of `Noteable` that support resolvable notes.
   RESOLVABLE_TYPES = %w(MergeRequest).freeze

app/models/concerns/participable.rb

+# frozen_string_literal: true
+
 # == Participable concern
 #
 # Contains functionality related to objects that can have participants, such as

app/models/concerns/presentable.rb

+# frozen_string_literal: true
+
 module Presentable
   extend ActiveSupport::Concern
 

app/models/concerns/project_features_compatibility.rb

+# frozen_string_literal: true
+
 # Makes api V4 compatible with old project features permissions methods
 #
 # After migrating issues_enabled merge_requests_enabled builds_enabled snippets_enabled and wiki_enabled

app/models/concerns/prometheus_adapter.rb

+# frozen_string_literal: true
+
 module PrometheusAdapter
   extend ActiveSupport::Concern
 

app/models/concerns/protected_branch_access.rb

+# frozen_string_literal: true
+
 module ProtectedBranchAccess
   extend ActiveSupport::Concern
 

app/models/concerns/protected_ref.rb

+# frozen_string_literal: true
+
 module ProtectedRef
   extend ActiveSupport::Concern
 

app/models/concerns/protected_ref_access.rb

+# frozen_string_literal: true
+
 module ProtectedRefAccess
   extend ActiveSupport::Concern
 

app/models/concerns/protected_tag_access.rb

+# frozen_string_literal: true
+
 module ProtectedTagAccess
   extend ActiveSupport::Concern
 

app/models/concerns/reactive_caching.rb

+# frozen_string_literal: true
+
 # The ReactiveCaching concern is used to fetch some data in the background and
 # store it in the Rails cache, keeping it up-to-date for as long as it is being
 # requested.  If the data hasn't been requested for +reactive_cache_lifetime+,

app/models/concerns/reactive_service.rb

+# frozen_string_literal: true
+
 module ReactiveService
   extend ActiveSupport::Concern
 

app/models/concerns/redis_cacheable.rb

+# frozen_string_literal: true
+
 module RedisCacheable
   extend ActiveSupport::Concern
   include Gitlab::Utils::StrongMemoize

app/models/concerns/referable.rb

+# frozen_string_literal: true
+
 # == Referable concern
 #
 # Contains functionality related to making a model referable in Markdown, such

app/models/concerns/relative_positioning.rb

+# frozen_string_literal: true
+
 module RelativePositioning
   extend ActiveSupport::Concern
 

app/models/concerns/resolvable_discussion.rb

+# frozen_string_literal: true
+
 module ResolvableDiscussion
   extend ActiveSupport::Concern
   include ::Gitlab::Utils::StrongMemoize

app/models/concerns/resolvable_note.rb

+# frozen_string_literal: true
+
 module ResolvableNote
   extend ActiveSupport::Concern
 

app/models/concerns/routable.rb

+# frozen_string_literal: true
+
 # Store object full path in separate table for easy lookup and uniq validation
 # Object must have name and path db fields and respond to parent and parent_changed? methods.
 module Routable

app/models/concerns/select_for_project_authorization.rb

+# frozen_string_literal: true
+
 module SelectForProjectAuthorization
   extend ActiveSupport::Concern
 

app/models/concerns/sha_attribute.rb

+# frozen_string_literal: true
+
 module ShaAttribute
   extend ActiveSupport::Concern
 

app/models/concerns/sortable.rb

+# frozen_string_literal: true
+
 # == Sortable concern
 #
 # Set default scope for ordering objects

app/models/concerns/spammable.rb

+# frozen_string_literal: true
+
 module Spammable
   extend ActiveSupport::Concern
 

app/models/concerns/storage/legacy_namespace.rb

+# frozen_string_literal: true
+
 module Storage
   module LegacyNamespace
     extend ActiveSupport::Concern

app/models/concerns/storage/legacy_project_wiki.rb

+# frozen_string_literal: true
+
 module Storage
   module LegacyProjectWiki
     extend ActiveSupport::Concern

app/models/concerns/storage/legacy_repository.rb

+# frozen_string_literal: true
+
 module Storage
   module LegacyRepository
     extend ActiveSupport::Concern

app/models/concerns/strip_attribute.rb

+# frozen_string_literal: true
+
 # == Strip Attribute module
 #
 # Contains functionality to clean attributes before validation

app/models/concerns/subscribable.rb

+# frozen_string_literal: true
+
 # == Subscribable concern
 #
 # Users can subscribe to these models.

app/models/concerns/taskable.rb

+# frozen_string_literal: true
+
 require 'task_list'
 require 'task_list/filter'
 

app/models/concerns/throttled_touch.rb

+# frozen_string_literal: true
+
 # ThrottledTouch can be used to throttle the number of updates triggered by
 # calling "touch" on an ActiveRecord model.
 module ThrottledTouch

app/models/concerns/time_trackable.rb

+# frozen_string_literal: true
+
 # == TimeTrackable concern
 #
 # Contains functionality related to objects that support time tracking.

app/models/concerns/token_authenticatable.rb

+# frozen_string_literal: true
+
 module TokenAuthenticatable
   extend ActiveSupport::Concern
 

app/models/concerns/triggerable_hooks.rb

+# frozen_string_literal: true
+
 module TriggerableHooks
   AVAILABLE_TRIGGERS = {
     repository_update_hooks:  :repository_update_events,

app/models/concerns/uniquify.rb

+# frozen_string_literal: true
+
 # Uniquify
 #
 # Return a version of the given 'base' string that is unique

app/models/concerns/updated_at_filterable.rb

+# frozen_string_literal: true
+
 module UpdatedAtFilterable
   extend ActiveSupport::Concern
 

app/models/concerns/valid_attribute.rb

+# frozen_string_literal: true
+
 module ValidAttribute
   extend ActiveSupport::Concern
 

app/models/concerns/with_uploads.rb

+# frozen_string_literal: true
+
 # Mounted uploaders are destroyed by carrierwave's after_commit
 # hook. This hook fetches upload location (local vs remote) from
 # Upload model. So it's neccessary to make sure that during that

app/models/conversational_development_index/card.rb

+# frozen_string_literal: true
+
 module ConversationalDevelopmentIndex
   class Card
     attr_accessor :metric, :title, :description, :feature, :blog, :docs

app/models/conversational_development_index/idea_to_production_step.rb

+# frozen_string_literal: true
+
 module ConversationalDevelopmentIndex
   class IdeaToProductionStep
     attr_accessor :metric, :title, :features

app/models/conversational_development_index/metric.rb

+# frozen_string_literal: true
+
 module ConversationalDevelopmentIndex
   class Metric < ActiveRecord::Base
     include Presentable

app/models/diff_viewer/added.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Added < Base
     include Simple

app/models/diff_viewer/base.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Base
     PARTIAL_PATH_PREFIX = 'projects/diffs/viewers'.freeze

app/models/diff_viewer/client_side.rb

+# frozen_string_literal: true
+
 module DiffViewer
   module ClientSide
     extend ActiveSupport::Concern

app/models/diff_viewer/deleted.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Deleted < Base
     include Simple

app/models/diff_viewer/image.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Image < Base
     include Rich

app/models/diff_viewer/mode_changed.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class ModeChanged < Base
     include Simple

app/models/diff_viewer/no_preview.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class NoPreview < Base
     include Simple

app/models/diff_viewer/not_diffable.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class NotDiffable < Base
     include Simple

app/models/diff_viewer/renamed.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Renamed < Base
     include Simple

app/models/diff_viewer/rich.rb

+# frozen_string_literal: true
+
 module DiffViewer
   module Rich
     extend ActiveSupport::Concern

app/models/diff_viewer/server_side.rb

+# frozen_string_literal: true
+
 module DiffViewer
   module ServerSide
     extend ActiveSupport::Concern

app/models/diff_viewer/simple.rb

+# frozen_string_literal: true
+
 module DiffViewer
   module Simple
     extend ActiveSupport::Concern

app/models/diff_viewer/static.rb

+# frozen_string_literal: true
+
 module DiffViewer
   module Static
     extend ActiveSupport::Concern

app/models/diff_viewer/text.rb

+# frozen_string_literal: true
+
 module DiffViewer
   class Text < Base
     include Simple

app/models/hooks/project_hook.rb

+# frozen_string_literal: true
+
 class ProjectHook < WebHook
   include TriggerableHooks
 

app/models/hooks/service_hook.rb

+# frozen_string_literal: true
+
 class ServiceHook < WebHook
   belongs_to :service
   validates :service, presence: true

app/models/hooks/system_hook.rb

+# frozen_string_literal: true
+
 class SystemHook < WebHook
   include TriggerableHooks
 

app/models/hooks/web_hook.rb

+# frozen_string_literal: true
+
 class WebHook < ActiveRecord::Base
   include Sortable
 

app/models/hooks/web_hook_log.rb

+# frozen_string_literal: true
+
 class WebHookLog < ActiveRecord::Base
   belongs_to :web_hook
 

app/models/issue/metrics.rb

+# frozen_string_literal: true
+
 class Issue::Metrics < ActiveRecord::Base
   belongs_to :issue
 

app/models/members/group_member.rb

+# frozen_string_literal: true
+
 class GroupMember < Member
   SOURCE_TYPE = 'Namespace'.freeze
 

app/models/members/project_member.rb

+# frozen_string_literal: true
+
 class ProjectMember < Member
   SOURCE_TYPE = 'Project'.freeze
 

app/models/merge_request/metrics.rb

+# frozen_string_literal: true
+
 class MergeRequest::Metrics < ActiveRecord::Base
   belongs_to :merge_request
   belongs_to :pipeline, class_name: 'Ci::Pipeline', foreign_key: :pipeline_id

app/models/network/commit.rb

+# frozen_string_literal: true
+
 module Network
   class Commit
     include ActionView::Helpers::TagHelper

app/models/network/graph.rb

+# frozen_string_literal: true
+
 module Network
   class Graph
     attr_reader :days, :commits, :map, :notes, :repo