Enforce new RuboCop cops for all Ruby files

Gemfile

@@ -1,7 +1,7 @@
 source "https://rubygems.org"
 git_source(:github) { |repo| "https://github.com/#{repo}.git" }
 
-ruby '3.1.4'
+ruby "3.1.4"
 
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
 gem "rails", "~> 7.0.4.3"
@@ -32,113 +32,113 @@ gem "jbuilder"
 # gem 'image_processing', '~> 1.2'
 
 # Reduces boot times through caching; required in config/boot.rb
+gem "active_model_serializers"
 gem "bootsnap", ">= 1.4.2", require: false
 gem "rack"
-gem "active_model_serializers"
 # Use CoffeeScript for .coffee assets and views
 gem "coffee-rails", "~> 5.0.0"
 
 # Use Redis adapter to run Action Cable in production
 # gem 'redis', '~> 3.0'
-gem "shrine"
 gem "fastimage"
-gem "streamio-ffmpeg"
-gem "pdf-reader"
-gem "mini_magick"
 gem "image_processing"
+gem "mini_magick"
+gem "pdf-reader"
+gem "shrine"
+gem "streamio-ffmpeg"
 # Use ActiveModel has_secure_password
 # gem 'bcrypt', '~> 3.1.7'
 gem "filesize"
 # Use Capistrano for deployment
 # gem 'capistrano-rails', group: :development
-gem "rgl"
-gem "responders"
-gem "pg"
-gem "devise"
-gem "erubis"
-gem "cancancan"
-gem "jquery-rails"
-gem "jquery-ui-rails"
-gem "js-routes", '1.4.9'
+gem "activerecord-import",
+    git: "https://github.com/zdennis/activerecord-import.git",
+    branch: "master"
+gem "acts_as_list"
+gem "acts_as_tree"
+gem "acts_as_votable"
+gem "barby"
 gem "bootstrap", "~>5"
 gem "bootstrap_form"
+gem "cancancan"
+gem "clipboard-rails"
+gem "commontator"
+gem "coveralls", require: false
+gem "devise"
 gem "devise-bootstrap-views"
+gem "erubis"
+gem "exception_handler", "~> 0.8.0.0"
+gem "faraday", "~> 1.8"
 gem "fuzzy-string-match"
-gem "coveralls", require: false
+gem "globalize"
+gem "globalize-accessors"
+gem "jquery-rails"
+gem "jquery-ui-rails"
+gem "js-routes", "1.4.9"
 gem "kaminari"
-gem "acts_as_list"
-gem "acts_as_tree"
-gem "activerecord-import",
-  git: "https://github.com/zdennis/activerecord-import.git",
-  branch: "master"
-gem "thredded"
-gem "kramdown-parser-gfm"
-gem "thredded-markdown_katex",
-  git: "https://github.com/thredded/thredded-markdown_katex.git",
-  branch: "main"
-gem "rails-i18n"
 gem "kaminari-i18n"
-gem "trix-rails", require: "trix"
-gem "sunspot_rails",
-  github: 'sunspot/sunspot',
-  glob: 'sunspot_rails/*.gemspec'
-gem "sunspot_solr"
+gem "kramdown-parser-gfm"
+gem "net-smtp"
+gem "pg"
+gem "premailer-rails"
 gem "progress_bar"
-gem "barby"
+gem "rails-i18n"
+gem "responders"
+gem "rgl"
 gem "rqrcode"
+gem "rubyzip", "~> 2.3.0"
 gem "sidekiq"
 gem "sidekiq-cron", "~> 1.1"
-gem "faraday", "~> 1.8"
-gem "globalize"
-gem "globalize-accessors"
-gem "commontator"
-gem "acts_as_votable"
 gem "sprockets-rails",
-  git: "https://github.com/rails/sprockets-rails",
-  branch: "master"
-gem "premailer-rails"
-gem "clipboard-rails"
-gem "rubyzip", "~> 2.3.0"
-gem "exception_handler", "~> 0.8.0.0"
-gem 'webpacker', '~> 5.x'
-gem 'net-smtp'
+    git: "https://github.com/rails/sprockets-rails",
+    branch: "master"
+gem "sunspot_rails",
+    github: "sunspot/sunspot",
+    glob: "sunspot_rails/*.gemspec"
+gem "sunspot_solr"
+gem "thredded"
+gem "thredded-markdown_katex",
+    git: "https://github.com/thredded/thredded-markdown_katex.git",
+    branch: "main"
+gem "trix-rails", require: "trix"
+gem "webpacker", "~> 5.x"
 
 group :development, :docker_development, :test do
   # Call 'byebug' anywhere in the code to stop execution and get a debugger console
   gem "byebug", platforms: [:mri, :mingw, :x64_mingw]
-  gem "rspec-rails"
   gem "factory_bot_rails"
+  gem "rspec-rails"
 end
 
 group :development, :docker_development do
   # Access an interactive console on exception pages or by calling 'console' anywhere in the code.
-  gem "web-console", ">= 3.3.0"
   gem "listen", ">= 3.0.5", "< 3.2"
   gem "rails-erd"
+  gem "web-console", ">= 3.3.0"
   # Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
+  gem "marcel"
+  gem "pgreset"
+  gem "rubocop", "~> 1.57", require: false
+  gem "rubocop-performance", "~> 1.16", require: false
+  gem "rubocop-rails", "~> 2.22", ">= 2.22.1", require: false
   gem "spring"
   gem "spring-watcher-listen", "~> 2.0.0"
-  gem "rubocop", "~> 1.57", require: false
-  gem 'rubocop-performance', '~> 1.16', require: false
-  gem 'rubocop-rails', '~> 2.22', '>= 2.22.1', require: false
-  gem "pgreset"
-  gem "marcel"
   #  gem 'bullet'
 end
 
 group :test do
   # Adds support for Capybara system testing and selenium driver
   gem "selenium-webdriver"
   # Easy installation and use of web drivers to run system tests with browsers
-  gem 'webdrivers'
-  gem 'faker'
-  gem 'database_cleaner'
-  gem 'launchy'
-  gem 'simplecov', require: false
+  gem "database_cleaner"
+  gem "faker"
+  gem "launchy"
+  gem "simplecov", require: false
+  gem "webdrivers"
 end
 
-group :test, :development, :docker_development do
-  gem 'cypress-on-rails', '~> 1.0'
-  gem 'simplecov-cobertura'
+group :test, :development, :docker_development do # rubocop:todo Bundler/DuplicatedGroup
+  gem "cypress-on-rails", "~> 1.0"
+  gem "simplecov-cobertura"
 end
-gem 'prometheus_exporter'
+gem "prometheus_exporter"

Rakefile

@@ -1,6 +1,6 @@
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 
-require_relative 'config/application'
+require_relative "config/application"
 
 Rails.application.load_tasks

app/abilities/main_ability.rb

@@ -1,7 +1,7 @@
 class MainAbility
   include CanCan::Ability
 
-  def initialize(user)
+  def initialize(_user)
     can :start, :main
   end
 end

app/abilities/medium_ability.rb

@@ -9,7 +9,7 @@ def initialize(user)
 
     can [:show, :show_comments], Medium do |medium|
       medium.visible_for_user?(user) &&
-        !(medium.sort.in?(['Question', 'Remark']) && !user.can_edit?(medium))
+        !(medium.sort.in?(["Question", "Remark"]) && !user.can_edit?(medium))
     end
 
     can :inspect, Medium do |medium|
@@ -46,7 +46,7 @@ def initialize(user)
       !user.generic? && user.can_edit?(medium)
     end
 
-    can [:register_download], Medium do |medium|
+    can [:register_download], Medium do |_medium|
       !user.new_record?
     end
   end

app/abilities/profile_ability.rb

@@ -1,7 +1,7 @@
 class ProfileAbility
   include CanCan::Ability
 
-  def initialize(user)
+  def initialize(_user)
     clear_aliased_actions
 
     can [:edit, :update, :check_for_consent, :add_consent,

app/abilities/search_ability.rb

@@ -1,7 +1,7 @@
 class SearchAbility
   include CanCan::Ability
 
-  def initialize(user)
+  def initialize(_user)
     clear_aliased_actions
 
     can :index, :search

app/abilities/tutorial_ability.rb

@@ -9,11 +9,11 @@ def initialize(user)
       user.can_update_personell?(tutorial.lecture)
     end
 
-    can :overview, Tutorial do |tutorial, lecture|
+    can :overview, Tutorial do |_tutorial, lecture|
       user.editor_or_teacher_in?(lecture)
     end
 
-    can :index, Tutorial do |tutorial, lecture|
+    can :index, Tutorial do |_tutorial, lecture|
       user.in?(lecture.tutors) || user.editor_or_teacher_in?(lecture)
     end
 

app/controllers/administration_controller.rb

@@ -5,7 +5,7 @@ class AdministrationController < ApplicationController
   # tell cancancan there is no model for this controller, but authorize
   # nevertheless
   authorize_resource class: false
-  layout 'administration'
+  layout "administration"
 
   def current_ability
     @current_ability ||= AdministrationAbility.new(current_user)
@@ -26,6 +26,6 @@ def classification
   end
 
   def search
-    @tags = params[:sort] == 'tag'
+    @tags = params[:sort] == "tag"
   end
 end

app/controllers/announcements_controller.rb

@@ -1,6 +1,6 @@
 # AnnouncementsController
 class AnnouncementsController < ApplicationController
-  layout 'administration'
+  layout "administration"
   before_action :set_announcement, except: [:new, :create, :index]
   authorize_resource except: [:new, :create, :index]
 
@@ -17,7 +17,7 @@
   end
 
   def new
     @lecture = Lecture.find_by_id(params[:lecture])
     @announcement = Announcement.new(announcer: current_user, lecture: @lecture)
     authorize! :new, @announcement
   end
@@ -33,14 +33,14 @@
       # send notification email
       send_notification_email
       # redirection depending from where the announcement was created
       unless @announcement.lecture.present?
         redirect_to announcements_path
         return
       end
       redirect_to edit_lecture_path(@announcement.lecture)
       return
     end
-    @errors = @announcement.errors[:details].join(', ')
+    @errors = @announcement.errors[:details].join(", ")
   end
 
   def propagate
@@ -66,12 +66,12 @@
         User
       end
       notifications = []
-      users_to_notify.update_all(updated_at: Time.now)
+      users_to_notify.update_all(updated_at: Time.now) # rubocop:todo Rails/SkipsModelValidations
       users_to_notify.find_each do |u|
         notifications << Notification.new(recipient: u,
                                           notifiable_id: @announcement.id,
-                                          notifiable_type: 'Announcement',
-                                          action: 'create')
+                                          notifiable_type: "Announcement",
+                                          action: "create")
       end
       # use activerecord-import gem to use only one SQL instruction
       Notification.import notifications
@@ -86,19 +86,19 @@
       end
       I18n.available_locales.each do |l|
         local_recipients = recipients.where(locale: l)
-        if local_recipients.any?
-          NotificationMailer.with(recipients: local_recipients.pluck(:id),
-                                  locale: l,
-                                  announcement: @announcement)
-                            .announcement_email.deliver_later
-        end
+        next unless local_recipients.any?
+
+        NotificationMailer.with(recipients: local_recipients.pluck(:id),
+                                locale: l,
+                                announcement: @announcement)
+                          .announcement_email.deliver_later
       end
     end
 
     def set_announcement
       @announcement = Announcement.find_by_id(params[:id])
       return if @announcement.present?
 
-      redirect_to :root, alert: I18n.t('controllers.no_announcement')
+      redirect_to :root, alert: I18n.t("controllers.no_announcement")
     end
 end

app/controllers/answers_controller.rb

@@ -8,8 +8,8 @@
   end
 
   def new
     question = Question.find_by_id(params[:question_id])
-    @answer = Answer.new(value: true, question: question)
+    @answer = Answer.new(value: true, question:)
     authorize! :new, @answer
     I18n.locale = question&.locale_with_inheritance
   end
@@ -36,16 +36,16 @@
 
   def update_answer_box
     @answer_id = params[:answer_id].to_i
-    @value = params[:value] == 'true'
+    @value = params[:value] == "true"
   end
 
   private
 
     def set_answer
       @answer = Answer.find_by_id(params[:id])
       return if @answer.present?
 
-      redirect_to root_path, alert: I18n.t('controllers.no_answer')
+      redirect_to root_path, alert: I18n.t("controllers.no_answer")
     end
 
     def answer_params