Apparently, returning false in before_destroy does not work anymore i…

…n rails 5.

app/models/catalogue.rb

@@ -278,7 +278,7 @@ def check_dependencies
         #{self.referring_catalogues.count} catalogues and 
         #{self.referring_people.count} people
         #{self.referring_holdings.count} holdings}
-      return false
+      throw :abort
     end
   end
 

app/models/institution.rb

@@ -249,7 +249,7 @@ def check_dependencies
         #{self.referring_institutions.count} institutions, 
         #{self.referring_catalogues.count} catalogues and 
         #{self.referring_people.count} people}
-      return false
+      throw :abort
     end
   end
 

app/models/liturgical_feast.rb

@@ -70,7 +70,7 @@ def reindex
   def check_dependencies
     if (self.referring_sources.count > 0)
       errors.add :base, "The liturgical fease could not be deleted because it is used"
-      return false
+      throw :abort
     end
   end
 

app/models/person.rb

@@ -257,7 +257,7 @@ def check_dependencies
         #{self.referring_catalogues.count} catalogues and 
         #{self.referring_people.count} people
         #{self.referring_holdings.count} holdings}
-      return false
+      throw :abort
     end
   end
 

app/models/place.rb

@@ -86,7 +86,7 @@ def check_dependencies
         #{self.referring_catalogues.count} catalogues and 
         #{self.referring_people.count} people
         #{self.referring_holdings.count} holdings}
-      return false
+      throw :abort
     end
   end
 

app/models/source.rb

@@ -302,19 +302,19 @@ def reindex
   def check_dependencies
     if (self.child_sources.count > 0)
       errors.add :base, "The source could not be deleted because it has #{self.child_sources.count} child source(s)"
-      return false
+      throw :abort
     end
     if (self.digital_objects.count > 0)
       errors.add :base, "The source could not be deleted because it has digital objects attached"
-      return false
+      throw :abort
     end
     if (self.sources.count > 0)
       errors.add :base, "The source could not be deleted because it refers to #{self.sources.count} source(s)"
-      return false
+      throw :abort
     end
     if (self.referring_sources.count > 0)
       errors.add :base, "The source could not be deleted because it has #{self.referring_sources.count} subsequent entry(s)"
-      return false
+      throw :abort
     end
   end
 

app/models/standard_term.rb

@@ -76,7 +76,7 @@ def check_dependencies
         #{self.referring_sources.count} sources,
         #{self.referring_institutions.count} institutions and 
         #{self.referring_catalogues.count} catalogues}
-      return false
+      throw :abort
     end
   end
 

app/models/standard_title.rb

@@ -92,7 +92,7 @@ def reindex
   def check_dependencies
     if (self.referring_sources.count > 0)
       errors.add :base, "The standard title could not be deleted because it is used"
-      return false
+      throw :abort
     end
   end
 

app/models/work.rb

@@ -142,7 +142,7 @@ def set_object_fields
   end
 
   def check_dependencies
-    return false if self.referring_sources.count > 0
+    throw :abort if self.referring_sources.count > 0
   end
 
   def self.get_viaf(str)