needle_in_a_haystack 1.0.3 → 1.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 47f3b28b275f615fb0cbfabb20d76cf70ef0adbf63875c6a2aae2c0b19e27e39
-  data.tar.gz: 9807a09a01410b9f299a268c828880df1e5fbbcf9cf6c7c16769b29da1fbb366
+  metadata.gz: af267c9afbbd079edbdcb7ac293721697491a7e47dac4d19963ad9af83fbd60d
+  data.tar.gz: e100eff5dfe19e038ffb8c6e8086b96eb1a034f87d3de1b66861ac91a05493b6
 SHA512:
-  metadata.gz: a940a2d003014274c368e4daf453f1ab35711057eaec59a465d61ff3b319395806486fa95d3d19c7e4071f6c2e0e49cc6514079ce14b7921df722345ba9c651b
-  data.tar.gz: 29b755d8403392d44cb9561d2a119b1c5b61bb3ebbb269ea4b7f78c241280e396c34c52bb4a1d895e5474d9cf90bae4a74945214f31803724d4734b318a28239
+  metadata.gz: d2870fbe87246fe0e39b6818fcd2ebffb5767af2a39a1d0b3636c2f93b8fc069ebdda4a4b0c7f15ae4af878fa5507571db6d29f367213510857a8f64d0d5f214
+  data.tar.gz: 6e49496b73f7d4bee5ee41e1bcc34e8e322eae66ac5af71419516c8532e7b79a2066827679dcce59680d3627cd9af4c2c968501cd9c14d07a299adb628fe72fa

data/lib/needle_in_a_haystack/concerns/base_factory.rb

@@ -0,0 +1,13 @@
+class BaseFactory
+  def create_tag(name, description)
+    raise NotImplementedError, "Subclasses must implement a create_tag-method"
+  end
+
+  def create_tagging(tag, taggable)
+    raise NotImplementedError, "Subclasses must implement a create_tagging-method"
+  end
+
+  def find_or_create_tag(name, attributes = {})
+    raise NotImplementedError, "Subclasses must implement a find_or_create_tag-method"
+  end
+end

data/lib/needle_in_a_haystack/concerns/base_tag.rb

@@ -1,11 +1,31 @@
 class BaseTag < ApplicationRecord
   self.abstract_class = true
 
-  def name
-    raise NotImplementedError, "Subclasses must implement a name-method"
+  def full_path
+    raise NotImplementedError, "Subclasses must implement a full_path-method"
   end
 
-  def description
-    raise NotImplementedError, "Subclasses must implement a description-method"
+  def prevent_circular_reference
+    raise NotImplementedError, "Subclasses must implement a prevent_circular_reference-method"
+  end
+
+  def ancestors
+    raise NotImplementedError, "Subclasses must implement an ancestors-method"
+  end
+
+  def descendants
+    raise NotImplementedError, "Subclasses must implement a descendants-method"
+  end
+
+  def siblings
+    raise NotImplementedError, "Subclasses must implement a siblings-method"
+  end
+
+  def root?
+    raise NotImplementedError, "Subclasses must implement a root?-method"
+  end
+
+  def leaf?
+    raise NotImplementedError, "Subclasses must implement a leaf?-method"
   end
 end

data/lib/needle_in_a_haystack/models/haystack_ontology.rb

@@ -48,19 +48,25 @@ class HaystackOntology < ApplicationRecord
       tag = HaystackTag.find_or_create_by(name: name.to_s)
       tag.update(description: data["description"], parent_tag_id: parent_tag&.id, haystack_marker: data["marker"])
 
-      Rails.logger.info("Created tag: #{tag.name}, Parent: #{parent_tag&.name}")
+      Rails.logger.info("Created tag: #{tag.name}, Parent: #{parent_tag&.name}, Parent ID: #{parent_tag&.id}")
 
-      create_tag_hierarchy(data["children"], tag) if data["children"]
+      if data["children"]
+        Rails.logger.info("Processing children for tag: #{tag.name}")
+        create_tag_hierarchy(data["children"], tag)
+      end
     end
   end
 
   def self.find_or_create_tag(name)
+    factory = HaystackFactory.new
     tag_data = find_tag(name)
     return nil if tag_data.nil?
 
-    tag = HaystackTag.find_or_create_by(name: name)
-    tag.update(description: tag_data["description"], haystack_marker: tag_data["marker"])
-    Rails.logger.info("Tag aangemaakt of gevonden: #{tag.name}")
-    tag
+    attributes = { description: tag_data["description"], haystack_marker: tag_data["marker"] }
+    factory.find_or_create_tag(name, attributes)
+  end
+
+  def self.import_full_ontology
+    create_tags
   end
 end

data/lib/needle_in_a_haystack/models/haystack_tag.rb

@@ -8,14 +8,6 @@ class HaystackTag < BaseTag
   validates :description, presence: true
   validate :prevent_circular_reference
 
-  def name
-    self[:name]
-  end
-
-  def description
-    self[:description]
-  end
-
   def ancestors
     ancestors = []
     current = self
@@ -26,23 +18,26 @@ class HaystackTag < BaseTag
     ancestors
   end
 
+  def full_path
+    ancestors.reverse.map(&:name).join(" > ") + " > " + name
+  end
+
   def self.find_by_path(path)
     keys = path.split(".")
     current = nil
     keys.each do |key|
-      current = current ? current.child_tags.find_by(name: key) : find_by(name: key)
+      current = current ? current.children.find_by(name: key) : find_by(name: key)
       return nil unless current
     end
     current
   end
 
   def descendants
-    children = child_tags
     children + children.flat_map(&:descendants)
   end
 
   def siblings
-    parent_tag ? parent_tag.child_tags.where.not(id: id) : self.class.where(parent_tag_id: nil).where.not(id: id)
+    parent_tag ? parent_tag.children.where.not(id: id) : self.class.where(parent_tag_id: nil).where.not(id: id)
   end
 
   def root?
@@ -50,7 +45,7 @@ class HaystackTag < BaseTag
   end
 
   def leaf?
-    child_tags.empty?
+    children.empty?
   end
 
   def depth
@@ -60,6 +55,6 @@ class HaystackTag < BaseTag
   def prevent_circular_reference
     return unless parent_tag == self || ancestors.include?(self)
 
-    errors.add(:parent_tag, "kan geen circulaire referentie bevatten")
+    errors.add(:parent_tag, "cannot contain a circular reference")
   end
 end

data/lib/needle_in_a_haystack/services/haystack_factory.rb

@@ -0,0 +1,15 @@
+class HaystackFactory < BaseFactory
+  def create_tag(name, description)
+    HaystackTag.create(name: name, description: description)
+  end
+
+  def create_tagging(tag, taggable)
+    HaystackTagging.create(haystack_tag: tag, taggable: taggable)
+  end
+
+  def find_or_create_tag(name, attributes = {})
+  tag = HaystackTag.find_or_create_by(name: name)
+  tag.update(attributes)
+  tag
+  end
+end

data/lib/needle_in_a_haystack/version.rb

@@ -1 +1 @@
-VERSION = "1.0.3".freeze
+VERSION = "1.0.4".freeze

data/lib/needle_in_a_haystack.rb

@@ -4,10 +4,12 @@ require "needle_in_a_haystack/application_record"
 require "needle_in_a_haystack/configuration"
 require "needle_in_a_haystack/concerns/base_tagging"
 require "needle_in_a_haystack/concerns/base_tag"
+require "needle_in_a_haystack/concerns/base_factory"
 
 require "needle_in_a_haystack/models/haystack_ontology"
 require "needle_in_a_haystack/models/haystack_tag"
 require "needle_in_a_haystack/models/haystack_tagging"
+require "needle_in_a_haystack/services/haystack_factory"
 
 module NeedleInAHaystack
   class << self

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: needle_in_a_haystack
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.0.4
 platform: ruby
 authors:
 - Frans Verberne
@@ -277,6 +277,7 @@ files:
 - config/locales/en.yml
 - lib/needle_in_a_haystack.rb
 - lib/needle_in_a_haystack/application_record.rb
+- lib/needle_in_a_haystack/concerns/base_factory.rb
 - lib/needle_in_a_haystack/concerns/base_tag.rb
 - lib/needle_in_a_haystack/concerns/base_tagging.rb
 - lib/needle_in_a_haystack/concerns/taggable.rb
@@ -285,6 +286,7 @@ files:
 - lib/needle_in_a_haystack/models/haystack_ontology.rb
 - lib/needle_in_a_haystack/models/haystack_tag.rb
 - lib/needle_in_a_haystack/models/haystack_tagging.rb
+- lib/needle_in_a_haystack/services/haystack_factory.rb
 - lib/needle_in_a_haystack/version.rb
 - lib/tasks/location_create.rake
 - spec/factories/haystack_tags.rb