tradesman 1.0.3 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 11e364aa8aa559b1fde2cc8dbe60a017ceae2b99
-  data.tar.gz: 490d82d0fdd65109eb01f8a1a3a6d9f0282029c2
+  metadata.gz: 10813cc7d5e09a97feaeeafd533d659bcecbfb90
+  data.tar.gz: b43ef980ddba913bd45345334a39aee135bd333f
 SHA512:
-  metadata.gz: d6924c2f37b47fefe39b121951b52407d7f8a7251e931a5207b2d6c8df6be6c297f0b7c1f5395f12f0211c53d31edbac90e8b41979f61893d95be67cc01b4db7
-  data.tar.gz: 635b74cfa73a87ae22900419d71d11c0c7c1110468079f39087cda3b25b3d5008c3e9e269253bf7a3a8ba8d374111aaa4da3747e843c565e62db9cafce844637
+  metadata.gz: 3dc5fad716ab91b6b8f7503ff3617987841afdb44a6da420f293e50d05d154d482791ae7a046f076a195ba195a2995a0d4f63508f2b9b900c8fcb5c4bc4f2eca
+  data.tar.gz: 879af35ef09b7e5058e134c6436e3fd6cc2d621ac8d9f370d773f13e3ebbe47fde8819bca6dcf19a728d7aaeab7c4b01bc9d44c3f10ef2ba06ab910ca5bc93c9

data/README.md

@@ -52,6 +52,9 @@ Tradesman::CreateUserForEmployer.go(employer, [user_params, user_params, user_pa
 # Update multiple records with 1 set of parameters
 Tradesman::UpdateUser.go([user1, user2, user3], update_params)
 
+# Update multiple records based on a query hash
+Tradesman::UpdateUser.go({ first_name: 'Blake' }, update_params)
+
 # Update n records with n sets of parameters
 update_params = {
   user1.id => user1_params,
@@ -63,6 +66,9 @@ Tradesman::UpdateUser.go(update_params.keys, update_params.values)
 
 # Delete multiple records
 Tradesman::DeleteUser.go([id1, id2, id3])
+
+# Delete multiple records based on a query hash
+Tradesman::DeleteUser.go(first_name: 'Blake')
 ```
 
 ## Parsing Rules

data/lib/tradesman/class_methods.rb

@@ -19,7 +19,8 @@ module Tradesman
     end
 
     def prepare_ids(obj)
-      return id_from_obj(obj) unless obj.is_a? Array
+      return obj if (obj.is_a?(Hash) && !obj.respond_to?(:id))
+      return id_from_obj(obj) unless (obj.is_a?(Array) || obj.respond_to?(:push))
       obj.map { |object| id_from_obj(object) }
     end
 

data/lib/tradesman/mixins/existing_records_multiple_execute.rb

@@ -2,6 +2,11 @@ module Tradesman
   module ExistingRecordsMultipleExecute
     private
 
+    def query_for_ids(params)
+      params[:id] = self.class.adapter.find_all(conditions: params[:id]).collect &:id
+      params
+    end
+
     def execute_multiple(params_hash)
       params = params_hash[:params] || params_hash.except(:id)
 

data/lib/tradesman/template.rb

@@ -6,6 +6,7 @@ module Tradesman
     extend ::Tradesman::ErrorHandling
 
     def call(params)
+      params = query_for_ids(params) if (params.is_a?(Hash) && params[:id] && params[:id].is_a?(Hash))
       return execute_single(params) unless (params.is_a?(Array) || params[:id].is_a?(Array) || params[:params].is_a?(Array))
       execute_multiple(params)
     rescue *self.class.expected_horza_errors_map.keys => e
@@ -14,6 +15,10 @@ module Tradesman
 
     private
 
+    def query_for_ids(params)
+      raise Tradesman::MethodNotImplemented.new('You must implement query_for_ids in a child class')
+    end
+
     def execute_single(params)
       raise Tradesman::MethodNotImplemented.new('You must implement execute_single in a child class')
     end

data/spec/tradesman_spec.rb

@@ -260,6 +260,25 @@ describe Tradesman do
           expect(results.last.last_name).to eq params.last[:last_name]
         end
       end
+
+      context 'passing query hash and single valid parameters' do
+        let(:query_params) { { last_name: 'Smith' } }
+        let(:update_params) { { last_name: 'Turner' } }
+        let(:other_params) { { last_name: 'Sharkasy' } }
+        let!(:records) { Tradesman::CreateStrictUser.go([valid_params, valid_params, valid_params]).result }
+        let!(:extraneous_records) { Tradesman::CreateStrictUser.go([other_params, other_params]).result }
+
+        before do
+          Tradesman::CreateStrictUser.go([query_params, query_params, query_params])
+        end
+        it 'updates all records that match the query' do
+          outcome = Tradesman::UpdateStrictUser.go(query_params, update_params)
+          expect(outcome.success?).to be true
+          expect(outcome.result.length).to eq 3
+          expect(outcome.result.first.last_name).to eq 'Turner'
+          expect(outcome.result.last.last_name).to eq 'Turner'
+        end
+      end
     end
   end
 
@@ -298,6 +317,24 @@ describe Tradesman do
           expect(outcome.result.uniq.first).to eq true
         end
       end
+
+      context 'passing query hash and single valid parameters' do
+        let(:query_params) { { last_name: 'Smith' } }
+        let(:update_params) { { last_name: 'Turner' } }
+        let(:records) { Tradesman::CreateStrictUser.go([valid_params, valid_params, valid_params]).result }
+
+        before do
+          Tradesman::CreateStrictUser.go([query_params, query_params, query_params])
+        end
+        it 'updates all records that match the query' do
+          outcome = Tradesman::DeleteStrictUser.go(query_params)
+          expect(outcome.success?).to be true
+          expect(outcome.result.length).to eq 3
+          expect(outcome.result.first).to be true
+          expect(outcome.result.second).to be true
+          expect(outcome.result.third).to be true
+        end
+      end
     end
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tradesman
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.5
 platform: ruby
 authors:
 - Blake Turner
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-01 00:00:00.000000000 Z
+date: 2015-07-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: horza
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.8
+        version: 0.3.9
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.3.8
+        version: 0.3.9
 - !ruby/object:Gem::Dependency
   name: tzu
   requirement: !ruby/object:Gem::Requirement