lean_tag 1.0.1 → 1.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9a5597de96b7f9ec49591cf11c3d4c4969d2beb1
-  data.tar.gz: 88328d047bad479c91c3249181be49decee783d9
+  metadata.gz: f369a5cfbafb5c23c6842f54d5d4559fa7f66569
+  data.tar.gz: 8f603c5cd549aaae3d8fe1b6cd955eb1c48341ec
 SHA512:
-  metadata.gz: ab0269cf7dc8a631806289b29a1f3713c71e19a55ca0fcd43be9b22ec6c6ec98961b425cd2b723e7a7e8fc6dcd99ebfe1d2553d7ced573cf079a7c9bc9b38e3e
-  data.tar.gz: 13dafdafb371590755c1d77d04d6e8ecd501fcc88b0c15954729dea98dd13e003b2afd3ba9b751d2a7d8e371dc57b3b7f7d612503e934c000ef37149ea05db7b
+  metadata.gz: f765bdb09d95df190fddc8c54c12bf074368f24ad7ac10ced9338a6e6896c25eea9a04629d01fed5d0606c41248511afd0cd72e2bf2673f726c278926cfe9749
+  data.tar.gz: 4129c3c05ba79ee0d85faad922d520cffcbe7a51d31d1f924c16212bb04c4c15f8fdaa4ed1eb00d6906b0b88dda9ab2090fc152cd269f8b3c9fd4ab6c3599620

data/lib/lean_tag/engine.rb

@@ -6,6 +6,8 @@ require 'lean_tag/tagging'
 
 module LeanTag
   class Engine < ::Rails::Engine
+
     isolate_namespace LeanTag
+
   end
 end

data/lib/lean_tag/tag.rb

@@ -1,5 +1,6 @@
 module LeanTag
   class Tag < ActiveRecord::Base
+
     self.table_name = :tags
 
     has_many :records, through: :taggings
@@ -12,9 +13,10 @@ module LeanTag
 
     def name=(value)
       if value.present?
-        self[:name] = value.gsub(/[^0-9a-zA-Z]+/, "")
+        self[:name] = value.gsub(LeanTag.config.legal_characters, "")
         self[:name] = self[:name].downcase if LeanTag.config.force_lowercase
       end
     end
+
   end
 end

data/lib/lean_tag/taggable.rb

@@ -1,6 +1,8 @@
 module LeanTag
   module Taggable
+
     def self.extended(base)
+
       def taggable_on(relation="tags")
         extend ClassMethods
 
@@ -18,13 +20,16 @@ module LeanTag
           define_method "add_#{tag_relation.to_s.singularize}!", ->(tag) { _add_tag!(tag, tag_relation) }
           define_method "#{tag_relation.to_s.singularize}_list", ->() { _get_tag_list(tag_relation) }
           define_method "remove_#{tag_relation.to_s.singularize}", ->(tag) { _remove_tag(tag, tag_relation) }
-          define_method "remove_#{tag_relation.to_s.singularize}!", ->(tag) { _remove_tag!(tag, tag_relation) }
+          define_method "remove_#{tag_relation.to_s.singularize}!", ->(tag) { _remove_tag(tag, tag_relation) }
           define_method "#{tag_relation.to_s.singularize}_list=", ->(list) { _set_tag_list(list, tag_relation) }
         end
       end
+
     end
 
+
     module ClassMethods
+
       ##
       # Returns all records which include at least one of the passed tags
       def tagged_with(list, filter="tags")
@@ -34,25 +39,27 @@ module LeanTag
         return self.where(id: taggings.select(:record_id).distinct)
       end
 
+
       ##
       # Get all tags used by this class
       def tags(filter="tags")
         taggings = Tagging.where(record_type: self.name, filter: filter).where(filter: filter)
         return Tag.where(id: taggings.select(:tag_id).distinct)
       end
+
     end
 
+
     module InstanceMethods
+
       ##
       # Adds a single tag on parent save
       def _add_tag(tag, filter="tags")
         if tag.is_a?(String)
-          record = Tag.find_by_name(tag)
+          record = Tag.find_or_create_by(name: tag)
         end
 
-        if record.nil?
-          self._tags(filter).build(name: tag)
-        elsif !self._taggings(filter).exists?(tag_id: record.id)
+        unless self._taggings(filter).exists?(tag_id: record.id)
           self._taggings(filter).build(tag_id: record.id)
         end
       end
@@ -97,8 +104,9 @@ module LeanTag
           tag_names = []
         end
 
-        tags = Tag.where(id: self._taggings(filter).select(:tag_id).distinct)
-        tags.reject { |t| t.name.in?(tag_names) }.each { |t| self._remove_tag(t, filter) }
+        tags = self._taggings(filter).map(&:tag)
+        tags.reject! { |t| t.name.in?(tag_names) }
+        tags.each { |t| self._remove_tag(t, filter) }
 
         # Add any new tags
         tag_names.each { |t| self._add_tag(t, filter) }
@@ -115,6 +123,8 @@ module LeanTag
       def _taggings(filter)
         self.send("#{filter}_taggings")
       end
+
     end
+
   end
 end

data/lib/lean_tag/tagging.rb

@@ -1,5 +1,6 @@
 module LeanTag
   class Tagging < ActiveRecord::Base
+
     self.table_name = :taggings
 
     belongs_to :record, polymorphic: true, inverse_of: :taggings
@@ -7,8 +8,8 @@ module LeanTag
 
     scope :for, -> (filter) { where(filter: filter) }
 
-    validates :record_id, presence: true
     validates :record_type, presence: true
     validates :tag, presence: true, uniqueness: { scope: [:record_type, :record_id] }
+
   end
 end

data/lib/lean_tag/version.rb

@@ -1,3 +1,5 @@
 module LeanTag
-  VERSION = '1.0.1'
+
+  VERSION = '1.0.5'
+
 end

data/lib/lean_tag.rb

@@ -5,6 +5,7 @@ require 'active_support/core_ext/module'
 require_relative 'lean_tag/engine' if defined?(Rails)
 
 module LeanTag
+
   def self.config
     @configuration ||= Configuration.new
   end
@@ -14,13 +15,18 @@ module LeanTag
     yield self.config if block_given?
   end
 
+
   class Configuration
-    attr_accessor :delimiter, :force_lowercase, :remove_unused
+
+    attr_accessor :delimiter, :force_lowercase, :legal_characters, :remove_unused
 
     def initialize
       self.delimiter = ','
       self.force_lowercase = true
+      self.legal_characters = /[^0-9a-zA-Z]+/
       self.remove_unused = true
     end
+
   end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: lean_tag
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.5
 platform: ruby
 authors:
 - Matt Ellis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-19 00:00:00.000000000 Z
+date: 2015-04-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -103,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14
+rubygems_version: 2.4.4
 signing_key: 
 specification_version: 4
 summary: A lightweight method for tagging content