xapian_db 1.3.7.4 → 1.3.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1979b59ffcc42e298d5696f7729c76e4e59394fc35e230c88c3bdbf0830279d7
-  data.tar.gz: 672b7bbbe1c2e2833fc09e57e5bf8916bae46812c31608f122e7b9732d39af6f
+  metadata.gz: a9a56c8dcca3d9af676803407a5f0ba9ca07d72a33c877909f9e5b7bc0ab884c
+  data.tar.gz: 19f87585356eb95bbe54c4e22d10b770b953c09c74856ffe10441fcf053c530d
 SHA512:
-  metadata.gz: 3094d0d66b745bf3dee99f9569ada5239d990812c50a7c995fd588cd140ce8f11b659a504340ae345db17009dc1e2873d47b9934636989e4bdfe7851e113cdb5
-  data.tar.gz: 9ff35b2818e51c8e7654f3ee4fae591a32c8cdf03c5ab7c99097a728df4ddaa4adadf5e04fbf9ea515ca9ae5b0fc7b55752835f9e80722a95039e28b6725828c
+  metadata.gz: 107aa90d69efc2dccd06f104a66fd9b0a252491c1110ebbc879e3998c74fd7619b1c41c523af7f8017611287ad339290f5a0aec72b07825687873af9f6780ae4
+  data.tar.gz: 3052ca4107227f97f48bbbc023478279e2f06c952b604d77269bb52ae3add718675940ce044a99b616037e801144ed09c474ad53ed7344853046e8200ce7d2aa

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.3.8 (July 19th, 2021)
+
+Fixes:
+
+- xapian docs are deleted in the after commit hook in order to respect transaction rollbacks
+
 ## 1.3.7.4 (October 27th, 2020)
 
 Changes:

data/README.rdoc

@@ -61,7 +61,7 @@ You can override these defaults by placing a config file named 'xapian_db.yml' i
 
   # XapianDb configuration
   defaults: &defaults
-    adapter: datamapper # Avaliable adapters: :active_record, :datamapper
+    adapter: datamapper # Available adapters: :active_record, :datamapper
     language: de        # Global language; can be overridden for specific blueprints
     term_min_length: 2  # Ignore single character terms
     enabled_query_flags: FLAG_PHRASE, FLAG_SPELLING_CORRECTION
@@ -226,7 +226,8 @@ If you want to manage the (re)indexing of your objects on your own, turn off aut
     blueprint.autoindex false
   end
 
-This will turn off the auto-reindexing for any object of the configured class. Use XapianDb.reindex(object) to trigger the reindexing logic in your code. Please note that destroying an object will always remove it from the xapian index.
+This will turn off the auto-reindexing for any object of the configured class. Use XapianDb.reindex(object) to trigger the reindexing logic in your code.
+It will also turn off the auto-deletion of the doc, when the object gets destroyed. Use XapianDb.delete_doc_with(object.xapian_id) to trigger deletion logic in your code.
 
 
 Place these configurations either into the corresponding class or - I prefer to have the index configurations outside

data/lib/xapian_db/adapters/active_record_adapter.rb

@@ -48,19 +48,16 @@ module XapianDb
            klass.class_eval do
              # add the after commit logic, unless the blueprint has autoindexing turned off
              if XapianDb::DocumentBlueprint.blueprint_for(klass.name).autoindex?
-               after_commit do
-                 unless self.destroyed?
-                   XapianDb.reindex(self, true, changed_attrs: self.previous_changes.keys)
-                   XapianDb::DocumentBlueprint.dependencies_for(klass.name, self.previous_changes.keys).each do |dependency|
-                     dependency.block.call(self).each{ |model| XapianDb.reindex model, true, changed_attrs: self.previous_changes.keys }
-                   end
+               after_commit on: [:create, :update] do
+                 XapianDb.reindex(self, true, changed_attrs: self.previous_changes.keys)
+                 XapianDb::DocumentBlueprint.dependencies_for(klass.name, self.previous_changes.keys).each do |dependency|
+                   dependency.block.call(self).each{ |model| XapianDb.reindex model, true, changed_attrs: self.previous_changes.keys }
                  end
                end
-             end
 
-             # add the after destroy logic
-             after_destroy do
-               XapianDb.delete_doc_with(self.xapian_id)
+               after_commit on: :destroy do
+                 XapianDb.delete_doc_with(self.xapian_id)
+               end
              end
 
              # Add a method to reindex all models of this class

data/lib/xapian_db/adapters/datamapper_adapter.rb

@@ -47,22 +47,22 @@ module XapianDb
            end
 
            klass.class_eval do
+             # add the after save/destroy logic, unless the blueprint has autoindexing turned off
+             if XapianDb::DocumentBlueprint.blueprint_for(klass.name).autoindex?
+               after :save do
+                 blueprint = XapianDb::DocumentBlueprint.blueprint_for klass.to_s
+                 if blueprint.should_index?(self)
+                   XapianDb.index(self)
+                 else
+                   XapianDb.delete_doc_with(self.xapian_id)
+                 end
+               end
 
-             # add the after save logic
-             after :save do
-               blueprint = XapianDb::DocumentBlueprint.blueprint_for klass.to_s
-               if blueprint.should_index?(self)
-                 XapianDb.index(self)
-               else
+               after :destroy do
                  XapianDb.delete_doc_with(self.xapian_id)
                end
              end
 
-             # add the after destroy logic
-             after :destroy do
-               XapianDb.delete_doc_with(self.xapian_id)
-             end
-
              # Add a method to reindex all models of this class
              define_singleton_method(:rebuild_xapian_index) do |options={}|
                options[:primary_key] = klass.serial.name

data/lib/xapian_db/document_blueprint.rb

@@ -352,7 +352,7 @@ module XapianDb
     # @param [Array] args An array of arguments; you can pass a method name, an array of method names
     #   or a method name and an options hash.
     # @param [Block] &block An optional block for complex configurations
-    # Avaliable options:
+    # Available options:
     # - :weight (default: 1) The weight for this indexed value
     # @example Simple index declaration
     #   blueprint.index :name

data/lib/xapian_db/index_writers/beanstalk_worker.rb

@@ -13,7 +13,7 @@ module XapianDb
       def index_task(options)
         klass = constantize options[:class]
         obj   = klass.respond_to?(:get) ? klass.get(options[:id]) : klass.find(options[:id])
-        DirectWriter.index obj, true, changed_attrs: options[:changed_attrs]
+        DirectWriter.index obj, options[:commit], changed_attrs: options[:changed_attrs]
       end
 
       def delete_doc_task(options)

data/lib/xapian_db/index_writers/beanstalk_writer.rb

@@ -19,7 +19,7 @@ module XapianDb
         # Update an object in the index
         # @param [Object] obj An instance of a class with a blueprint configuration
         def index(obj, commit=true, changed_attrs: [])
-          beanstalk.put( { :task => "index_task", :class => obj.class.name, :id => obj.id, :changed_attrs => changed_attrs }.to_json )
+          beanstalk.put( { :task => "index_task", :class => obj.class.name, :id => obj.id, :changed_attrs => changed_attrs, :commit => commit }.to_json )
         end
 
         # Remove an object from the index

data/lib/xapian_db/indexer.rb

@@ -59,13 +59,16 @@ module XapianDb
     # Index all configured text methods
     def index_text
       term_generator = Xapian::TermGenerator.new
-      term_generator.database = @database.writer
       term_generator.document = @xapian_doc
       if XapianDb::Config.stemmer
         term_generator.stemmer  = XapianDb::Config.stemmer
         term_generator.stopper  = XapianDb::Config.stopper if XapianDb::Config.stopper
         # Enable the creation of a spelling dictionary if the database is not in memory
-        term_generator.set_flags Xapian::TermGenerator::FLAG_SPELLING if @database.is_a? XapianDb::PersistentDatabase
+        if @database.is_a?(XapianDb::PersistentDatabase) &&
+           XapianDb::Config.query_flags.include?(Xapian::QueryParser::FLAG_SPELLING_CORRECTION)
+          term_generator.database = @database.writer
+          term_generator.set_flags Xapian::TermGenerator::FLAG_SPELLING
+        end
       end
 
       # Index the primary key as a unique term

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: xapian_db
 version: !ruby/object:Gem::Version
-  version: 1.3.7.4
+  version: 1.3.8
 platform: ruby
 authors:
 - Gernot Kogler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-08-27 00:00:00.000000000 Z
+date: 2021-07-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: daemons
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.22
+        version: 1.4.17
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.22
+        version: 1.4.17
 - !ruby/object:Gem::Dependency
   name: pry-rails
   requirement: !ruby/object:Gem::Requirement