card-mod-collection 0.13.3 → 0.14.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d25d5cc2046eb52c82cc76da483084c60d4049ccb4e57d93ad23c6418e4ccf5f
-  data.tar.gz: f242af51ce605508d5448e104e12b4abb34f8a599f8e47454c8842b04d368a19
+  metadata.gz: 24b0a4b03b7c0b961f921db2be11daed010b3965ff1822e5cd4eb26e0639cb25
+  data.tar.gz: 00fed392559e8c1c576454406afab9325a9b12638e30741ba4f64e3bd32ba705
 SHA512:
-  metadata.gz: 0466f45f83d16fd0aab344a2a7402c98c2a54a24194a72cd4ca166090cfd522a4405a5f1090863cea05226d59c0191e89e0f38327b0e2bf2ec86b3d4d867226c
-  data.tar.gz: 07a5efb540ef038c4bae665796993eb8be7a5fa42f2b1dcf7c491e4794d486c62dca9b92ab7dafaae0c86ba1edd497418249eb93b91b4e7335a175a62b6c5aab
+  metadata.gz: ab9ed226b6098e53b8acdf1b9f9655a95f0fdb5c0b4ed57345bae7165becd341d3a2a884921d8eb27fdc6bde1ab2d2474d191961f3eaa1cf0302c361f8bcc134
+  data.tar.gz: 7c1105a83e423a1168c020f42367fcf3fe59cf42ee3b93796cb11bfa0ea39255ec6bd28490b754c6b2bf31ceaec57085dae8c470b2fea596fa6f6fe247ce138d

data/set/abstract/items.rb

@@ -76,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
 
@@ -92,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
@@ -106,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
@@ -123,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
@@ -156,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.3
+  version: 0.14.2
 platform: ruby
 authors:
 - Ethan McCutchen
@@ -10,7 +10,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-24 00:00:00.000000000 Z
+date: 2022-01-08 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.3
+        version: 1.104.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.103.3
+        version: 1.104.2
 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)