prevent_destroy_if_any 0.1 → 0.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: b885f6dcbeaf3bcdd279b1118a1e5e9708acc89b
+  data.tar.gz: b9f2cb72b3c6d949c23529468bd4d9a1873d71f8
+SHA512:
+  metadata.gz: a64e8452ad65dd5ae1d2f4ca1853b5bfd6026df8c27304bfc50f85f43de5b03965a03d3e3d591f6c2ee75f228153f25241ac10deba14a5a235468ecc45502f1c
+  data.tar.gz: 00848b72ff168bd3c1ab551d35faaa700fe9668de6d841a1177afd2126eaffa5aed804d0e2bd91b3f88c764c02adb5b81a52ae73d8f257e8f1e404cc469370be

data/CHANGELOG.md

@@ -0,0 +1,10 @@
+# Change Log
+All notable changes to this project will be documented in this file.
+
+## 0.2 - 2014-11-04
+### Added
+- Ability to customize the error message
+- Created the English, Spanish and French default error messages
+
+## 0.1 - 2011-08-07
+- Initial release

data/README.md

@@ -1,46 +1,56 @@
-prevent_destroy_if_any
-===
+# prevent_destroy_if_any
 
 This plugin adds ActiveRecord models a way to prevent destroy if specified `has_many`, `has_one` and/or `belongs_to` associations are present. This is achieved by adding a `before_destroy` callback that aborts the destroy and adds a base error on the instance when detecting associations.
 
 It's inspired from [this stackoverflow answer](http://stackoverflow.com/questions/4054112/how-do-i-prevent-deletion-of-parent-if-it-has-child-records/4054170#4054170) and was created to factorize this solution when you need it for various models in an application.
 
-It has been tested only with ActiveRecord 3.
+It has been tested with ActiveRecord 3 and 4, with Ruby 1.9 and 2+.
 
-Installation
----
+## Installation
 
-`gem install prevent_destroy_if_any`
+Add this line to your application’s Gemfile:
 
-If you use Bundler, edit your `Gemfile` to add `gem 'prevent_destroy_if_any'`.
+`gem 'prevent_destroy_if_any'`
 
-
-Example usage
----
+## Example usage
 
 A Person model for which you want to prevent destroy when it has an associated user, projects or public profile. Destroy will work if it has associated logs.
 
-    class Person < ActiveRecord::Base
-      belongs_to :user
-      has_many   :projects
-      has_one    :public_profile
-      has_many   :logs
+```ruby
+class Person < ActiveRecord::Base
+  belongs_to :user
+  has_many   :projects
+  has_one    :public_profile
+  has_many   :logs
 
-      prevent_destroy_if_any :user, :projects, :public_profile
-    end
+  prevent_destroy_if_any :user, :projects, :public_profile
+end
+```
 
 Then in Rails you can write your PeopleController destroy action like this:
 
-    def destroy
-      person = Person.find params[:id]
-      if person.destroy
-        redirect_to people_url, :notice => "Person successfully deleted"
-      else
-        redirect_to person, :alert => person.errors.full_messages.join
-      end
-    end
+```ruby
+def destroy
+  person = Person.find params[:id]
+  if person.destroy
+    redirect_to people_url, notice: "Person successfully deleted"
+  else
+    redirect_to person, alert: person.errors.full_messages.join
+  end
+end
+```
+
+## Error message customization
+
+You can change the default base error message by setting the following key in your locale file(s):
+
+```yaml
+en:
+  prevent_destroy_if_any:
+    messages:
+      cannot_delete_parent_object: "Hey!!! You cannot delete %{parent_object} while %{associated_objects} exist"
+```
 
-License
----
+## License
 
-Copyright © 2011 Florent Guilleux, released under the MIT license.
+Copyright © 2013 Florent Guilleux, released under the MIT license.

data/config/locales/en.yml

@@ -0,0 +1,4 @@
+en:
+  prevent_destroy_if_any:
+    messages:
+      cannot_delete_parent_object: "Cannot delete %{parent_object} while %{associated_objects} exist"

data/config/locales/es.yml

@@ -0,0 +1,4 @@
+es:
+  prevent_destroy_if_any:
+    messages:
+      cannot_delete_parent_object: "No se puede eliminar %{parent_object} mientras %{associated_objects} exista"

data/config/locales/fr.yml

@@ -0,0 +1,4 @@
+fr:
+  prevent_destroy_if_any:
+    messages:
+      cannot_delete_parent_object: "Suppression de %{parent_object} impossible car il y a %{associated_objects}"

data/lib/prevent_destroy_if_any.rb

@@ -1,24 +1,31 @@
-require "prevent_destroy_if_any/version"
+require 'prevent_destroy_if_any/version'
+I18n.load_path += Dir[File.expand_path(File.dirname(__FILE__)) + "/../config/locales/*.yml"]
 
 class ActiveRecord::Base
   def self.prevent_destroy_if_any(*association_names)
     before_destroy do |model|
-      associations_present = []
-
+      available_associations = []
       association_names.each do |association_name|
         association = model.send association_name
-        if association.class == Array
-          associations_present << association_name if association.any?
-        else
-          associations_present << association_name if association
+        unless association.blank?
+          available_associations << if association.is_a?(ActiveRecord::Associations::CollectionProxy)
+            association.first.class.model_name.human(:count => 2)
+          else
+            association.class.model_name.human
+          end.downcase
         end
       end
-
-      if associations_present.any?
-        errors.add :base, "Cannot delete #{model.class.model_name.human.downcase} while #{associations_present.join ', '} exist"
-        return false
+      if available_associations.any?
+        errors.add(
+          :base,
+          I18n.t(
+            'prevent_destroy_if_any.messages.cannot_delete_parent_object',
+            :parent_object => model.class.model_name.human.downcase,
+            :associated_objects => available_associations.join(', ')
+          )
+        )
+        false
       end
-
     end
   end
 end

data/lib/prevent_destroy_if_any/version.rb

@@ -1,3 +1,3 @@
 module PreventDestroyIfAny
-  VERSION = "0.1"
+  VERSION = "0.2"
 end

data/prevent_destroy_if_any.gemspec

@@ -19,9 +19,10 @@ Gem::Specification.new do |s|
   s.require_paths = ["lib"]
 
   s.add_dependency "activerecord", '>= 3.0.0'
+  s.add_dependency "i18n"
   s.add_development_dependency "bundler", '>= 1.0.0'
   s.add_development_dependency "rspec", '>= 0'
   s.add_development_dependency "database_cleaner", '>= 0'
-  s.add_development_dependency "sqlite3-ruby", '>= 0'
+  s.add_development_dependency "sqlite3", '>= 0'
   s.add_development_dependency "rake", '>= 0'
 end

data/spec/prevent_destroy_if_any_spec.rb

@@ -9,33 +9,35 @@ describe "prevent_destroy_if_any" do
     it "destroy preventing works for has_many association" do
       person.projects.create
       person.destroy
-      person.destroyed?.should be_false
+      expect(person.destroyed?).to be_falsey
     end
 
     it "destroy preventing works for has_one association" do
       person.create_public_profile
       person.destroy
-      person.destroyed?.should be_false
+      expect(person.destroyed?).to be_falsey
     end
 
     it "destroy preventing works for belongs_ association" do
       person.create_user
       person.destroy
-      person.destroyed?.should be_false
+      expect(person.destroyed?).to be_falsey
     end
 
     it "adds an error message" do
       person.projects.create
       person.create_public_profile
       person.destroy
-      person.errors[:base].first.should == "Cannot delete person while projects, public_profile exist"
+      expect(person.errors[:base].first).to eq I18n.t('prevent_destroy_if_any.messages.cannot_delete_parent_object',
+          :parent_object => 'person',
+          :associated_objects => 'project, public profile'
+        )
     end
 
-
     it "does not prevent the destroy if the association is not listed as preventing the destroy" do
       person.logs.create
       person.destroy
-      person.destroyed?.should be_true
+      expect(person.destroyed?).to be_truthy
     end
   end
 
@@ -43,7 +45,7 @@ describe "prevent_destroy_if_any" do
 
     it "does not prevent the destroy" do
       person.destroy
-      person.destroyed?.should be_true
+      expect(person.destroyed?).to be_truthy
     end
 
   end

data/spec/spec_helper.rb

@@ -10,7 +10,7 @@ require "database_cleaner"
 require "prevent_destroy_if_any"
 
 ActiveRecord::Base.configurations = { 'test' => { :adapter => 'sqlite3', :database => ':memory:' } }
-ActiveRecord::Base.establish_connection('test')
+ActiveRecord::Base.establish_connection(:test)
 
 class Person < ActiveRecord::Base
   belongs_to :user
@@ -37,7 +37,7 @@ class Log < ActiveRecord::Base
   belongs_to :person
 end
 
-ActiveRecord::Base.silence do
+ActiveRecord::Base.silence(:stdout) do
   ActiveRecord::Migration.verbose = false
   ActiveRecord::Schema.define :version => 0 do
     create_table(:people)          { |t| t.string :name; t.integer :user_id }

metadata

@@ -1,83 +1,113 @@
 --- !ruby/object:Gem::Specification
 name: prevent_destroy_if_any
 version: !ruby/object:Gem::Version
-  version: '0.1'
-  prerelease: 
+  version: '0.2'
 platform: ruby
 authors:
 - Florent Guilleux
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-08-07 00:00:00.000000000 -05:00
-default_executable: 
+date: 2014-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &2157972440 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 3.0.0
   type: :runtime
   prerelease: false
-  version_requirements: *2157972440
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+- !ruby/object:Gem::Dependency
+  name: i18n
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &2157971360 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *2157971360
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2157970640 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2157970640
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
-  requirement: &2157969820 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2157969820
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: sqlite3-ruby
-  requirement: &2157968200 !ruby/object:Gem::Requirement
-    none: false
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2157968200
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2157709680 !ruby/object:Gem::Requirement
-    none: false
+  requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2157709680
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: ActiveRecord plugin to prevent destroy when associations are present
 email:
 - florent2@gmail.com
@@ -85,46 +115,43 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
+- CHANGELOG.md
 - Gemfile
 - README.md
 - Rakefile
+- config/locales/en.yml
+- config/locales/es.yml
+- config/locales/fr.yml
 - lib/prevent_destroy_if_any.rb
 - lib/prevent_destroy_if_any/version.rb
 - prevent_destroy_if_any.gemspec
 - spec/prevent_destroy_if_any_spec.rb
 - spec/spec_helper.rb
-has_rdoc: true
 homepage: https://github.com/Florent2/prevent_destroy_if_any
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 3762508194909461440
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 3762508194909461440
 requirements: []
 rubyforge_project: prevent_destroy_if_any
-rubygems_version: 1.6.2
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: ActiveRecord plugin to prevent destroy when associations are present
 test_files:
 - spec/prevent_destroy_if_any_spec.rb
 - spec/spec_helper.rb
+has_rdoc: