wagn 1.18.6 → 1.19.0

data/lib/wagn/tasks/cucumber.rake

@@ -7,42 +7,42 @@
 unless ARGV.any? { |a| a =~ /^gems/ } # Don't load anything when running the gems:* tasks
 
   vendored_cucumber_bin = Dir["#{Rails.root}/vendor/{gems,plugins}/cucumber*/bin/cucumber"].first
-$LOAD_PATH.unshift(File.dirname(vendored_cucumber_bin) + '/../lib') unless vendored_cucumber_bin.nil?
+$LOAD_PATH.unshift(File.dirname(vendored_cucumber_bin) + "/../lib") unless vendored_cucumber_bin.nil?
 
 begin
-  require 'cucumber/rake/task'
+  require "cucumber/rake/task"
 
   namespace :cucumber do
-    Cucumber::Rake::Task.new({ ok: 'db:test:prepare' }, 'Run features that should pass') do |t|
+    Cucumber::Rake::Task.new({ ok: "db:test:prepare" }, "Run features that should pass") do |t|
       t.binary = vendored_cucumber_bin # If nil, the gem's binary is used.
       t.fork = true # You may get faster startup if you set this to false
-      t.profile = 'default'
+      t.profile = "default"
     end
 
-    Cucumber::Rake::Task.new({ wip: 'db:test:prepare' }, 'Run features that are being worked on') do |t|
+    Cucumber::Rake::Task.new({ wip: "db:test:prepare" }, "Run features that are being worked on") do |t|
       t.binary = vendored_cucumber_bin
       t.fork = true # You may get faster startup if you set this to false
-      t.profile = 'wip'
+      t.profile = "wip"
     end
 
-    Cucumber::Rake::Task.new({ rerun: 'db:test:prepare' }, 'Record failing features and run only them if any exist') do |t|
+    Cucumber::Rake::Task.new({ rerun: "db:test:prepare" }, "Record failing features and run only them if any exist") do |t|
       t.binary = vendored_cucumber_bin
       t.fork = true # You may get faster startup if you set this to false
-      t.profile = 'rerun'
+      t.profile = "rerun"
     end
 
-    desc 'Run all features'
+    desc "Run all features"
     task all: [:ok, :wip]
 
     task :statsetup do
-      require 'rails/code_statistics'
-      ::STATS_DIRECTORIES << %w(Cucumber\ features features) if File.exist?('features')
-      ::CodeStatistics::TEST_TYPES << 'Cucumber features' if File.exist?('features')
-      ::STATS_DIRECTORIES << %w(Mods mods) if File.exist?('mods') # hack!  should be elsewhere
+      require "rails/code_statistics"
+      ::STATS_DIRECTORIES << %w(Cucumber\ features features) if File.exist?("features")
+      ::CodeStatistics::TEST_TYPES << "Cucumber features" if File.exist?("features")
+      ::STATS_DIRECTORIES << %w(Mods mods) if File.exist?("mods") # hack!  should be elsewhere
     end
   end
-  desc 'Alias for cucumber:ok'
-  task cucumber: 'cucumber:ok'
+  desc "Alias for cucumber:ok"
+  task cucumber: "cucumber:ok"
 
   task default: :cucumber
 
@@ -51,14 +51,14 @@ begin
   end
 
   # In case we don't have ActiveRecord, append a no-op task that we can depend upon.
-  task 'db:test:prepare' do
+  task "db:test:prepare" do
   end
 
-  task stats: 'cucumber:statsetup'
+  task stats: "cucumber:statsetup"
 rescue LoadError
-  desc 'cucumber rake task not available (cucumber not installed)'
+  desc "cucumber rake task not available (cucumber not installed)"
   task :cucumber do
-    abort 'Cucumber rake task is not available. Be sure to install cucumber as a gem or plugin'
+    abort "Cucumber rake task is not available. Be sure to install cucumber as a gem or plugin"
   end
 end
 

data/lib/wagn/tasks/db.rake

@@ -1,4 +1,4 @@
-require 'rake'
+require "rake"
 
 # This code lets us redefine existing Rake tasks, which is extremely
 # handy for modifying existing Rails rake tasks.
@@ -33,22 +33,22 @@ namespace :db do
   namespace :fixtures do
     desc "Load fixtures into the current environment's database.  Load specific fixtures using FIXTURES=x,y"
     task load: :environment do
-      require 'active_record/fixtures'
-      fixture_path = File.join(Cardio.gem_root, 'db', 'seed', 'test', 'fixtures')
+      require "active_record/fixtures"
+      fixture_path = File.join(Cardio.gem_root, "db", "seed", "test", "fixtures")
       ActiveRecord::Base.establish_connection(::Rails.env.to_sym)
-      (ENV['FIXTURES'] ? ENV['FIXTURES'].split(/,/) : Dir.glob(File.join(fixture_path, '*.{yml,csv}'))).each do |fixture_file|
-        ActiveRecord::FixtureSet.create_fixtures(fixture_path, File.basename(fixture_file, '.*'))
+      (ENV["FIXTURES"] ? ENV["FIXTURES"].split(/,/) : Dir.glob(File.join(fixture_path, "*.{yml,csv}"))).each do |fixture_file|
+        ActiveRecord::FixtureSet.create_fixtures(fixture_path, File.basename(fixture_file, ".*"))
       end
     end
   end
 
   namespace :test do
-    desc 'Prepare the test database and load the schema'
+    desc "Prepare the test database and load the schema"
     Rake::Task.redefine_task(prepare: :environment) do
-      if ENV['RELOAD_TEST_DATA'] == 'true' || ENV['RUN_CODE_RUN']
+      if ENV["RELOAD_TEST_DATA"] == "true" || ENV["RUN_CODE_RUN"]
         puts `env RAILS_ENV=test rake wagn:seed`
       else
-        puts 'skipping loading test data.  to force, run `env RELOAD_TEST_DATA=true rake db:test:prepare`'
+        puts "skipping loading test data.  to force, run `env RELOAD_TEST_DATA=true rake db:test:prepare`"
       end
     end
   end

data/lib/wagn/tasks/test.rake

@@ -1,14 +1,14 @@
 TEST_SEED_PATH = File.join(
-  ENV['DECKO_SEED_REPO_PATH'] || [Cardio.gem_root, 'db', 'seed'], 'test'
+  ENV["DECKO_SEED_REPO_PATH"] || [Cardio.gem_root, "db", "seed"], "test"
 )
 
 namespace :test do
   task all: :environment do
-    puts 'This is not yet working; only first invocation takes effect'
-    Rake::Task['test:functionals'].invoke
-    puts 'put 2'
-    Rake::Task['test:functionals'].invoke
-    puts 'put 3'
+    puts "This is not yet working; only first invocation takes effect"
+    Rake::Task["test:functionals"].invoke
+    puts "put 2"
+    Rake::Task["test:functionals"].invoke
+    puts "put 3"
 
     #    Rake::Task['test'].invoke
     #    Rake::Task['spec'].invoke
@@ -17,40 +17,40 @@ namespace :test do
 
   ## FIXME: this generates an "Adminstrator links" card with the wrong reader_id, I have been
   ##  setting it by hand after fixture generation.
-  desc 'recreate test fixtures from fresh db'
+  desc "recreate test fixtures from fresh db"
   task generate_fixtures: :environment do
-    ENV['GENERATE_FIXTURES'] = 'true'
-    raise 'must be test env' unless Rails.env == 'test'
+    ENV["GENERATE_FIXTURES"] = "true"
+    raise "must be test env" unless Rails.env == "test"
 
-    Rake::Task['wagn:reset_cache']
+    Rake::Task["wagn:reset_cache"]
 
-    puts 'reseed test db'
-    Rake::Task['wagn:seed'].invoke
+    puts "reseed test db"
+    Rake::Task["wagn:seed"].invoke
 
-    puts '>>populating test data'
+    puts ">>populating test data"
     puts `rake test:populate_template_database --trace`
 
-    puts '>>extracting to fixtures'
+    puts ">>extracting to fixtures"
     puts `rake test:extract_fixtures --trace`
   end
 
-  desc 'dump current db to test fixtures'
+  desc "dump current db to test fixtures"
   task extract_fixtures: :environment do
-    raise 'must be test env' unless Rails.env == 'test'
-    YAML::ENGINE.yamler = 'syck' if RUBY_VERSION !~ /^(2|1\.9)/
+    raise "must be test env" unless Rails.env == "test"
+    YAML::ENGINE.yamler = "syck" if RUBY_VERSION !~ /^(2|1\.9)/
     # use old engine while we're supporting ruby 1.8.7 because it can't support Psych,
     # which dumps with slashes that syck can't understand (also !!null stuff)
 
-    sql = 'SELECT * FROM %s'
-    tables = %w( cards card_acts card_actions card_changes card_references )
+    sql = "SELECT * FROM %s"
+    tables = %w(cards card_acts card_actions card_changes card_references)
     ActiveRecord::Base.establish_connection
     tables.each do |table_name|
-      i = '000'
-      File.open("#{TEST_SEED_PATH}/fixtures/#{table_name}.yml", 'w') do |file|
+      i = "000"
+      File.open("#{TEST_SEED_PATH}/fixtures/#{table_name}.yml", "w") do |file|
         data = ActiveRecord::Base.connection.select_all(sql % table_name)
         file.write data.inject({}) { |hash, record|
-          record['trash'] = false if record.key? 'trash'
-          record['draft'] = false if record.key? 'draft'
+          record["trash"] = false if record.key? "trash"
+          record["draft"] = false if record.key? "draft"
           hash["#{table_name}_#{i.succ!}"] = record
           hash
         }.to_yaml
@@ -58,9 +58,9 @@ namespace :test do
     end
   end
 
-  desc 'create sample data for testing'
+  desc "create sample data for testing"
   task populate_template_database: :environment do
-    raise 'must be test env' unless Rails.env == 'test'
+    raise "must be test env" unless Rails.env == "test"
     puts "populate test data\n"
     load "#{TEST_SEED_PATH}/seed.rb"
     SharedData.add_test_data

data/lib/wagn/tasks/wagn.rake

@@ -1,14 +1,14 @@
-require 'wagn/application'
+require "wagn/application"
 
 WAGN_SEED_TABLES = %w( cards card_actions card_acts card_changes
                        card_references ).freeze
 WAGN_SEED_PATH = File.join(
-  ENV['DECKO_SEED_REPO_PATH'] || [Cardio.gem_root, 'db', 'seed'], 'new'
+  ENV["DECKO_SEED_REPO_PATH"] || [Cardio.gem_root, "db", "seed"], "new"
 )
 
 def run_card_migration core_or_deck
   prepare_migration
-  verbose = ENV['VERBOSE'] ? ENV['VERBOSE'] == 'true' : true
+  verbose = ENV["VERBOSE"] ? ENV["VERBOSE"] == "true" : true
   Cardio.schema_mode(core_or_deck) do |paths|
     ActiveRecord::Migrator.migrations_paths = paths
     ActiveRecord::Migration.verbose = verbose
@@ -18,7 +18,7 @@ end
 
 def prepare_migration
   Card::Cache.reset_all
-  ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
+  ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
   Card::Cache.reset_all
   Card.config.action_mailer.perform_deliveries = false
   Card.reset_column_information
@@ -28,183 +28,180 @@ def prepare_migration
 end
 
 namespace :wagn do
-  desc 'create a wagn database from scratch, load initial data'
+  desc "create a wagn database from scratch, load initial data"
   task :seed do
-    ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
-    puts 'dropping'
+    ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
+    puts "dropping"
     # FIXME: this should be an option, but should not happen on standard
     # creates!
     begin
-      Rake::Task['db:drop'].invoke
+      Rake::Task["db:drop"].invoke
     rescue
-      puts 'not dropped'
+      puts "not dropped"
     end
 
-    puts 'creating'
-    Rake::Task['db:create'].invoke
+    puts "creating"
+    Rake::Task["db:create"].invoke
 
-    puts 'loading schema'
-    Rake::Task['db:schema:load'].invoke
+    puts "loading schema"
+    Rake::Task["db:schema:load"].invoke
 
-    Rake::Task['wagn:load'].invoke
+    Rake::Task["wagn:load"].invoke
   end
 
-  desc 'clear and load fixtures with existing tables'
+  desc "clear and load fixtures with existing tables"
   task :reseed do
-    ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
+    ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
 
-    Rake::Task['wagn:clear'].invoke
+    Rake::Task["wagn:clear"].invoke
 
-    Rake::Task['wagn:load'].invoke
+    Rake::Task["wagn:load"].invoke
   end
 
-  desc 'empty the card tables'
+  desc "empty the card tables"
   task :clear do
     conn = ActiveRecord::Base.connection
 
-    puts 'delete all data in bootstrap tables'
+    puts "delete all data in bootstrap tables"
     WAGN_SEED_TABLES.each do |table|
       conn.delete "delete from #{table}"
     end
   end
 
-  desc 'Load bootstrap data into database'
+  desc "Load bootstrap data into database"
   task :load do
-    require 'decko/engine'
-    puts 'update card_migrations'
-    Rake::Task['wagn:assume_card_migrations'].invoke
+    require "decko/engine"
+    puts "update card_migrations"
+    Rake::Task["wagn:assume_card_migrations"].invoke
 
-    if Rails.env == 'test' && !ENV['GENERATE_FIXTURES']
-      puts 'loading test fixtures'
-      Rake::Task['db:fixtures:load'].invoke
+    if Rails.env == "test" && !ENV["GENERATE_FIXTURES"]
+      puts "loading test fixtures"
+      Rake::Task["db:fixtures:load"].invoke
     else
-      puts 'loading bootstrap'
-      Rake::Task['wagn:bootstrap:load'].invoke
+      puts "loading bootstrap"
+      Rake::Task["wagn:bootstrap:load"].invoke
     end
 
-    puts 'set symlink for assets'
-    Rake::Task['wagn:update_assets_symlink'].invoke
+    puts "set symlink for assets"
+    Rake::Task["wagn:update_assets_symlink"].invoke
 
-    puts 'reset cache'
-    system 'bundle exec rake wagn:reset_cache' # needs loaded environment
+    puts "reset cache"
+    system "bundle exec rake wagn:reset_cache" # needs loaded environment
   end
 
-  desc 'update wagn gems and database'
+  desc "update wagn gems and database"
   task :update do
-    ENV['NO_RAILS_CACHE'] = 'true'
+    ENV["NO_RAILS_CACHE"] = "true"
     # system 'bundle update'
-    if Wagn.paths['tmp'].existent
-      FileUtils.rm_rf Wagn.paths['tmp'].first, secure: true
+    if Wagn.paths["tmp"].existent
+      FileUtils.rm_rf Wagn.paths["tmp"].first, secure: true
     end
-    Dir.mkdir Wagn.paths['tmp'].first
-    Rake::Task['wagn:migrate'].invoke
+    Dir.mkdir Wagn.paths["tmp"].first
+    Rake::Task["wagn:migrate"].invoke
     # FIXME: remove tmp dir / clear cache
-    puts 'set symlink for assets'
-    Rake::Task['wagn:update_assets_symlink'].invoke
+    puts "set symlink for assets"
+    Rake::Task["wagn:update_assets_symlink"].invoke
   end
 
-  desc 'reset cache'
+  desc "reset cache"
   task reset_cache: :environment do
     Card::Cache.reset_all
   end
 
-  desc 'set symlink for assets'
+  desc "set symlink for assets"
   task :update_assets_symlink do
-    assets_path = File.join(Rails.public_path, 'assets')
+    assets_path = File.join(Rails.public_path, "assets")
     if Rails.root.to_s != Wagn.gem_root && !File.exist?(assets_path)
       FileUtils.rm assets_path if File.symlink? assets_path
-      FileUtils.ln_s(Decko::Engine.paths['gem-assets'].first, assets_path)
+      FileUtils.ln_s(Decko::Engine.paths["gem-assets"].first, assets_path)
     end
   end
 
-  desc 'migrate structure and cards'
+  desc "migrate structure and cards"
   task migrate: :environment do
-    ENV['NO_RAILS_CACHE'] = 'true'
-    ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
+    ENV["NO_RAILS_CACHE"] = "true"
+    ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
 
-    stamp = ENV['STAMP_MIGRATIONS']
+    stamp = ENV["STAMP_MIGRATIONS"]
 
-    puts 'migrating structure'
-    Rake::Task['wagn:migrate:structure'].invoke
-    Rake::Task['wagn:migrate:stamp'].invoke :structure if stamp
+    puts "migrating structure"
+    Rake::Task["wagn:migrate:structure"].invoke
+    Rake::Task["wagn:migrate:stamp"].invoke :structure if stamp
 
-    puts 'migrating core cards'
+    puts "migrating core cards"
     Card::Cache.reset_all
     # not invoke because we don't want to reload environment
-    Rake::Task['wagn:migrate:core_cards'].execute
+    Rake::Task["wagn:migrate:core_cards"].execute
     if stamp
-      Rake::Task['wagn:migrate:stamp'].reenable
-      Rake::Task['wagn:migrate:stamp'].invoke :core_cards
+      Rake::Task["wagn:migrate:stamp"].reenable
+      Rake::Task["wagn:migrate:stamp"].invoke :core_cards
     end
 
-    puts 'migrating deck cards'
+    puts "migrating deck cards"
     # not invoke because we don't want to reload environment
-    Rake::Task['wagn:migrate:deck_cards'].execute
+    Rake::Task["wagn:migrate:deck_cards"].execute
     if stamp
-      Rake::Task['wagn:migrate:stamp'].reenable
-      Rake::Task['wagn:migrate:stamp'].invoke :deck_cards
+      Rake::Task["wagn:migrate:stamp"].reenable
+      Rake::Task["wagn:migrate:stamp"].invoke :deck_cards
     end
 
     Card::Cache.reset_all
   end
 
-  desc 'insert existing card migrations into schema_migrations_cards to avoid re-migrating'
+  desc "insert existing card migrations into schema_migrations_cards to avoid re-migrating"
   task :assume_card_migrations do
-    require 'decko/engine'
+    require "decko/engine"
 
     Cardio.assume_migrated_upto_version :core_cards
   end
 
   namespace :migrate do
-    desc 'migrate cards'
-    task cards: :environment do
-      Rake::Task['wagn:migrate:core_cards'].invoke
-      Rake::Task['wagn:migrate:deck_cards'].execute
-    end
+    desc "migrate cards"
+    task cards: [:core_cards, :deck_cards]
 
-    desc 'migrate structure'
+    desc "migrate structure"
     task structure: :environment do
-      ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
+      ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
       Cardio.schema_mode(:structure) do |paths|
         ActiveRecord::Migrator.migrations_paths = paths
         ActiveRecord::Migrator.migrate paths, version
-        Rake::Task['db:_dump'].invoke   # write schema.rb
+        Rake::Task["db:_dump"].invoke   # write schema.rb
       end
     end
 
-    desc 'migrate core cards'
+    desc "migrate core cards"
     task core_cards: :environment do
-      require 'card/core_migration'
+      require "card/migration/core"
       run_card_migration :core_cards
     end
 
-    desc 'migrate deck cards'
+    desc "migrate deck cards"
     task deck_cards: :environment do
-      require 'card/migration'
+      require "card/migration"
       run_card_migration :deck_cards
     end
 
     desc 'Runs the "up" for a given deck cards migration VERSION.'
     task up: :environment do
-      version = ENV['VERSION'] ? ENV['VERSION'].to_i : nil
-      verbose = ENV['VERBOSE'] ? ENV['VERBOSE'] == 'true' : true
-      raise 'VERSION is required' unless version
+      version = ENV["VERSION"] ? ENV["VERSION"].to_i : nil
+      verbose = ENV["VERBOSE"] ? ENV["VERBOSE"] == "true" : true
+      raise "VERSION is required" unless version
       ActiveRecord::Migration.verbose = verbose
       ActiveRecord::Migrator.run :up, Cardio.migration_paths(:deck_cards),
                                  version
     end
 
     # maybe we should move this to a method?
-    desc 'write the version to a file (not usually called directly)'
+    desc "write the version to a file (not usually called directly)"
     task :stamp, :type do |_t, args|
-      ENV['SCHEMA'] ||= "#{Cardio.gem_root}/db/schema.rb"
+      ENV["SCHEMA"] ||= "#{Cardio.gem_root}/db/schema.rb"
       Cardio.config.action_mailer.perform_deliveries = false
 
       stamp_file = Cardio.schema_stamp_path(args[:type])
 
       Cardio.schema_mode args[:type] do
         version = ActiveRecord::Migrator.current_version
-        if version.to_i > 0 && (file = open(stamp_file, 'w'))
+        if version.to_i > 0 && (file = open(stamp_file, "w"))
           puts ">>  writing version: #{version} to #{stamp_file}"
           file.puts version
         end
@@ -216,7 +213,7 @@ namespace :wagn do
     task rescue_watchers: :environment do
       follower_hash = Hash.new { |h, v| h[v] = [] }
 
-      Card.where('right_id' => 219).each do |watcher_list|
+      Card.where("right_id" => 219).each do |watcher_list|
         watcher_list.include_set_modules
         next unless watcher_list.left
         watching = watcher_list.left.name
@@ -225,16 +222,16 @@ namespace :wagn do
         end
       end
 
-      Card.search(right: { codename: 'following' }).each do |following|
+      Card.search(right: { codename: "following" }).each do |following|
         Card::Auth.as_bot do
-          following.update_attributes! content: ''
+          following.update_attributes! content: ""
         end
       end
 
       follower_hash.each do |user, items|
         next unless (card = Card.fetch(user)) && card.account
         Card::Auth.as(user) do
-          following = card.fetch trait: 'following', new: {}
+          following = card.fetch trait: "following", new: {}
           following.items = items
         end
       end
@@ -242,7 +239,7 @@ namespace :wagn do
   end
 
   namespace :bootstrap do
-    desc 'rid template of unneeded cards, acts, actions, changes, and references'
+    desc "rid template of unneeded cards, acts, actions, changes, and references"
     task clean: :environment do
       Card::Cache.reset_all
       clear_history
@@ -252,28 +249,28 @@ namespace :wagn do
       Card::Cache.reset_all
     end
 
-    desc 'dump db to bootstrap fixtures'
+    desc "dump db to bootstrap fixtures"
     task dump: :environment do
       Card::Cache.reset_all
 
       # FIXME: temporarily taking this out!!
-      Rake::Task['wagn:bootstrap:copy_mod_files'].invoke
+      Rake::Task["wagn:bootstrap:copy_mod_files"].invoke
 
-      YAML::ENGINE.yamler = 'syck' if RUBY_VERSION !~ /^(2|1\.9)/
+      YAML::ENGINE.yamler = "syck" if RUBY_VERSION !~ /^(2|1\.9)/
       # use old engine while we're supporting ruby 1.8.7 because it can't
       # support Psych, which dumps with slashes that syck can't understand
 
       WAGN_SEED_TABLES.each do |table|
-        i = '000'
-        File.open(File.join(WAGN_SEED_PATH, "#{table}.yml"), 'w') do |file|
+        i = "000"
+        File.open(File.join(WAGN_SEED_PATH, "#{table}.yml"), "w") do |file|
           data = ActiveRecord::Base.connection.select_all(
             "select * from #{table}"
           )
           file.write YAML.dump(data.each_with_object({}) do |record, hash|
-            record['trash'] = false if record.key? 'trash'
-            record['draft'] = false if record.key? 'draft'
-            if record.key? 'content'
-              record['content'] = record['content'].gsub(/\u00A0/, ' ')
+            record["trash"] = false if record.key? "trash"
+            record["draft"] = false if record.key? "draft"
+            if record.key? "content"
+              record["content"] = record["content"].gsub(/\u00A0/, " ")
               # sych was handling nonbreaking spaces oddly.
               # would not be needed with psych.
             end
@@ -283,22 +280,22 @@ namespace :wagn do
       end
     end
 
-    desc 'copy files from template database to standard mod and update cards'
+    desc "copy files from template database to standard mod and update cards"
     task copy_mod_files: :environment do
       source_files_dir = "#{Wagn.root}/files"
 
       # mark mod files as mod files
       Card::Auth.as_bot do
-        Card.search(type: %w(in Image File), ne: '').each do |card|
-          if card.mod_file? || card.codename == 'new_file' ||
-             card.codename == 'new_image'
+        Card.search(type: %w(in Image File), ne: "").each do |card|
+          if card.mod_file? || card.codename == "new_file" ||
+             card.codename == "new_image"
             puts "skipping #{card.name}: already in code"
             next
           else
             puts "working on #{card.name}"
           end
 
-          raise 'need codename for file' unless card.codename.present?
+          raise "need codename for file" unless card.codename.present?
 
           files = { original: card.attachment.path }
           card.attachment.versions.each_key do |version|
@@ -307,13 +304,13 @@ namespace :wagn do
 
           # make card a mod file card
           mod_name = if (l = card.left) && l.type_id == Card::SkinID
-                       '06_bootstrap'
+                       "bootstrap"
                      else
-                       '05_standard'
+                       "standard"
                      end
           card.update_column :db_content,
                              card.attachment.db_content(mod: mod_name)
-          card.last_action.change_for(2).first
+          card.last_action.change(:content)
               .update_column :value, card.attachment.db_content(mod: mod_name)
           card.expire
           card = Card.fetch card.name
@@ -327,12 +324,12 @@ namespace :wagn do
       end
     end
 
-    desc 'load bootstrap fixtures into db'
+    desc "load bootstrap fixtures into db"
     task load: :environment do
       # FIXME: shouldn't we be more standard and use seed.rb for this code?
       Rake.application.options.trace = true
       puts "bootstrap load starting #{WAGN_SEED_PATH}"
-      require 'active_record/fixtures'
+      require "active_record/fixtures"
       ActiveRecord::FixtureSet.create_fixtures WAGN_SEED_PATH, WAGN_SEED_TABLES
     end
   end
@@ -348,14 +345,14 @@ end
 
 def delete_unwanted_cards
   Card::Auth.as_bot do
-    if (ignoramus = Card['*ignore'])
+    if (ignoramus = Card["*ignore"])
       ignoramus.item_cards.each(&:delete!)
     end
     Card::Cache.reset_all
     # FIXME: can this be associated with the machine module somehow?
-    %w( machine_input machine_output ).each do |codename|
+    %w(machine_input machine_output).each do |codename|
       Card.search(right: { codename: codename }).each do |card|
-        FileUtils.rm_rf File.join('files', card.id.to_s), secure: true
+        FileUtils.rm_rf File.join("files", card.id.to_s), secure: true
         card.delete!
       end
     end
@@ -367,8 +364,8 @@ def clear_history
   Card::Change.delete_actionless
 
   conn = ActiveRecord::Base.connection
-  conn.execute('truncate card_acts')
-  conn.execute('truncate sessions')
+  conn.execute("truncate card_acts")
+  conn.execute("truncate sessions")
   act = Card::Act.create! actor_id: Card::WagnBotID,
                           card_id: Card::WagnBotID
   Card::Action.find_each do |action|
@@ -377,5 +374,5 @@ def clear_history
 end
 
 def version
-  ENV['VERSION'] ? ENV['VERSION'].to_i : nil
+  ENV["VERSION"] ? ENV["VERSION"].to_i : nil
 end