active_merge 1.0.0 → 1.0.1

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 9f04e96b337986ab01516a571764953292c4cf46
+  data.tar.gz: e51483094adf87e9f17596876c74feda706dddad
+SHA512:
+  metadata.gz: b1c2930d0ffb8c6c0e996fb2aeef7701d211031dfcad1ff6e90f0ef117620de45f676028347c1763785f2a51acab8311b26592873ccd483ea982972f449aecc2
+  data.tar.gz: a4365f27dd04f85929042e8f07a25837386af8bfc0c96d12dd5f7c0ddaca69d867a74b50c5853c319f8dd503cd86399fc2b59269a5f13b265a5b56a69a52d9f2

data/lib/active_merge/service.rb

@@ -67,99 +67,67 @@ module ActiveMerge
   # то объединения не произойдет!
   #
   class Service
+    include ActiveModel::Validations
 
     def initialize(list = [])
-      list = _extract_from list
+      list = extract_from list
       @item, @items = list.first, Array(list[1..-1])
     end
 
     attr_reader :item, :items, :klass, :klasses
-
-    # Возвращает класс объединяемых объектов
-    def klass
-      @klass ||= item.class
-    end
-
-    # Возвращает хэш, в котором ключами выступают связи has_many,
-    # а значениями - foreign keys
-    def klasses
-      @klasses ||= klass.reflect_on_all_associations(:has_many).
-        inject({}){ |hash, item| hash.merge(item.name => item.foreign_key) }
-    end
+    validates   :items, presence: true
 
     # Объединяет все записи из переменной #items с записью в переменной #item
     #
     # При этом:
-    # * все ссылки на записи из массива #items (связь has_many) перепривязываются к #item
+    # * все ссылки на записи из массива #items перепривязываются к #item
     # * все записи #items удаляются
     #
     # При любой ошибке все сделанные изменения отменяются.
     #
     def provide
       ActiveRecord::Base.transaction requires_new: true do
-        _raise :blank if items.blank?
-        items.each{ |item| _merge_one(item) }
+        raise unless valid?
+        items.each{ |item| merge self.item, item }
       end
     end
 
     private
 
-      # Вызывает исключение указанного типа с переданными парамертами
-      def _raise(type, item = nil)
-        options = item ? { type: item.class.name.underscore, id: item.id } : {}
-        options.merge! scope: %w(active_merge errors service)
-        raise I18n.t(type, options)
-      end
-
       # Извлекает из списка только сохраненные объекты ActiveRecord
       # Если список содержит элементы ActiveRecord разных классов, возвращает
       # пустой массив.
-      def _extract_from(list)
+      def extract_from(list)
         begin
-          list  = list.select{ |item| _valid? item }
-          klass = list.first.class
-          list.each { |item| raise unless item.class == klass }
-          list.sort_by { |item| item.id }
+          list = select_active_records_from list
+          list = select_by_class_from list
         rescue
           []
         end
       end
 
-      # Проверяет, что объект является сохраненным объектом ActiveRecord
-      def _valid?(item)
-        begin
+      # Выбирает сохраненные объекты ActiveRecord
+      def select_active_records_from(list)
+        list.select do |item|
           item.class.ancestors.include?(ActiveRecord::Base) && item.persisted?
-        rescue
-          false
-        end
-      end
-
-      # Объединяет указанный объект с объектом из переменной #item
-      # * Перепривязывает ссылки на объект #item
-      # * Удаляет объект, переданный в аргументе
-      def _merge_one(item)
-        klasses.each do |list, foreign_key|
-          item.send(list).each { |ref| _rebind! ref, foreign_key }
         end
-        _destroy! item
       end
 
-      # Перепривязывает запись к первому объединяемому объекту
-      def _rebind!(item, foreign_key)
-        begin
-          item.send "#{ foreign_key }=", self.item.id
-          item.save!
-        rescue
-          _raise :rebind, item
-        end
+      # Выбирает объекты того же класса, что и у первой записи списка
+      def select_by_class_from(list)
+        klass = list.first.class
+        list.each { |item| raise unless item.class == klass }
+        list.sort_by { |item| item.id }
       end
 
-      # Удаляет запись, сохраняя ошибки
-      def _destroy!(item)
+      # Объединяет две записи.
+      # Переносит ошибки в текущий объект.
+      def merge(first, second)
+        service = ActiveMerge::SimpleService.new(first, second)
         begin
-          item.destroy!
+          service.provide
         rescue
-          _raise :destroy, item
+          service.errors.each{ |key, val| errors.add key, val } and raise
         end
       end
   end

data/lib/active_merge/simple_service.rb

@@ -0,0 +1,77 @@
+# encoding: utf-8
+module ActiveMerge
+
+  # Сервисный объект (паттерн Service Object), отвечающий за объединение
+  # двух записей ActiveRecord
+  #
+  # В ходе объединения все ссылки на вторую запись перепривязываются к первой,
+  # затем вторая запись удаляется
+  #
+  # В случае любой ошибки вызывается исключение, а ошибка добавляется в массив
+  # <tt>errors</tt>
+  #
+  class SimpleService
+    include ActiveModel::Validations
+
+    def initialize(first, second)
+
+      if first.class.ancestors.include?(ActiveRecord::Base) && first.persisted?
+        @first = first
+      end
+
+      if @first && (second.class == @first.class) && second.persisted?
+        @second = second
+      end
+    end
+    attr_reader :first, :second
+
+    validates :second, presence: true
+
+    def provide
+      ActiveRecord::Base.transaction requires_new: true do
+        raise unless valid?
+        refs.each{ |item, key| rebind(item, key) }
+        destroy
+      end
+    end
+
+    private
+
+      # Возвращает хэш, в котором ключами выступают связи has_many,
+      # а значениями - foreign keys
+      def klasses
+        second.class.reflect_on_all_associations(:has_many).
+          inject({}){ |hash, item| hash.merge(item.name => item.foreign_key) }
+      end
+
+      # Возвращает хэш, в котором ключами выступают объекты, ссылающихся на 
+      # #second, а значениями - методы присвоения ссылки на #first.
+      def refs
+        @refs = {}
+        klasses.each do |list, key|
+          second.send(list).each{ |item| @refs[item] = "#{ key }=" }
+        end
+        return @refs
+      end
+
+      # Выполняет перепривязку указанного объекта к первому аргументу
+      # Ошибки накапливаются в массиве <tt>errors</tt>
+      def rebind(item, key)
+        begin
+          item.send key, first.id
+          item.save!
+        rescue
+          item.errors.each{ |key, val| errors.add key, val } and raise
+        end
+      end
+
+      # Удаляет объединяемый объект
+      def destroy
+        begin
+          second.destroy!
+        rescue
+          second.errors.each{ |key, val| errors.add key, val } and raise
+        end
+      end
+  end
+end

data/lib/active_merge/version.rb

@@ -1,4 +1,4 @@
 module ActiveMerge
   # Текущая версия плагина
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/lib/active_merge.rb

@@ -7,6 +7,8 @@
 #
 module ActiveMerge
   extend ActiveSupport::Autoload
+
+  autoload :SimpleService
   autoload :Service
 
   # Объединение указанных записей.

metadata

@@ -1,94 +1,83 @@
 --- !ruby/object:Gem::Specification
 name: active_merge
 version: !ruby/object:Gem::Version
-  version: 1.0.0
-  prerelease: 
+  version: 1.0.1
 platform: ruby
 authors:
 - Andrew Kozin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-08 00:00:00.000000000 Z
+date: 2014-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 4.0.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 4.0.3
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: yard
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Declares the ActiveMerge module with the 'merge!' class method.
@@ -98,36 +87,36 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/active_merge/service.rb
-- lib/active_merge/version.rb
-- lib/active_merge.rb
 - MIT-LICENSE
-- Rakefile
 - README.rdoc
+- Rakefile
+- lib/active_merge.rb
+- lib/active_merge/service.rb
+- lib/active_merge/simple_service.rb
+- lib/active_merge/version.rb
 homepage: https://github.com/nepalez/active_merge
 licenses:
 - MIT
+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'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Merges instances of an ActiveRecord class.
 test_files: []
 has_rdoc: