cardname 0.11.0 → 0.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2aaef149c22c0d7904227d548b3b7b3aafe37385f345f76334bbf42b97ef062d
-  data.tar.gz: c6456801e22e9481be99555e791469ebcdaa4d2cdd7070aea54e38bf2559e97d
+  metadata.gz: 0e0d301d80ce8422ca4aeb9bf8ce2307dc57d81a9629206bf660f5ebebd63474
+  data.tar.gz: a52dd72d962a26173c028a0c52e55988853dfd59a810ab6a8e46cf174ef3cd8d
 SHA512:
-  metadata.gz: eed77c0d0f1437ffa49bf414b5a8ac51dc3af4833e0a3b71d90eca8daa15901373c41fae2309dcc27502ee495ce8aa12b52d39a22c11b448c6b51a58f6720c47
-  data.tar.gz: 3179e5d7defa1a0391297e975cc2ee942ef403d72d453b89b80b09c84a77e10c5000ba60bbf0a5114bbc91cc3a50e09ee67efd059897e10776662a7d6917ac0a
+  metadata.gz: 7b9de9bf4b55d4a40ac188f8150f421f4d3ee8267b5a4eea07742842645853e8fa0360bec8dbd0ca3aa2581ad6512f8dfa596c9bb2c016bcb25b7f158ff4b6a3
+  data.tar.gz: 534c1e4c669960e45e445a074a0c380ba9a2212f83807ede593b09c7fc132a1b0ac2e97f5e3c3be692912aeaf1a7f03dc809d235f96dff3548b17a654bf90c3d

data/Rakefile

@@ -1,39 +1,37 @@
+# -*- encoding : utf-8 -*-
 
-# encoding: utf-8
+require "rubygems"
+require "bundler"
 
-require 'rubygems'
-require 'bundler'
-
-VERSION = File.exist?('VERSION') ? File.read('VERSION') : ""
+VERSION = File.exist?("VERSION") ? File.read("VERSION") : ""
 
 begin
   Bundler.setup(:default, :development)
 rescue Bundler::BundlerError => e
-  $stderr.puts e.message
-  $stderr.puts "Run `bundle install` to install missing gems"
+  warn e.message
+  warn "Run `bundle install` to install missing gems"
   exit e.status_code
 end
 
-require 'rake'
+require "rake"
 
-require 'rspec/core'
-require 'rspec/core/rake_task'
+require "rspec/core"
+require "rspec/core/rake_task"
 RSpec::Core::RakeTask.new(:spec) do |spec|
-  spec.pattern = FileList['spec/**/*_spec.rb']
+  spec.pattern = FileList["spec/**/*_spec.rb"]
 end
 
 RSpec::Core::RakeTask.new(:rcov) do |spec|
-  spec.pattern = 'spec/**/*_spec.rb'
+  spec.pattern = "spec/**/*_spec.rb"
   spec.rcov = true
 end
 
-task :default => :spec
+task default: :spec
 
-require 'rdoc/task'
+require "rdoc/task"
 Rake::RDocTask.new do |rdoc|
-
-  rdoc.rdoc_dir = 'rdoc'
+  rdoc.rdoc_dir = "rdoc"
   rdoc.title = "cardname #{VERSION}"
-  rdoc.rdoc_files.include('README*')
-  rdoc.rdoc_files.include('lib/**/*.rb')
+  rdoc.rdoc_files.include("README*")
+  rdoc.rdoc_files.include("lib/**/*.rb")
 end

data/lib/cardname.rb

@@ -5,12 +5,14 @@ require "htmlentities"
 
 class Cardname < String
   require_relative "cardname/parts"
+  require_relative "cardname/pieces"
   require_relative "cardname/variants"
   require_relative "cardname/contextual"
   require_relative "cardname/predicates"
   require_relative "cardname/manipulate"
 
   include Parts
+  include Pieces
   include Variants
   include Contextual
   include Predicates
@@ -18,12 +20,11 @@ class Cardname < String
 
   OK4KEY_RE = '\p{Word}\*'
 
-  cattr_accessor :joint, :banned_array, :var_re, :uninflect, :params,
-                 :session, :stabilize
+  cattr_accessor :joint, :banned_array, :var_re, :uninflect, :params, :session, :stabilize
 
   self.joint          = "+"
   self.banned_array   = []
-  self.var_re         = /\{([^\}]*\})\}/
+  self.var_re         = /\{([^}]*\})\}/
   self.uninflect      = :singularize
   self.stabilize      = false
 
@@ -116,7 +117,6 @@ class Cardname < String
     end
   end
 
-  # dangerous, too
   def []= index, val
     p = parts
     p[index] = val

data/lib/cardname/contextual.rb

@@ -4,7 +4,8 @@ class Cardname
 
     # @return true if name is left or right of context
     def child_of? context
-      return false unless junction?
+      return false unless compound?
+
       context_key = context.to_name.key
       absolute_name(context).parent_keys.include? context_key
     end
@@ -26,7 +27,7 @@ class Cardname
     end
 
     def starts_with_joint?
-      junction? && parts.first.empty?
+      compound? && parts.first.empty?
     end
 
     def from *from
@@ -36,16 +37,19 @@ class Cardname
     # if possible, relativize name into one beginning with a "+".  The new name must absolutize back to the correct
     # original name in the context of "from"
     def name_from *from
-      return self unless (remaining = remove_context *from)
+      return self unless (remaining = remove_context(*from))
+
       compressed = remaining.compact.unshift(nil).to_name  # exactly one nil at beginning
       key == compressed.absolute_name(from).key ? compressed : self
     end
 
     def remove_context *from
       return false unless from.compact.present?
-      remaining = parts_excluding *from
+
+      remaining = parts_excluding(*from)
       return false if remaining.compact.empty? || # all name parts in context
                       remaining == parts          # no name parts in context
+
       remaining
     end
 
@@ -60,6 +64,7 @@ class Cardname
         next if part.empty?
         next if part =~ /^_/ # this removes relative parts.  why?
         next if keys_to_ignore.member? part.to_name.key
+
         part
       end
     end
@@ -68,6 +73,7 @@ class Cardname
       context = (context || "").to_name
       new_parts = absolutize_contextual_parts context
       return "" if new_parts.empty?
+
       absolutize_extremes new_parts, context.s
       new_parts.join self.class.joint
     end
@@ -90,10 +96,10 @@ class Cardname
         when /^_main$/i            then self.class.params[:main_name]
         when /^(_self|_whole|_)$/i then context.s
         when /^_left$/i            then context.trunk
-        # note - inconsistent use of left v. trunk
+        # NOTE: - inconsistent use of left v. trunk
         when /^_right$/i           then context.tag
-        when /^_(\d+)$/i           then ordinal_part $~[1].to_i, context
-        when /^_(L*)(R?)$/i        then partmap_part $~, context
+        when /^_(\d+)$/i           then ordinal_part $LAST_MATCH_INFO[1].to_i, context
+        when /^_(L*)(R?)$/i        then partmap_part $LAST_MATCH_INFO, context
         else                            part
         end.to_s.strip
       end
@@ -109,20 +115,24 @@ class Cardname
     end
 
     def partmap_part match, context
-      l_s, r_s = match[1].size, !match[2].empty?
+      l_s = match[1].size
+      r_s = !match[2].empty?
       l_part = context.nth_left l_s
       r_s ? l_part.tag : l_part.s
     end
 
     def absolutize_extremes new_parts, context
       [0, -1].each do |i|
-        next if new_parts[i].present?
-        # following avoids recontextualizing with relative contexts.
-        # Eg, `+A+B+.absolute('+A')` should be +A+B, not +A+A+B.
-        next if new_parts.to_name.send "#{[ :start, :end ][i]}s_with_parts?", context
-        new_parts[i] = context
+        new_parts[i] = context if absolutize_extreme? new_parts, context, i
       end
     end
 
+    def absolutize_extreme? new_parts, context, index
+      return false if new_parts[index].present?
+
+      # following avoids recontextualizing with relative contexts.
+      # Eg, `+A+B+.absolute('+A')` should be +A+B, not +A+A+B.
+      !new_parts.to_name.send "#{%i[start end][index]}s_with_parts?", context
+    end
   end
 end

data/lib/cardname/manipulate.rb

@@ -8,6 +8,7 @@ class Cardname
       return self if old_name.num_parts > num_parts
       return swap_part(old_name, new_name) if old_name.simple?
       return self unless include? old_name
+
       swap_all_subsequences(old_name, new_name).to_name
     end
 
@@ -29,6 +30,7 @@ class Cardname
       return swap_part oldpiece, newpiece if oldpiece.simple?
       return self unless starts_with_parts?(oldpiece)
       return newpiece if oldpiece.num_parts == num_parts
+
       self.class.new [newpiece, self[oldpiece.num_parts..-1]].flatten
     end
 
@@ -72,6 +74,7 @@ class Cardname
 
     def ensure_simpleness part, msg=nil
       return if part.to_name.simple?
+
       raise StandardError, "'#{part}' has to be simple. #{msg}"
     end
   end

data/lib/cardname/parts.rb

@@ -4,24 +4,12 @@ class Cardname
   # the same methods without _name return strings
   module Parts
     attr_reader :parts, :part_keys, :simple
-
     alias_method :to_a, :parts
 
     def parts
       @parts = Cardname.split_parts s
     end
 
-    def simple
-      @simple = parts.size <= 1
-    end
-    alias simple? simple
-
-    # @return true if name has more than one part
-    def compound?
-      !simple?
-    end
-    alias junction? compound?
-
     def part_keys
       @part_keys ||= simple ? [simple_key] : parts.map { |p| p.to_name.simple_key }
     end
@@ -51,15 +39,15 @@ class Cardname
     end
 
     def parents
-      @parents ||= junction? ? [left, right] : []
+      @parents ||= compound? ? [left, right] : []
     end
 
     def parent_names
-      @parent_names ||= junction? ? [left_name, right_name] : []
+      @parent_names ||= compound? ? [left_name, right_name] : []
     end
 
     def parent_keys
-      @parent_keys ||= junction? ? [left_key, right_key] : []
+      @parent_keys ||= compound? ? [left_key, right_key] : []
     end
 
     # Note that all names have a trunk and tag,
@@ -89,27 +77,8 @@ class Cardname
       @piece_names ||= pieces.map(&:to_name)
     end
 
-    # self and all ancestors (= parts and recursive lefts)
-    # @example
-    #   "A+B+C+D".to_name.pieces
-    #   # => ["A", "B", "C", "D", "A+B", "A+B+C", "A+B+C+D"]
-    def pieces
-      @pieces ||=
-        if simple?
-          [self]
-        else
-          junction_pieces = []
-          parts[1..-1].inject parts[0] do |left, right|
-            piece = [left, right] * self.class.joint
-            junction_pieces << piece
-            piece
-          end
-          parts + junction_pieces
-        end
-    end
-
     def ancestors
-      @ancestors ||= pieces.reject { |p| p == self}
+      @ancestors ||= pieces.reject { |p| p == self }
     end
 
     # def + other

data/lib/cardname/pieces.rb

@@ -0,0 +1,26 @@
+class Cardname
+  # Cards never have more than two "parts" (left and right), but they can have many
+  # "pieces".  A card's pieces are all the other cards whose existence its existence
+  # implies. For example if A+B+C exists, that implies that A, B, C, and A+B do too.
+  module Pieces
+    # self and all ancestors (= parts and recursive lefts)
+    # @example
+    #   "A+B+C+D".to_name.pieces
+    #   # => ["A", "B", "C", "D", "A+B", "A+B+C", "A+B+C+D"]
+    def pieces
+      @pieces ||= simple? ? [self] : (parts + junction_pieces)
+    end
+
+    private
+
+    def junction_pieces
+      [].tap do |pieces|
+        parts[1..-1].inject parts[0] do |left, right|
+          piece = [left, right] * self.class.joint
+          pieces << piece
+          piece
+        end
+      end
+    end
+  end
+end

data/lib/cardname/predicates.rb

@@ -1,7 +1,19 @@
 class Cardname
   module Predicates
+    def simple?
+      @simple.nil? ? (@simple = parts.size <= 1) : @simple
+    end
+    alias_method :simple, :simple? # @deprecated
+
+    # @return true if name has more than one part
+    def compound?
+      !simple?
+    end
+    alias_method :junction?, :compound? # @deprecated
+
     def valid?
       return true if self.class.nothing_banned?
+
       !parts.find do |pt|
         pt.match self.class.banned_re
       end

data/lib/cardname/variants.rb

@@ -2,19 +2,20 @@ class Cardname
   module Variants
     def simple_key
       return "" if empty?
+
       decoded
         .underscore
-        .gsub(/[^#{OK4KEY_RE}]+/, '_')
+        .gsub(/[^#{OK4KEY_RE}]+/, "_")
         .split(/_+/)
         .reject(&:empty?)
         .map { |key| self.class.stable_key(key) }
-        .join('_')
+        .join("_")
     end
 
     def url_key
       @url_key ||= part_names.map do |part_name|
-        stripped = part_name.decoded.gsub(/[^#{OK4KEY_RE}]+/, ' ').strip
-        stripped.gsub(/[\s\_]+/, '_')
+        stripped = part_name.decoded.gsub(/[^#{OK4KEY_RE}]+/, " ").strip
+        stripped.gsub(/[\s_]+/, "_")
       end * self.class.joint
     end
 
@@ -22,11 +23,11 @@ class Cardname
     # but the key is no longer unique.
     # For example "A-XB" and "A+*B" have the same safe_key
     def safe_key
-      @safe_key ||= key.tr('*', 'X').tr self.class.joint, '-'
+      @safe_key ||= key.tr("*", "X").tr self.class.joint, "-"
     end
 
     def decoded
-      @decoded ||= s.index('&') ? HTMLEntities.new.decode(s) : s
+      @decoded ||= s.index("&") ? HTMLEntities.new.decode(s) : s
     end
 
     def to_sym

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cardname
 version: !ruby/object:Gem::Version
-  version: 0.11.0
+  version: 0.11.5
 platform: ruby
 authors:
 - Ethan McCutchen
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-24 00:00:00.000000000 Z
+date: 2021-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -53,13 +53,19 @@ files:
 - lib/cardname/contextual.rb
 - lib/cardname/manipulate.rb
 - lib/cardname/parts.rb
+- lib/cardname/pieces.rb
 - lib/cardname/predicates.rb
 - lib/cardname/variants.rb
 - lib/core_ext.rb
-homepage: http://decko.org
+homepage: https://decko.org
 licenses:
 - GPL-3.0
-metadata: {}
+metadata:
+  source_code_uri: https://github.com/decko-commons/decko
+  homepage_uri: https://decko.org
+  bug_tracker_uri: https://github.com/decko-commons/decko/issues
+  wiki_uri: https://decko.org
+  documentation_url: http://docs.decko.org/
 post_install_message:
 rdoc_options: []
 require_paths:
@@ -75,7 +81,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: Card names without all the cards