card 1.101.4 → 1.101.5

data/db/migrate_core_cards/20150202143810_import_bootstrap_layout.rb

@@ -10,7 +10,7 @@ class ImportBootstrapLayout < Cardio::Migration::Core
     end
 
     import_json "bootstrap_layout.json" # , pristine: true, output_file: nil
-    if layout && layout.pristine? && (all = Card[:all])
+    if layout&.pristine? && (all = Card[:all])
       layout_rule_card = all.fetch :layout
       style_rule_card  = all.fetch :style
       if layout_rule_card.pristine? && style_rule_card.pristine?

data/db/migrate_core_cards/20150220134731_following_to_follow_rule.rb

@@ -2,13 +2,12 @@
 
 class FollowingToFollowRule < Cardio::Migration::Core
   def up
-    Card.search(right: { codename: "following" }, left: { type: "user" }).each do |following_card|
+    Card.search(right: { codename: "following" },
+                left: { type: "user" }).each do |following_card|
       user_name = following_card.name.left
       following_card.item_names.each do |followed_set_name|
         set_card = Card.fetch(followed_set_name, new: {})
-        if set_card.type_code != :set
-          set_card = set_card.follow_set_card
-        end
+        set_card = set_card.follow_set_card if set_card.type_code != :set
         rule = Card.fetch set_card.follow_rule_name(user_name), new: { type: "pointer" }
         rule.content = "[[*always]]"
         rule.save!

data/db/migrate_core_cards/20150317162412_bootstrap_themes.rb

@@ -2,10 +2,13 @@
 
 class BootstrapThemes < Cardio::Migration::Core
   def up
-    Card.create! name: "themeless bootstrap skin", type_code: :skin, content: "[[style: bootstrap]]\n[[style: jquery-ui-smoothness]]\n[[style: cards]]\n[[style: right sidebar]]\n[[style: bootstrap cards]]"
-    %w(cerulean cosmo cyborg darkly flatly journal lumen paper readable sandstone simplex slate spacelab superhero united yeti).each do |theme|
+    Card.create! name: "themeless bootstrap skin", type_code: :skin,
+                 content: "[[style: bootstrap]]\n[[style: jquery-ui-smoothness]]\n[[style: cards]]\n[[style: right sidebar]]\n[[style: bootstrap cards]]"
+    %w[cerulean cosmo cyborg darkly flatly journal lumen paper readable sandstone simplex
+       slate spacelab superhero united yeti].each do |theme|
       Card.create! name: "theme: #{theme}", type_code: :css, codename: "theme_#{theme}"
-      Card.create! name: "#{theme} skin", type_code: :skin, codename: "#{theme}_skin", content: "[[themeless bootstrap skin]]\n[[theme: #{theme}]]"
+      Card.create! name: "#{theme} skin", type_code: :skin, codename: "#{theme}_skin",
+                   content: "[[themeless bootstrap skin]]\n[[theme: #{theme}]]"
     end
 
     if credit_card = Card["*credit"]
@@ -30,7 +33,8 @@ class BootstrapThemes < Cardio::Migration::Core
     import_json "skin_images.json"
 
     if sidebar_card = Card["*sidebar"]
-      new_content = sidebar_card.db_content.gsub(/(\*(logo|credit))\|content/, '\1|content_panel')
+      new_content = sidebar_card.db_content.gsub(/(\*(logo|credit))\|content/,
+                                                 '\1|content_panel')
       sidebar_card.update! content: new_content
     end
   end

data/db/migrate_core_cards/20150326205655_bootswatch_themes.rb

@@ -8,20 +8,27 @@ class BootswatchThemes < Cardio::Migration::Core
     bs.update! codename: nil
     bs.delete!
 
-    Card.create! name: "bootswatch shared", type_code: :scss, codename: "bootswatch_shared"
-    Card.create! name: "bootswatch theme+*right+*structure", type_id: Card::ScssID, content: "{{_left+variables}}{{bootswatch shared}}{{_left+style}}"
-    %w(bootstrap_default cerulean cosmo cyborg darkly flatly journal lumen paper readable sandstone simplex slate spacelab superhero united yeti).each do |theme_name|
+    Card.create! name: "bootswatch shared", type_code: :scss,
+                 codename: "bootswatch_shared"
+    Card.create! name: "bootswatch theme+*right+*structure", type_id: Card::ScssID,
+                 content: "{{_left+variables}}{{bootswatch shared}}{{_left+style}}"
+    %w[bootstrap_default cerulean cosmo cyborg darkly flatly journal lumen paper readable
+       sandstone simplex slate spacelab superhero united yeti].each do |theme_name|
       path = data_path "themes/#{theme_name}"
       theme = Card.fetch "#{theme_name} skin"
       if theme
         theme.update! type_id: Card::SkinID, content: "[[themeless bootstrap skin]]\n[[+bootswatch theme]]", subcards: {
-          "+variables" => { type_id: Card::ScssID, content: File.read(File.join path, "_variables.scss") },
-          "+style"     => { type_id: Card::ScssID, content: File.read(File.join path, "_bootswatch.scss") }
+          "+variables" => { type_id: Card::ScssID,
+                            content: File.read(File.join(path, "_variables.scss")) },
+          "+style" => { type_id: Card::ScssID,
+                        content: File.read(File.join(path, "_bootswatch.scss")) }
         }
       else
         Card.create! name: "#{theme_name.sub('_', ' ')} skin", type_id: Card::SkinID, content: "[[themeless bootstrap skin]]\n[[+bootswatch theme]]", subcards: {
-          "+variables" => { type_id: Card::ScssID, content: File.read(File.join path, "_variables.scss") },
-          "+style"     => { type_id: Card::ScssID, content: File.read(File.join path, "_bootswatch.scss") }
+          "+variables" => { type_id: Card::ScssID,
+                            content: File.read(File.join(path, "_variables.scss")) },
+          "+style" => { type_id: Card::ScssID,
+                        content: File.read(File.join(path, "_bootswatch.scss")) }
         }
       end
     end

data/db/migrate_core_cards/20150429090551_search_card_context.rb

@@ -7,13 +7,14 @@ class SearchCardContext < Cardio::Migration::Core
       ["[lr]+", 'l\\1'],
       ["[LR]+", 'L\\1'],
       ["(?=[LR]*[lr]+)(?=[lr]*[LR]+)[lrLR]+", 'l\\1'],   # mix of lowercase and uppercase l's and r's
-      %w(left LL),
-      %w(right LR),
-      %w(self left),
+      %w[left LL],
+      %w[right LR],
+      %w[self left],
       ["",       "left"]
     ]
     Card.search(type_id: ["in", Card::SearchTypeID, Card::SetID]).each do |card|
-      next unless card.name.junction? && card.real?
+      next unless card.name.compound? && card.real?
+
       content = card.db_content
       replace.each do |key, val|
         content.gsub!(/(#{sep})_(#{key})(?=#{sep})/, "\\1_#{val}")
@@ -22,6 +23,7 @@ class SearchCardContext < Cardio::Migration::Core
       card.actions.each do |action|
         next unless (content_change = action.change :db_content)
         next if content_change.new_record?
+
         content = content_change.value
         replace.each do |key, val|
           content.gsub!(/(#{sep})_(#{key})(?=#{sep})/, "\\1_#{val}")

data/db/migrate_core_cards/20150508212032_menu_compatibility.rb

@@ -5,14 +5,16 @@ class MenuCompatibility < Cardio::Migration::Core
     # Add bootswatch shared to old skins so that menu works
     bootswatch_shared = Card[:bootswatch_shared]
     Card.search(type_id: Card::SkinID) do |skin|
-      if skin.item_cards.find { |item_card| item_card.codename == :style_bootstrap_compatible }
+      if skin.item_cards.find do |item_card|
+           item_card.codename == :style_bootstrap_compatible
+         end
         skin.add_item! bootswatch_shared.name
       end
     end
 
     # Delete output files so all the styling and js changes take effect.
     # (this can be removed if/when later migrations update those things directly)
-    [:style, :script].each do |setting|
+    %i[style script].each do |setting|
       Card.search(
         right_id: Card::MachineOutputID,
         left: { right: { codename: setting.to_s } }

data/db/migrate_core_cards/20150510031118_fix_skin_codenames.rb

@@ -4,7 +4,8 @@ class FixSkinCodenames < Cardio::Migration::Core
   def up
     # This is needed because the bootswatch_themes migration removed codenames.
     # They were no longer needed for style handling but are still needed for images
-    %w(bootstrap_default cerulean cosmo cyborg darkly flatly journal lumen paper readable sandstone simplex slate spacelab superhero united yeti).each do |theme_name|
+    %w[bootstrap_default cerulean cosmo cyborg darkly flatly journal lumen paper readable
+       sandstone simplex slate spacelab superhero united yeti].each do |theme_name|
       theme_name = "#{theme_name}_skin"
       Card.fetch(theme_name).update! codename: theme_name
     end

data/db/migrate_core_cards/20150601133433_add_recent_setting_session_card.rb

@@ -9,9 +9,9 @@ class AddRecentSettingSessionCard < Cardio::Migration::Core
       subcards: {
         "+*self+*options" => { type_code: :search_type,
                                content: '{"type":"setting"}' },
-        "+*self+*update"  => { content: "[[Anyone]]" },
-        "+*self**create"  => { content: "[[Anyone]]" },
-        "+*self**read"    => { content: "[[Anyone]]" }
+        "+*self+*update" => { content: "[[Anyone]]" },
+        "+*self**create" => { content: "[[Anyone]]" },
+        "+*self**read" => { content: "[[Anyone]]" }
       }
     )
   end

data/db/migrate_core_cards/20150610171702_add_debugger_session_card.rb

@@ -6,6 +6,7 @@ class AddDebuggerSessionCard < Cardio::Migration::Core
                  subcards: {
                    "+*self+*options" => "[[on]]",
                    "+*self+*input" => "[[checkbox]]",
-                   "+*self+*help" => "show more useful error pages" }
+                   "+*self+*help" => "show more useful error pages"
+                 }
   end
 end

data/db/migrate_core_cards/20150610180019_add_recaptcha_key_and_admin_info_cards.rb

@@ -14,7 +14,7 @@ class AddRecaptchaKeyAndAdminInfoCards < Cardio::Migration::Core
     admin_only name: "*google_analytics_key",
                codename: "google_analytics_key"
 
-    codenames = %w(debugger recaptcha_settings)
+    codenames = %w[debugger recaptcha_settings]
     content =
       codenames.map do |cn|
         "[[#{Card[cn.to_sym].name}]]"
@@ -28,6 +28,7 @@ class AddRecaptchaKeyAndAdminInfoCards < Cardio::Migration::Core
 
   def add_admin_info_to_home_card
     return unless (home = Card[Card[:home].db_content])
+
     new_content = home.db_content.prepend "{{*admin info|content}}\n"
     home.update! content: new_content
   end
@@ -39,7 +40,7 @@ class AddRecaptchaKeyAndAdminInfoCards < Cardio::Migration::Core
                         "[[+private key]]\n" \
                         "[[+proxy]]"
     Card::Cache.reset_all
-    %w(public_key private_key proxy).each do |name|
+    %w[public_key private_key proxy].each do |name|
       Card.create!(
         name: "#{Card[:recaptcha_settings].name}+#{name.tr('_', ' ')}",
         codename: "recaptcha_#{name}"
@@ -49,7 +50,7 @@ class AddRecaptchaKeyAndAdminInfoCards < Cardio::Migration::Core
 
   def admin_only args
     create_or_update! args.reverse_merge(type_id: Card::PhraseID)
-    %w(*read *update *delete).each do |perm|
+    %w[*read *update *delete].each do |perm|
       create_or_update! name: "#{args[:name]}+*self+#{perm}",
                         content: "[[Administrator]]"
     end

data/db/migrate_core_cards/20150724123438_update_file_and_image_cards.rb

@@ -15,6 +15,7 @@ class UpdateFileAndImageCards < Cardio::Migration::Core
     Card.search(type: [:in, "file", "image"]).each do |card|
       update_history card
       next unless card.db_content.present?
+
       update_db_content card
       update_filenames card
     end
@@ -37,9 +38,11 @@ class UpdateFileAndImageCards < Cardio::Migration::Core
   # swap variant and action_id/type_code in file name
   def update_filenames card
     return unless Dir.exist? card.store_dir
+
     symlink_target_hash =
       Dir.entries(card.store_dir).each_with_object({}) do |file, symlink_target|
         next unless (new_filename = get_new_file_name(file))
+
         file_path = File.join(card.store_dir, file)
         if File.symlink?(file_path)
           symlink_target[new_filename] = File.readlink(file_path)
@@ -69,11 +72,12 @@ class UpdateFileAndImageCards < Cardio::Migration::Core
   end
 
   def add_skin_thumbnails
-    %w(cerulean_skin cosmo_skin cyborg_skin darkly_skin flatly_skin
+    %w[cerulean_skin cosmo_skin cyborg_skin darkly_skin flatly_skin
        journal_skin lumen_skin paper_skin readable_skin sandstone_skin
        simplex_skin slate_skin spacelab_skin superhero_skin united_skin
-       yeti_skin).each do |name|
+       yeti_skin].each do |name|
       next unless (card = Card[name.to_sym])
+
       card.update! codename: nil
       if (card = Card.fetch name, :image)
         card.update_column :codename, "#{name}_image"

data/db/migrate_core_cards/20150807205221_create_references_for_search_cards.rb

@@ -6,7 +6,7 @@ class CreateReferencesForSearchCards < Cardio::Migration::Core
       type_id: Card::SearchTypeID
     ).find_each.with_index do |card, index|
       card.update_references_out
-      puts "completed #{index} search cards" if index % 100 == 0
+      puts "completed #{index} search cards" if (index % 100).zero?
     end
   end
 end

data/db/migrate_core_cards/20150824135418_update_file_history.rb

@@ -6,8 +6,10 @@ class UpdateFileHistory < Cardio::Migration::Core
       card.actions.each do |action|
         next unless (content_change = action.change :db_content)
         next if content_change.new_record?
+
         original_filename, file_type, action_id, mod = content_change.value.split("\n")
         next unless file_type.present? && action_id.present?
+
         value =
           if mod.present?
             ":#{card.codename}/#{mod}#{::File.extname(original_filename)}"

data/db/migrate_core_cards/20160804112510_reorganize_scripts.rb

@@ -15,7 +15,7 @@ class ReorganizeScripts < Cardio::Migration::Core
     Card.search(type_id: Card::PointerID,
                 right: { codename: "script" },
                 link_to: "script: slot").each do |script_rule|
-      [:script_tinymce, :script_ace, :bootstrap_js].each do |codename|
+      %i[script_tinymce script_ace bootstrap_js].each do |codename|
         name = Card[codename].name
         script_rule.drop_item name
       end

data/db/migrate_core_cards/20160811115836_rename_stats_to_admin.rb

@@ -3,6 +3,7 @@
 class RenameStatsToAdmin < Cardio::Migration::Core
   def up
     return if Card::Codename.exist?(:admin) || !Card::Codename.exist?(:stats)
+
     Card[:stats].update! name: "*admin", codename: "admin"
   end
 end

data/db/migrate_core_cards/20160914132636_fix_mod_files.rb

@@ -6,6 +6,7 @@ class FixModFiles < Cardio::Migration::Core
       next unless card.coded?
       next unless card.db_content.include?("05_standard") ||
                   card.db_content.include?("06_bootstrap")
+
       new_content = card.db_content.sub("05_standard", "standard")
                         .sub("06_bootstrap", "bootstrap")
       card.update_column :db_content, new_content
@@ -17,6 +18,7 @@ class FixModFiles < Cardio::Migration::Core
   def update_history card
     card.actions.each do |action|
       next unless (content_change = action.change(:db_content))
+
       new_value = content_change.value.gsub("05_standard", "standard")
                                 .gsub("06_bootstrap", "bootstrap")
       content_change.update_column :value, new_value

data/db/migrate_core_cards/20161102202156_tweak_recaptcha_setting_cards.rb

@@ -4,12 +4,12 @@ class TweakRecaptchaSettingCards < Cardio::Migration::Core
   def up
     ensure_card name: "*recaptcha settings", type_id: Card::BasicID
     ensure_card name: "*recaptcha settings+*self+*structure",
-                content: <<-STRING
-{{+public key}}
-{{+private key}}
-{{+proxy}}
-    STRING
-    %w(public_key private_key proxy).each do |name|
+                content: <<~STRING
+                  {{+public key}}
+                  {{+private key}}
+                  {{+proxy}}
+                STRING
+    %w[public_key private_key proxy].each do |name|
       ensure_card name: "#{Card[:recaptcha_settings].name}+#{name.tr('_', ' ')}",
                   type_id: Card::PhraseID
     end

data/db/migrate_core_cards/20161103154836_update_keys.rb

@@ -5,6 +5,7 @@ class UpdateKeys < Cardio::Migration::Core
     Card.pluck(:id, :name, :key).each do |id, name, key|
       new_key = name.to_name.key
       next if new_key == key
+
       update_key id, key, new_key
     end
   end
@@ -34,7 +35,7 @@ class UpdateKeys < Cardio::Migration::Core
       # the living "matthia_tax".
       # The living card is the one that has been updated more recently.
       longer_untouched = [Card.find(id), Card.find_by_key(new_key)]
-                           .min { |a, b| a.updated_at <=> b.updated_at}
+                         .min { |a, b| a.updated_at <=> b.updated_at }
       Card.where(id: longer_untouched).delete_all
       update_key id, key, new_key if longer_untouched != id
     end

data/db/migrate_core_cards/20170209132834_email_test_context.rb

@@ -35,11 +35,13 @@ class EmailTestContext < Cardio::Migration::Core
 
   def update_if_unchanged name, content
     return if (card = Card[name]) && card.updater.codename != :wagn_bot
+
     ensure_card name, content: content
   end
 
   def add_test_context email_template, content
     return unless Card[email_template]
+
     ensure_card [email_template, :test_context],
                 content: content,
                 type_id: Card::PointerID

data/db/migrate_core_cards/20170314175313_add_notification_event_card.rb

@@ -3,16 +3,16 @@
 class AddNotificationEventCard < Cardio::Migration::Core
   def up
     ensure_card "Notification template", codename: "notification_template",
-                type_id: Card::CardtypeID
+                                         type_id: Card::CardtypeID
     ensure_trait "*message", "message", default: { type_id: Card::PhraseID }
     ensure_trait "*disappear", "disappear", default: { type_id: Card::ToggleID }
     ensure_trait "*contextual class", "contextual_class",
                  default: { type_id: Card::PointerID },
                  input: "radio",
                  options: { type_id: Card::PointerID,
-                            content: %w(success info warning danger) }
+                            content: %w[success info warning danger] }
 
-    [:create, :update, :delete].each do |action|
+    %i[create update delete].each do |action|
       update ["on_#{action}".to_sym, :right, :help],
              content: "Configure events to be executed when card id #{action}d"
     end

data/db/migrate_core_cards/20170515101262_add_advanced_edit_cards.rb

@@ -9,7 +9,7 @@ class AddAdvancedEditCards < Cardio::Migration::Core
                 type_id: Card::RoleID
 
     Card::Cache.reset_all
-    ensure_card [:roles, :right, :options],
+    ensure_card %i[roles right options],
                 type_id: Card::SearchTypeID,
                 content: %({"type":"role", "not":{"codename":["in","anyone","anyone_signed_in, anyone_with_role"]}})
 

data/db/migrate_core_cards/20170608200649_add_input_options_codename.rb

@@ -2,7 +2,7 @@
 
 class AddInputOptionsCodename < Cardio::Migration::Core
   def up
-    ensure_card [:input, :right, :options],
+    ensure_card %i[input right options],
                 codename: "input_options"
   end
 end

data/db/migrate_core_cards/20180712042655_head_rule.rb

@@ -3,12 +3,11 @@
 class HeadRule < Cardio::Migration::Core
   def up
     update_card! :head, type_id: Card::SettingID
-    ensure_card [:all, :head],
+    ensure_card %i[all head],
                 type_id: Card::HtmlID,
                 content: "{{*head|core}}"
-    ensure_card [:head, :right, :help],
+    ensure_card %i[head right help],
                 content: "head tag content"
-    ensure_card [:head, :right, :default], type_id: Card::HtmlID
-
+    ensure_card %i[head right default], type_id: Card::HtmlID
   end
 end

data/db/migrate_core_cards/20180905061537_migrate_layouts.rb

@@ -24,12 +24,12 @@ class MigrateLayouts < Cardio::Migration::Core
   end
 
   def body_tag card
-    card.content[/<body[^>]*>.*<\/body>/mi] || add_body_tag(card)
+    card.content[%r{<body[^>]*>.*</body>}mi] || add_body_tag(card)
   end
 
   def add_body_tag card
-    content = card.content.gsub('<!DOCTYPE HTML>', '')
-                          .gsub(/\{\{\*head[^}]*\}\}/i, '').strip
+    content = card.content.gsub("<!DOCTYPE HTML>", "")
+                  .gsub(/\{\{\*head[^}]*\}\}/i, "").strip
     "<body>\n  #{content}\n</body>"
   end
 
@@ -48,7 +48,7 @@ class MigrateLayouts < Cardio::Migration::Core
   end
 
   def head_tag card
-    if (match = card.content.match(/<head>(.*)<\/head>/mi))
+    if (match = card.content.match(%r{<head>(.*)</head>}mi))
       match[1].strip
     end
   end

data/db/migrate_core_cards/20181120200937_update_layout.rb

@@ -4,4 +4,4 @@ class UpdateLayout < Cardio::Migration::Core
   def up
     merge_cards ["*header", "*main_menu"]
   end
-end
+end