card-mod-collection 0.13.2 → 0.14.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b436094f8129287bd5192f58cbb34d5b73322c4a8de0f5865f599ad803c22694
-  data.tar.gz: 4a947d0a8e9fc4b6e55d883c5fd3044a8c71589917ba63ea29acbd87a5aa38e6
+  metadata.gz: 3d5e60454dce875711bdae859b3e125f98dfeba6168a5f4c4aa5a1cbb0a2557c
+  data.tar.gz: 119712ab680e5e89409a0174c5d2d4afbef2ba8e8e4e131eef02bc88f0b4b0d4
 SHA512:
-  metadata.gz: 45d12838be8dd5f3e355c6ba058e60f43106d7423397abd712a50eba9d37ba9a726c416c9d0aa63f047e0422e027e643056a1c373e787aa63bcfeca678496af7
-  data.tar.gz: 717566daccb90e6aef8115b6a15c0af26630f3a90065c753ff9a04f5a138929676510d706d557d1a68648dc119bf63babce33c18b1c8e3bb422f1426951ac5c8
+  metadata.gz: ac0e9901966dfc46137561312733e8aae562de83efc5a96ed53e2cb94b32a53b1ded063040a605bd567bfdb5621fdf5151fa56f93663e6e32522bc52802b9ce8
+  data.tar.gz: 41e672cd8fbf173109cc35a4c34f84e35d549fb89c3c7e521cc74d3390fbf98054f0f39374c1542fcbba7995affc055f80906c10b70c2c4e4c2a47626e6f679d

data/set/abstract/items.rb

@@ -43,12 +43,16 @@ def item_ids args={}
   item_names(args).map(&:card_id).compact
 end
 
+def item_cards args={}
+  standard_item_cards args
+end
+
 # @return [Array] list of Card objects
 # @param args [Hash] see #item_names for additional options
 # @option args [String] :complete keyword to use in searching among items
 # @option args [True/False] :known_only if true, return only known cards
 # @option args [String] :type name of type to be used for unknown cards
-def item_cards args={}
+def standard_item_cards args={}
   return item_cards_search(args) if args[:complete]
   return known_item_cards(args) if args[:known_only]
 
@@ -72,7 +76,7 @@ end
 # set card content based on array and save card
 # @param array [Array] list of strings/names (Cardish)
 def items= array
-  items_to_content array
+  self.content = array
   save!
 end
 
@@ -88,8 +92,7 @@ end
 def add_item cardish, allow_duplicates=false
   return if !allow_duplicates && include_item?(cardish)
 
-  items = item_strings << cardish
-  items_to_content items
+  self.content = (item_strings << cardish)
 end
 
 # append item to list and save card
@@ -102,7 +105,7 @@ end
 # @param cardish [String, Card::Name] item to drop
 def drop_item cardish
   drop_item_name = Card::Name[cardish]
-  items_to_content(item_names.reject { |item_name| item_name == drop_item_name })
+  self.content = (item_names.reject { |item_name| item_name == drop_item_name })
 end
 
 # remove item from list and save card
@@ -119,7 +122,7 @@ def insert_item index, name
   new_names = item_names
   new_names.delete name
   new_names.insert index, name
-  items_to_content new_names
+  self.content = new_names
 end
 
 # insert item into list at specified location and save
@@ -152,7 +155,7 @@ end
 # TODO: support type_code and type_id. (currently type)
 # uses name, because its most common use is from CQL
 def item_type
-  opt = options_rule_card
+  opt = options_card
   # FIXME: need better recursion prevention
   return if !opt || opt == self
 

data/set/abstract/paging/paging_links.rb

@@ -2,6 +2,8 @@
 
 # render paging links
 class PagingLinks
+  MAX_PAGES = 100
+
   def initialize total_pages, current_page
     @total = total_pages
     @current = current_page
@@ -44,11 +46,10 @@ class PagingLinks
   end
 
   def right_part
-    [
-      (ellipse if @total > @window_end + 1),
-      (direct_page_link @total if @total > @window_end),
-      next_page_link
-    ].compact
+    parts = [next_page_link]
+    parts.unshift direct_page_link(@total) if @total <= MAX_PAGES && @total > @window_end
+    parts.unshift ellipse if @total > @window_end + 1
+    parts
   end
 
   def previous_page_link

data/set/abstract/paging.rb

@@ -39,20 +39,14 @@ format do
   end
 
   def env_search_param param
-    enforcing_legal_limit param do
-      val = Env.params[param]
-      val.to_i if focal? && val.present?
-    end
-  end
+    val = Env.params[param]
+    return unless focal? && val.present?
 
-  def enforcing_legal_limit param
-    yield.tap do |val|
-      enforce_legal_limit! val if param == :limit
-    end
+    legal_search_param val.to_i
   end
 
-  def enforce_legal_limit! val
-    return if Card::Auth.signed_in? || !val || val <= MAX_ANONYMOUS_SEARCH_PARAM
+  def legal_search_param val
+    return val if Card::Auth.signed_in? || val <= MAX_ANONYMOUS_SEARCH_PARAM
 
     raise Card::Error::PermissionDenied,
           "limit parameter exceeds maximum for anonymous users " \

data/set/all/extended.rb

@@ -18,11 +18,11 @@ private
 
 def extend_item_list items, list, book
   item = items.shift
-  return if already_extended? item, book
+  return if already_extended?(item, book)
 
   if item.collection?
     # keep items in order
-    items.unshift(*item.item_cards)
+    items.unshift(*item.item_cards.compact)
   else  # no further level of items
     list << item
   end

data/set/all/item.rb

@@ -21,13 +21,18 @@ def item_count args={}
   item_names(args).size
 end
 
-def items_to_content array
-  items = array.map { |i| standardize_item i }.reject(&:blank?)
-  self.content = items.to_pointer_content
+def items_content array
+  standardized_items(array).to_pointer_content
+end
+
+def standardized_items array
+  array.map { |i| standardize_item i }.reject(&:blank?)
 end
 
 def standardize_item item
   Card::Name[item]
+rescue Card::Error::NotFound
+  item
 end
 
 def include_item? item
@@ -35,23 +40,19 @@ def include_item? item
 end
 
 def add_item item
-  return if include_item? item
-
-  items_to_content(items_strings << item)
+  self.content = (item_strings << item) unless include_item? item
 end
 
 def drop_item item
   item = Card::Name[item]
-  return unless include_item? item
-
-  items_to_content(item_names.reject { |i| i == item })
+  self.content = (item_names.reject { |i| i == item }) if include_item? item
 end
 
 def insert_item index, name
   new_names = item_names
   new_names.delete name
   new_names.insert index, name
-  items_to_content new_names
+  self.content = new_names
 end
 
 def replace_item old, new

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: card-mod-collection
 version: !ruby/object:Gem::Version
-  version: 0.13.2
+  version: 0.14.1
 platform: ruby
 authors:
 - Ethan McCutchen
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-08-06 00:00:00.000000000 Z
+date: 2022-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: card
@@ -18,14 +18,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.103.2
+        version: 1.104.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.103.2
+        version: 1.104.1
 description: ''
 email:
 - info@decko.org
@@ -64,7 +64,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.2.15
 signing_key:
 specification_version: 4
 summary: collection (list and search)