card 1.101.2 → 1.102.0

data/db/migrate_core_cards/20140317035504_account_requests_to_signups.rb

@@ -27,12 +27,10 @@ class AccountRequestsToSignups < Cardio::Migration::Core
     end
 
     # get rid of old signup card unless there is other data there (most likely +*subject and +*message)
-    unless Card.search(return: :id, left_id: old_signup.id).first
-      old_signup.delete!
-    end
+    old_signup.delete! unless Card.search(return: :id, left_id: old_signup.id).first
 
     # turn captcha off by default on signup
-    rulename = [:signup, :type, :captcha].map { |code| Card[code].name } * "+"
+    rulename = %i[signup type captcha].map { |code| Card[code].name } * "+"
     captcha_rule = Card.fetch rulename, new: {}
     captcha_rule.content = "0"
     captcha_rule.save!

data/db/migrate_core_cards/20140512155840_add_script_cards.rb

@@ -37,8 +37,8 @@ class AddScriptCards < Cardio::Migration::Core
 
     # Permissions for JavaScript and CoffeeScript types
     # ( the same as for CSS and SCSS)
-    %w(JavaScript CoffeeScript).each do |type|
-      [:create, :update, :delete].each do |action|
+    %w[JavaScript CoffeeScript].each do |type|
+      %i[create update delete].each do |action|
         Card.gimme!("#{type}+#{Card[:type].name}+#{Card[action].name}",
                     content: "[[#{Card[:administrator].name}]]")
       end
@@ -48,8 +48,10 @@ class AddScriptCards < Cardio::Migration::Core
     Card.create! name: "*script", codename: :script, type_id: Card::SettingID
     script_set = "*script+#{Card[:right].name}"
     Card.create! name: "#{script_set}+#{Card[:default].name}", type_id: Card::PointerID
-    Card.create! name: "#{script_set}+#{Card[:read].name}",    content: "[[#{Card[:anyone].name}]]"
-    Card.create! name: "#{script_set}+#{Card[:options].name}", content: %( {"type":["in", "JavaScript", "CoffeeScript"] }), type: Card::SearchTypeID
+    Card.create! name: "#{script_set}+#{Card[:read].name}",
+                 content: "[[#{Card[:anyone].name}]]"
+    Card.create! name: "#{script_set}+#{Card[:options].name}",
+                 content: %( {"type":["in", "JavaScript", "CoffeeScript"] }), type: Card::SearchTypeID
     Card.create! name: "#{script_set}+#{Card[:input].name}",   content: "list"
     Card.create! name: "#{script_set}+#{Card[:help].name}",    content:       %{ JavaScript (or CoffeeScript) for card's page. }  # TODO: help link?
 
@@ -57,23 +59,28 @@ class AddScriptCards < Cardio::Migration::Core
     default_rule_ending = "#{Card[:right].name}+#{Card[:default].name}"
     Card.create! name: "*machine output", codename: :machine_output
     Card.create! name: "*machine output+#{default_rule_ending}", type_id: Card::FileID
-    Card.create! name: "*machine output+#{Card[:right].name}+#{Card[:read].name}", content: "_left"
+    Card.create! name: "*machine output+#{Card[:right].name}+#{Card[:read].name}",
+                 content: "_left"
     Card.create! name: "*machine input", codename: :machine_input
     Card.create! name: "*machine input+#{default_rule_ending}", type_id: Card::PointerID
 
     # create default script rule
     card_type = { "js" => "java_script", "coffee" => "coffee_script" }
-    scripts        = %w(jquery tinymce slot     card_menu jquery_helper html5shiv_printshiv)
-    types          = %w(js     js      coffee   js        js            js)
+    scripts = %w[jquery tinymce slot card_menu jquery_helper
+                 html5shiv_printshiv]
+    types = %w[js js coffee js js js]
     # jquery.mobile  (in jquery_helper) must be after card to avoid mobileinit nastiness
     cardnames = scripts.map { |name| "script: #{name.tr('_', ' ')}" }
 
     scripts.each_with_index do |name, index|
-      Card.create! name: cardnames[index], type: card_type[types[index]], codename: "script_#{name}"
+      Card.create! name: cardnames[index], type: card_type[types[index]],
+                   codename: "script_#{name}"
     end
 
     cardnames.pop # html5shiv_printshiv not in default list, only used for IE9 (handled in head.rb)
     Card::Cache.reset_all
-    Card.create! name: "#{Card[:all].name}+*script", content: cardnames.map { |name| "[[#{name}]]" }.join("\n")
+    Card.create! name: "#{Card[:all].name}+*script", content: cardnames.map { |name|
+                                                                "[[#{name}]]"
+                                                              }.join("\n")
   end
 end

data/db/migrate_core_cards/20140629222005_add_email_cards.rb

@@ -3,14 +3,15 @@
 class AddEmailCards < Cardio::Migration::Core
   def up
     # change notification rules
-    %w(create update delete).each do |action|
+    %w[create update delete].each do |action|
       Card.create! name: "*on #{action}", type_code: :setting, codename: "on_#{action}"
-      Card.create! name: "*on #{action}+*right+*help", content: "Configures email to be sent when card is #{action}d."
+      Card.create! name: "*on #{action}+*right+*help",
+                   content: "Configures email to be sent when card is #{action}d."
       Card.create! name: "*on #{action}+*right+*default", type_code: :pointer
     end
 
     # change email address list fields to pointers
-    [:to, :from, :cc, :bcc].each do |field|
+    %i[to from cc bcc].each do |field|
       set = Card[field].fetch(:right, new: {})
       default_rule = set.fetch(:default, new: {})
       default_rule.type_id = Card::PointerID
@@ -27,7 +28,8 @@ class AddEmailCards < Cardio::Migration::Core
     end
 
     # create new cardtype for email templates
-    Card.create! name: "Email template", codename: :email_template, type_id: Card::CardtypeID
+    Card.create! name: "Email template", codename: :email_template,
+                 type_id: Card::CardtypeID
     Card.create! name: "Email template+*type+*structure", content: %(
 {{+#{Card[:from].name} | labeled | link}}
 {{+#{Card[:to].name} | labeled | link}}
@@ -55,21 +57,25 @@ class AddEmailCards < Cardio::Migration::Core
     data = JSON.parse(json)
     data.each do |mail|
       mail = mail.symbolize_keys!
-      Card.create! name: mail[:name], codename: mail[:codename], type_id: Card::EmailTemplateID
-      Card.create! name: "#{mail[:name]}+*html message", content: File.read(File.join(dir, "#{mail[:codename]}.html"))
-      Card.create! name: "#{mail[:name]}+*text message", content: File.read(File.join(dir, "#{mail[:codename]}.txt"))
+      Card.create! name: mail[:name], codename: mail[:codename],
+                   type_id: Card::EmailTemplateID
+      Card.create! name: "#{mail[:name]}+*html message",
+                   content: File.read(File.join(dir, "#{mail[:codename]}.html"))
+      Card.create! name: "#{mail[:name]}+*text message",
+                   content: File.read(File.join(dir, "#{mail[:codename]}.txt"))
       Card.create! name: "#{mail[:name]}+*subject", content: mail[:subject]
     end
 
     # move old hard-coded signup alert email handling to new card-based on_create handling
     Card.create!(
-      name: ([:signup, :type, :on_create].map { |code| Card[code].name } * "+"),
+      name: (%i[signup type on_create].map { |code| Card[code].name } * "+"),
       type_id: Card::PointerID, content: "[[signup alert email]]"
     )
     if request_card = Card[:request]
-      [:to, :from].each do |field|
+      %i[to from].each do |field|
         if old_card = request_card.fetch(field) && !old_card.db_content.blank?
-          Card.create! name: "signup alert email+#{Card[field].name}", content: old_card.db_content
+          Card.create! name: "signup alert email+#{Card[field].name}",
+                       content: old_card.db_content
         end
       end
       request_card.codename = nil
@@ -87,7 +93,8 @@ class AddEmailCards < Cardio::Migration::Core
     wagn_bot = Card[:wagn_bot].account.email.present? ? Card[:wagn_bot].name : nil
     token_emails_from = Card::Rule.global_setting("*invite+*from") || wagn_bot || "_user"
     ["verification email", "password reset email"].each do |token_email_template_name|
-      Card.create! name: "#{token_email_template_name}+#{Card[:from].name}", content: token_emails_from
+      Card.create! name: "#{token_email_template_name}+#{Card[:from].name}",
+                   content: token_emails_from
     end
 
     if invite_card = Card[:invite]
@@ -118,6 +125,7 @@ class AddEmailCards < Cardio::Migration::Core
 
     send = Card[:send]
     return unless send
+
     send.update codename: nil
     send.delete!
   end

data/db/migrate_core_cards/20140725180118_config_card_updates.rb

@@ -2,7 +2,7 @@
 
 class ConfigCardUpdates < Cardio::Migration::Core
   def up
-    raw_json = File.read(data_path "1.13_config_text.json")
+    raw_json = File.read(data_path("1.13_config_text.json"))
     json = JSON.parse raw_json
     Card.merge_list json["card"]["value"], output_file: "tmp/unmerged_config_text.json"
   end

data/db/migrate_core_cards/20141115034214_config_descriptions_etc.rb

@@ -2,7 +2,7 @@
 
 class ConfigDescriptionsEtc < Cardio::Migration::Core
   def up
-    raw_json = File.read(data_path "1.14_config_descriptions_etc.json")
+    raw_json = File.read(data_path("1.14_config_descriptions_etc.json"))
     json = JSON.parse raw_json
     Card.merge_list json["card"]["value"], output_file: "tmp/unmerged_config_text.json"
 

data/db/migrate_core_cards/20141119001955_make_symlinks_relative.rb

@@ -3,7 +3,7 @@
 class MakeSymlinksRelative < Cardio::Migration
   def up
     Card.paths["files"].each do |file_path|
-      files = Dir.glob(File.join file_path, "**", "*")
+      files = Dir.glob(File.join(file_path, "**", "*"))
       symlinks = files.select { |f| File.symlink? f }
       symlinks.each do |symlink|
         base = File.basename(File.readlink(symlink))

data/db/migrate_core_cards/20141204061304_watchers_to_following.rb

@@ -13,6 +13,7 @@ class WatchersToFollowing < Cardio::Migration::Core
         card.include_set_modules
 
         next unless watched = card.left
+
         card.item_names.each do |user_name|
           follower_hash[user_name] << watched.name
         end
@@ -20,6 +21,7 @@ class WatchersToFollowing < Cardio::Migration::Core
 
       follower_hash.each do |user, items|
         next unless (card = Card.fetch(user)) && card.account
+
         following = card.fetch "following",  new: { type_code: :pointer }
         items.each { |item| following.add_item item }
         following.save!

data/db/migrate_core_cards/20141216155251_add_more_following_cards.rb

@@ -5,7 +5,8 @@ class AddMoreFollowingCards < Cardio::Migration::Core
     Card.create! name: "*follow", codename: "follow", type_code: :setting
     Card.create! name: "*follow+*right+*default", type_code: :pointer
     Card.create! name: "*follow+*right+*input", type_code: :pointer, content: "[[radio]]"
-    Card.create! name: "*follow+*right+*help", content: "Get notified about changes", type_code: :phrase
+    Card.create! name: "*follow+*right+*help", content: "Get notified about changes",
+                 type_code: :phrase
 
     Card.create! name: "*followers", codename: "followers"
     Card.create! name: "*follow fields", codename: "follow_fields", type_code: :setting
@@ -18,7 +19,8 @@ class AddMoreFollowingCards < Cardio::Migration::Core
     Card.create! name: "*never", codename: "never"
 
     # default follow rule
-    Card.create! name: "*follow defaults", codename: "follow_defaults", type_code: :pointer
+    Card.create! name: "*follow defaults", codename: "follow_defaults",
+                 type_code: :pointer
     Card.create! name: "*all+*all+*follow", type_code: :pointer, content: "[[*never]]"
   end
 end

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?
@@ -66,7 +66,7 @@ class ImportBootstrapLayout < Cardio::Migration::Core
     # update layouts to have explicit views in nests
     Card.search(type_id: Card::LayoutTypeID) do |lcard|
       lcontent = Card::Content.new lcard.db_content, lcard
-      lcontent.find_chunks(Card::Content::Chunk::Nest).each do |nest|
+      lcontent.find_chunks(:Nest).each do |nest|
         nest.explicit_view =
           nest.options[:nest_name] == "_main" ? "open" : "core"
       end

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"