Enable the Layout/EmptyLinesAroundClassBody cop

.rubocop.yml

@@ -55,7 +55,7 @@ Layout/EmptyLinesAroundBlockBody:
   Enabled: true
 
 Layout/EmptyLinesAroundClassBody:
-  Enabled: false
+  Enabled: true
 
 Layout/EmptyLinesAroundExceptionHandlingKeywords:
   Enabled: false

app/controllers/api/v1/procedures_controller.rb

@@ -13,5 +13,4 @@ class API::V1::ProceduresController < APIController
     Rails.logger.error(e.message)
     render json: {}, status: 404
   end
-
 end

app/controllers/backoffice/avis_controller.rb

@@ -1,5 +1,4 @@
 class Backoffice::AvisController < ApplicationController
-
   before_action :authenticate_gestionnaire!, except: [:sign_up, :create_gestionnaire]
   before_action :redirect_if_no_sign_up_needed, only: [:sign_up]
   before_action :check_avis_exists_and_email_belongs_to_avis, only: [:sign_up, :create_gestionnaire]

app/controllers/backoffice/dossiers/procedure_controller.rb

@@ -21,5 +21,4 @@ class Backoffice::Dossiers::ProcedureController < Backoffice::DossiersListContro
   def retrieve_procedure
     current_gestionnaire.procedures.find params[:id]
   end
-
 end

app/decorators/dossiers_decorator.rb

@@ -1,4 +1,3 @@
 class DossiersDecorator < Draper::CollectionDecorator
   delegate :current_page, :per_page, :offset, :total_entries, :total_pages
-
 end

app/decorators/etablissement_decorator.rb

@@ -1,4 +1,3 @@
 class EtablissementDecorator < Draper::Decorator
   delegate_all
-
 end

app/decorators/procedure_decorator.rb

@@ -1,5 +1,4 @@
 class ProcedureDecorator < Draper::Decorator
-
   delegate_all
 
   def lien

app/decorators/procedures_decorator.rb

@@ -1,4 +1,3 @@
 class ProceduresDecorator < Draper::CollectionDecorator
   delegate :current_page, :per_page, :offset, :total_entries, :total_pages
-
 end

app/decorators/type_de_champ_decorator.rb

@@ -1,5 +1,4 @@
 class TypeDeChampDecorator < Draper::Decorator
-
   delegate_all
 
   def button_up params

app/decorators/type_de_champ_private_decorator.rb

@@ -1,3 +1,2 @@
 class TypeDeChampPrivateDecorator < TypeDeChampDecorator
-
 end

app/facades/dossier_facades.rb

@@ -1,5 +1,4 @@
 class DossierFacades
-
   #TODO rechercher en fonction de la personne/email
   def initialize(dossier_id, email, champ_id = nil)
     @dossier = Dossier.find(dossier_id)

app/facades/dossiers_list_facades.rb

@@ -80,5 +80,4 @@ class DossiersListFacades
   def base_url liste
     @procedure.nil? ? backoffice_dossiers_path(liste: liste) : backoffice_dossiers_procedure_path(id: @procedure.id, liste: liste)
   end
-
 end

app/facades/invite_dossier_facades.rb

@@ -1,5 +1,4 @@
 class InviteDossierFacades < DossierFacades
-
   #TODO rechercher en fonction de la personne/email
   def initialize id, email
     @dossier = Invite.where(email: email, id: id).first!.dossier

app/lib/carto/geo_api/driver.rb

@@ -1,7 +1,6 @@
 module Carto
   module GeoAPI
     class Driver
-
       def self.regions
         call regions_url
       end
@@ -29,7 +28,6 @@ module Carto
       rescue RestClient::ServiceUnavailable
         nil
       end
-
     end
   end
 end

app/mailers/avis_mailer.rb

@@ -1,9 +1,7 @@
 class AvisMailer < ApplicationMailer
-
   def avis_invitation(avis)
     @avis = avis
     email = @avis.gestionnaire.try(:email) || @avis.email
     mail(to: email, subject: "Donnez votre avis sur le dossier nº #{@avis.dossier.id} (#{@avis.dossier.procedure.libelle})")
   end
-
 end

app/mailers/gestionnaire_mailer.rb

@@ -1,5 +1,4 @@
 class GestionnaireMailer < ApplicationMailer
-
   def new_gestionnaire email, password
     send_mail email, password, "Vous avez été nommé accompagnateur sur la plateforme TPS"
   end

app/mailers/invite_mailer.rb

@@ -1,5 +1,4 @@
 class InviteMailer < ApplicationMailer
-
   def invite_user invite
     vars_mailer invite
 

app/models/champ_private.rb

@@ -1,3 +1,2 @@
 class ChampPrivate < Champ
-
 end

app/models/champ_public.rb

@@ -1,3 +1,2 @@
 class ChampPublic < Champ
-
 end

app/models/invite.rb

@@ -1,5 +1,4 @@
 class Invite < ActiveRecord::Base
-
   belongs_to :dossier
   belongs_to :user
 

app/models/invite_gestionnaire.rb

@@ -1,3 +1,2 @@
 class InviteGestionnaire < Invite
-
 end

app/models/invite_user.rb

@@ -1,3 +1,2 @@
 class InviteUser < Invite
-
 end

app/models/type_de_champ_private.rb

@@ -1,3 +1,2 @@
 class TypeDeChampPrivate < TypeDeChamp
-
 end

app/models/type_de_champ_public.rb

@@ -1,3 +1,2 @@
 class TypeDeChampPublic < TypeDeChamp
-
 end

app/services/browser_service.rb

@@ -1,5 +1,4 @@
 class BrowserService
-
   def self.get_browser request
     BROWSER.value = Browser.new(request.user_agent)
   end
@@ -15,5 +14,4 @@ class BrowserService
 
     true
   end
-
 end

app/services/dossier_service.rb

@@ -1,5 +1,4 @@
 class DossierService
-
   def initialize dossier, siret, france_connect_information
     @dossier = dossier
     @siret = siret

app/services/dossiers_list_gestionnaire_service.rb

@@ -186,5 +186,4 @@ class DossiersListGestionnaireService
   def current_preference_smart_listing_page
     @current_devise_profil.preference_smart_listing_page
   end
-
 end

app/services/france_connect_salt_service.rb

@@ -1,5 +1,4 @@
 class FranceConnectSaltService
-
   attr_reader :model
 
   def initialize france_connect_information

app/services/notification_service.rb

@@ -1,5 +1,4 @@
 class NotificationService
-
   def initialize type_notif, dossier_id, attribut_change=''
     @type_notif = type_notif
     @dossier_id = dossier_id

app/services/render_partial_service.rb

@@ -1,5 +1,4 @@
 class RenderPartialService
-
   attr_accessor :controller, :method
 
   def initialize controller, method

app/services/sync_credentials_service.rb

@@ -1,5 +1,4 @@
 class SyncCredentialsService
-
   def initialize klass, email_was, email, encrypted_password
     @klass = klass
     @email_was = email_was

app/services/user_routes_authorization_service.rb

@@ -1,5 +1,4 @@
 class UserRoutesAuthorizationService
-
   def self.authorized_route? controller, dossier
     auth = controller.route_authorization
 

app/uploaders/procedure_logo_uploader.rb

@@ -1,7 +1,6 @@
 # encoding: utf-8
 
 class ProcedureLogoUploader < BaseUploader
-
   def root
     File.join(Rails.root, "public")
   end
@@ -47,5 +46,4 @@ class ProcedureLogoUploader < BaseUploader
   def generate_secure_token
     SecureRandom.uuid
   end
-
 end

app/validators/email_format_validator.rb

@@ -1,5 +1,4 @@
 class EmailFormatValidator < ActiveModel::Validator
-
   def email_regex
     /\A([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})\z/i
   end

app/validators/procedure_path_format_validator.rb

@@ -1,5 +1,4 @@
 class ProcedurePathFormatValidator < ActiveModel::Validator
-
   def path_regex
     /^[a-z0-9_]{3,30}$/
   end
@@ -8,5 +7,4 @@ class ProcedurePathFormatValidator < ActiveModel::Validator
     return false if record.path.blank?
     record.errors[:path] << "Path invalide" unless path_regex.match(record.path)
   end
-
 end

config/initializers/stringupcasepatch.rb

@@ -1,9 +1,7 @@
 require "unicode_utils/upcase"
 
 class String
-
   def upcase
     UnicodeUtils.upcase(self)
   end
-
 end

db/migrate/20151207140202_update_module_api_carto.rb

@@ -1,11 +1,8 @@
 class UpdateModuleAPICarto < ActiveRecord::Migration
-
   class Procedure < ActiveRecord::Base
-
   end
 
   class ModuleAPICarto < ActiveRecord::Base
-
   end
 
   def up

db/migrate/20160120094750_create_france_connect_information.rb

@@ -1,11 +1,8 @@
 class CreateFranceConnectInformation < ActiveRecord::Migration
-
   class User < ActiveRecord::Base
-
   end
 
   class FranceConnectInformation < ActiveRecord::Base
-
   end
 
   def up

db/migrate/20160317144949_piece_justificative_have_user.rb

@@ -8,7 +8,6 @@ class PieceJustificativeHaveUser < ActiveRecord::Migration
   end
 
   class User < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20160317153115_cerfa_have_user.rb

@@ -8,7 +8,6 @@ class CerfaHaveUser < ActiveRecord::Migration
   end
 
   class User < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20160524093540_set_original_filename_for_uploaders.rb

@@ -1,10 +1,8 @@
 class SetOriginalFilenameForUploaders < ActiveRecord::Migration
   class PieceJustificative < ActiveRecord::Base
-
   end
 
   class Cerfa < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20160609145737_delete_default_description_to_dossier.rb

@@ -1,18 +1,14 @@
 class DeleteDefaultDescriptionToDossier < ActiveRecord::Migration
   class Dossier < ActiveRecord::Base
-
   end
 
   class Champ < ActiveRecord::Base
-
   end
 
   class Procedure < ActiveRecord::Base
-
   end
 
   class TypeDeChamp < ActiveRecord::Base
-
   end
 
   def up

db/migrate/20160622081321_convert_all_datetime_to_date_on_database.rb

@@ -1,7 +1,5 @@
 class ConvertAllDatetimeToDateOnDatabase < ActiveRecord::Migration
-
   class TypeDeChamp < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20160803081304_fix_default_type_on_type_de_champ_table.rb

@@ -1,6 +1,5 @@
 class FixDefaultTypeOnTypeDeChampTable < ActiveRecord::Migration
   class TypeDeChamp < ActiveRecord::Base
-
   end
 
   def up

db/migrate/20160804130638_add_type_attr_in_champ_table.rb

@@ -1,5 +1,4 @@
 class AddTypeAttrInChampTable < ActiveRecord::Migration
-
   class TypeDeChamp < ActiveRecord::Base
     has_many :champs
   end

db/migrate/20160822142045_delete_value_of_filter_procedure.rb

@@ -1,6 +1,5 @@
 class DeleteValueOfFilterProcedure < ActiveRecord::Migration
   class  Gestionnaire < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20160901082824_initiated_all_received_mail_for_procedure.rb

@@ -4,7 +4,6 @@ class InitiatedAllReceivedMailForProcedure < ActiveRecord::Migration
   end
 
   class MailTemplate < ActiveRecord::Base
-
   end
 
   class ::MailReceived < MailTemplate

db/migrate/20160927154248_change_emptyto_nil_in_preference_list_dossier_table.rb

@@ -1,6 +1,5 @@
 class ChangeEmptytoNilInPreferenceListDossierTable < ActiveRecord::Migration
   class PreferenceListDossier < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20161004175442_fix_type_de_champ_header_section_with_mandatory_true.rb

@@ -1,6 +1,5 @@
 class FixTypeDeChampHeaderSectionWithMandatoryTrue < ActiveRecord::Migration
   class TypeDeChamp < ActiveRecord::Base
-
   end
 
   def change

db/migrate/20161007095443_reset_all_preference_list_dossier.rb

@@ -143,7 +143,6 @@ class ResetAllPreferenceListDossier < ActiveRecord::Migration
   class Procedure < ActiveRecord::Base
     has_many :assign_to, dependent: :destroy
     has_many :gestionnaires, through: :assign_to
-
   end
 
   def change

spec/mailers/previews/avis_mailer_preview.rb

@@ -1,8 +1,6 @@
 # Preview all emails at http://localhost:3000/rails/mailers/avis_mailer
 class AvisMailerPreview < ActionMailer::Preview
-
   def avis_invitation
     AvisMailer.avis_invitation(Avis.last)
   end
-
 end

spec/mailers/previews/notification_mailer_preview.rb

@@ -1,7 +1,5 @@
 class NotificationMailerPreview < ActionMailer::Preview
-
   def send_notification
     NotificationMailer.send_notification(Dossier.last, Dossier.last.procedure.initiated_mail_template)
   end
-
 end