gummi 1.0.0 → 1.0.2

data/bin/console

@@ -6,9 +6,9 @@ $: << ::File.expand_path('../..', __FILE__)
 
 require 'pry'
 require 'lib/gummi'
-require 'spec/models/db/person'
-require 'spec/models/person'
-require 'spec/models/people'
+require 'spec/support/models/db/person'
+require 'spec/support/models/person'
+require 'spec/support/models/people'
 RAILS_ENV = ENV['RAILS_ENV'] || 'development'
 
 def reload!
@@ -18,4 +18,4 @@ def reload!
 end
 
 Pry.config.prompt = [ proc { "gummi >> " }, proc { "... >> " }]
-Pry.start
+Pry.start

data/lib/gummi/db_layer/document.rb

@@ -18,6 +18,16 @@ module Gummi
         # Public Persistence API
         # ––––––––––––––––––––––
 
+        # Public: Checks if specified document exists
+        #
+        # id - The document id
+        #
+        # Examples
+        #
+        #   DB::Person.exists?('123')
+        #   # => true
+        #
+        # Returns true or false
         def exists?(id)
           client.exists index: index.name, type: document_type, id: id
         end
@@ -27,11 +37,21 @@ module Gummi
           instance.create ? instance : nil
         end
 
-        def get(*args)
-          get! *args
+        def get(id, parent_id = nil)
+          options = { index: index.name, type: document_type, id: id, fields: %w{ _source _parent } }
+          if parent_id
+            options.merge! parent: parent_id
+          elsif parent_document_type
+            raise ArgumentError, "The parent_id attribute is required for getting #{name} from Elastic Search"
+          end
+          response = ActiveSupport::Notifications.instrument "search.elasticsearch", name: "Document#get", search: options do
+            Hashie::Mash.new client.get options
+          end
+          hit_to_document response
         rescue ::Elasticsearch::Transport::Transport::Errors::NotFound
           nil
         end
+        alias :find :get
 
         def delete(*args)
           delete! *args
@@ -46,8 +66,10 @@ module Gummi
           else
             options = {}
           end
-          response = Hashie::Mash.new client.update options.merge(index: index.name, type: document_type, id: id, body: { doc: attributes })
-          response.ok
+          client.update options.merge(index: index.name, type: document_type, id: id, body: { doc: attributes })
+          true
+        rescue ::Elasticsearch::Transport::Transport::Errors::NotFound
+          nil
         end
 
         def delete_children_by_query(parent_id, children_query)
@@ -165,18 +187,6 @@ module Gummi
         # Internal Persistence API
         # ––––––––––––––––––––––––
 
-        def get!(id, parent_id = nil)
-            options = { index: index.name, type: document_type, id: id, fields: %w{ _source _parent } }
-            if parent_id
-              options.merge! parent: parent_id
-            elsif parent_document_type
-              raise ArgumentError, "The parent_id attribute is required for getting #{name} from Elastic Search"
-            end
-            response = ActiveSupport::Notifications.instrument "search.elasticsearch", name: "Document#get!", search: options do
-              Hashie::Mash.new client.get options
-            end
-            hit_to_document response
-        end
 
         def delete!(id, parent_id = nil)
           options = { index: index.name, type: document_type, id: id }
@@ -188,7 +198,7 @@ module Gummi
           response = ActiveSupport::Notifications.instrument "search.elasticsearch", name: "Document#delete!", search: options do
             Hashie::Mash.new client.delete options
           end
-          response.ok && response.found
+          response.found
         end
 
       end

data/lib/gummi/version.rb

@@ -1,3 +1,3 @@
 module Gummi
-  VERSION = '1.0.0'
+  VERSION = '1.0.2'
 end

data/spec/support/models/people.rb

@@ -1,5 +1,5 @@
-require 'db/person'
-require 'person'
+require_relative 'db/person'
+require_relative 'person'
 
 class People
   include Gummi::RepositoryLayer::Repository

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gummi
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-14 00:00:00.000000000 Z
+date: 2014-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: virtus
@@ -274,18 +274,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 3914732375805973470
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: 3914732375805973470
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -317,3 +311,4 @@ test_files:
 - spec/support/models/db/ship.rb
 - spec/support/models/people.rb
 - spec/support/models/person.rb
+has_rdoc: