Use delete_all instead of destroy_all to batch delete

This commit is contained in:
gregoirenovel 2017-08-29 12:20:28 +02:00
parent fa395191f3
commit 650a39b371
3 changed files with 6 additions and 6 deletions

View file

@ -5,7 +5,7 @@ class Admin::PrevisualisationsController < AdminController
@procedure
@dossier = Dossier.new(id: 0, procedure: @procedure)
PrevisualisationService.destroy_all_champs @dossier
PrevisualisationService.delete_all_champs @dossier
@dossier.build_default_champs
@champs = @dossier.ordered_champs

View file

@ -1,5 +1,5 @@
class PrevisualisationService
def self.destroy_all_champs dossier
Champ.where(dossier_id: dossier.id, type_de_champ_id: dossier.procedure.types_de_champ.ids).destroy_all
def self.delete_all_champs dossier
Champ.where(dossier_id: dossier.id, type_de_champ_id: dossier.procedure.types_de_champ.ids).delete_all
end
end

View file

@ -1,8 +1,8 @@
require 'spec_helper'
describe PrevisualisationService do
describe '.destroy_all_champs' do
subject { described_class.destroy_all_champs dossier }
describe '.delete_all_champs' do
subject { described_class.delete_all_champs dossier }
let(:procedure_1) { create :procedure, :with_type_de_champ }
let(:procedure_2) { create :procedure, :with_type_de_champ }
@ -14,7 +14,7 @@ describe PrevisualisationService do
it { expect(TypeDeChamp.all.size).to eq 2 }
it { expect(Champ.all.size).to eq 2 }
context 'when function destroy_all_champs is call' do
context 'when function delete_all_champs is call' do
let(:dossier) { dossier_1 }
before do