Enable the Layout/SpaceAfterComma cop

.rubocop.yml

@@ -200,7 +200,7 @@ Layout/SpaceAfterColon:
   Enabled: true
 
 Layout/SpaceAfterComma:
-  Enabled: false
+  Enabled: true
 
 Layout/SpaceAfterMethodName:
   Enabled: true

app/models/champs/datetime_champ.rb

@@ -12,7 +12,7 @@ class Champs::DatetimeChamp < Champ
       self.value =
         begin
           hash_date = YAML.safe_load(value.gsub('=>', ': '))
-          year, month, day, hour, minute = hash_date.values_at(1,2,3,4,5)
+          year, month, day, hour, minute = hash_date.values_at(1, 2, 3, 4, 5)
           Time.zone.local(year, month, day, hour, minute).strftime("%d/%m/%Y %H:%M")
         rescue
           nil

app/models/etablissement.rb

@@ -91,6 +91,6 @@ class Etablissement < ApplicationRecord
   def message_for_signature
     JSON.pretty_generate(as_json(include: {
       exercices: { only: [:ca, :date_fin_exercice, :date_fin_exercice_timestamp] }
-    }).delete_if { |k,v| v.blank? })
+    }).delete_if { |k, v| v.blank? })
   end
 end

app/services/render_partial_service.rb

@@ -17,7 +17,7 @@ class RenderPartialService
   def self.left_panel_exist?(left_panel_url)
     file = left_panel_url.split('/').last
 
-    File.exist?(Rails.root.join('app','views', 'layouts', 'left_panels', "_#{file}.html.haml"))
+    File.exist?(Rails.root.join('app', 'views', 'layouts', 'left_panels', "_#{file}.html.haml"))
   end
 
   private

lib/tasks/2017_10_30_copy_commentaire_piece_justificative_to_file.rake

@@ -22,7 +22,7 @@ namespace :'2017_10_30_copy_commentaire_piece_justificative_to_file' do
   end
 
   def sanitize_name(name) # from https://github.com/carrierwaveuploader/carrierwave/blob/master/lib/carrierwave/sanitized_file.rb#L323
-    name = name.gsub(/[^[:word:]\.\-\+]/,"_")
+    name = name.gsub(/[^[:word:]\.\-\+]/, "_")
     name = "_#{name}" if name.match?(/\A\.+\z/)
     name = "unnamed" if name.empty?
     return name.mb_chars.to_s

lib/tasks/2018_01_18_clean_datetime_in_champs.rake

@@ -19,11 +19,11 @@ namespace :'2018_01_18_clean_datetime_in_champs' do
 
     # Match "ddmmYYYY HH:MM" => "YYYY-mm-dd HH:MM"
     datetime_champs.select { |c| /^\d{8}\s\d{2}:\d{2}$/ =~ c.value }.each do |c|
-      day = c.value[0,2]
+      day = c.value[0, 2]
-      month = c.value[2,2]
+      month = c.value[2, 2]
-      year = c.value[4,4]
+      year = c.value[4, 4]
-      hours = c.value[9,2]
+      hours = c.value[9, 2]
-      minutes = c.value[12,2]
+      minutes = c.value[12, 2]
       formated_date = "#{year}-#{month}-#{day} #{hours}:#{minutes}"
       rake_puts "cleaning #{c.value} => #{formated_date}"
       c.update_columns(value: formated_date)

lib/tasks/admin.rake

@@ -2,7 +2,7 @@ namespace :admin do
   task :create_admin, [:email] => :environment do |t, args|
     email = args[:email]
     puts "Creating Administration for #{email}"
-    a = Administration.new(email: email, password: Devise.friendly_token[0,20])
+    a = Administration.new(email: email, password: Devise.friendly_token[0, 20])
     if a.save
       puts "#{a.email} created"
     else

spec/models/procedure_presentation_spec.rb

@@ -11,7 +11,7 @@ describe ProcedurePresentation do
         { "label" => "test1", "table" => "user", "column" => "email" },
         { "label" => "test2", "table" => "type_de_champ", "column" => first_type_de_champ_id }
       ],
-      sort: { "table" => "user","column" => "email","order" => "asc" },
+      sort: { "table" => "user", "column" => "email", "order" => "asc" },
       filters: { "a-suivre" => [], "suivis" => [{ "label" => "label1", "table" => "self", "column" => "created_at" }] }
     ).id
   }
@@ -22,7 +22,7 @@ describe ProcedurePresentation do
   end
 
   describe "#sort" do
-    it { expect(procedure_presentation.sort).to eq({ "table" => "user","column" => "email","order" => "asc" }) }
+    it { expect(procedure_presentation.sort).to eq({ "table" => "user", "column" => "email", "order" => "asc" }) }
   end
 
   describe "#filters" do
@@ -78,10 +78,10 @@ describe ProcedurePresentation do
       }
 
       before do
-        procedure.types_de_champ[2].update_attribute(:type_champ,TypeDeChamp.type_champs.fetch(:header_section))
+        procedure.types_de_champ[2].update_attribute(:type_champ, TypeDeChamp.type_champs.fetch(:header_section))
-        procedure.types_de_champ[3].update_attribute(:type_champ,TypeDeChamp.type_champs.fetch(:explication))
+        procedure.types_de_champ[3].update_attribute(:type_champ, TypeDeChamp.type_champs.fetch(:explication))
-        procedure.types_de_champ_private[2].update_attribute(:type_champ,TypeDeChamp.type_champs.fetch(:header_section))
+        procedure.types_de_champ_private[2].update_attribute(:type_champ, TypeDeChamp.type_champs.fetch(:header_section))
-        procedure.types_de_champ_private[3].update_attribute(:type_champ,TypeDeChamp.type_champs.fetch(:explication))
+        procedure.types_de_champ_private[3].update_attribute(:type_champ, TypeDeChamp.type_champs.fetch(:explication))
       end
 
       subject { create(:procedure_presentation, assign_to: create(:assign_to, procedure: procedure)) }