wize_upgrader 0.0.5 → 0.0.6

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 4a461e0f49e7da1a6767f07166f0f31ffa42fd91
+  data.tar.gz: da0b5c2aa8e764d754f0a6f8be7a01fbae975144
+SHA512:
+  metadata.gz: ca7b3c4231103939815e1b862e72fc23253cdec38ea7c4ea68ff3f8fbb157b8d505550c2b8e35c0ae8dff9011e58c345711689a16fe6049afa8eb4368b383366
+  data.tar.gz: 51fc50c373f1ce3cade4073105ff2137032a670ed03a45281c26c56a5df7e8023be43d7df685367173a61e64f16f1ff9e28f2db7e0ff8ea69c645926983dc91b

data/lib/wize_upgrader.rb

@@ -1,22 +1,22 @@
+# encoding: utf-8
 require 'fileutils'
 require 'active_support/inflector'
-require 'debugger'
 
 module Wize
   class Upgrader
     SOFT_DIR_MAPPINGS = {
-      ".git" => ".git",
-      ".gitignore" => ".gitignore",
-      "script" => "bin",
-      "db" => "db",
-      "lib" => "lib",
-      "spec" => "spec",
-      "vendor" => "vendor"
+      '.git' => '.git',
+      '.gitignore' => '.gitignore',
+      'script' => 'bin',
+      'db' => 'db',
+      'lib' => 'lib',
+      'spec' => 'spec',
+      'vendor' => 'vendor'
     }
     HARD_DIR_MAPPINGS = {
-      "app" => "app",
-      "config/locales" => "config/locales",
-      "config/initializers" => "config/initializers"
+      'app' => 'app',
+      'config/locales' => 'config/locales',
+      'config/initializers' => 'config/initializers'
     }
     COMMON_GEMS = [
       "gem 'jquery-rails'",
@@ -34,10 +34,10 @@ module Wize
       "gem 'rails', '3.2.15'"
     ]
     COMMON_FILES = [
-      "config/application.yml",
-      "config/database.yml",
-      "config/routes.rb",
-      "README.md"
+      'config/application.yml',
+      'config/database.yml',
+      'config/routes.rb',
+      'README.md'
     ]
 
     def initialize(old_app_name)
@@ -46,8 +46,7 @@ module Wize
     end
 
     def upgrade
-      puts "-------   UPGRADING #{ @new_name } FROM RAILS 3.2 to RAILS 4   -------"
-
+      puts "-------UPGRADING #{@new_name} FROM RAILS 3.2 to RAILS 4-------"
       begin
         rename_old
         rails_gen_new
@@ -60,10 +59,8 @@ module Wize
       rescue => e
         puts e.message
         puts e.backtrace
-        puts "something went wrong!"
-      ensure
       end
-      puts "------    DONE UPGRADING! HAVE FUN    ------"
+      puts '------    DONE UPGRADING! HAVE FUN    ------'
     end
 
     def install_rspec
@@ -78,52 +75,64 @@ module Wize
     end
 
     def fix_controllers
-      files_for("#{ @new_name }/app/controllers").each { |c| fix_controller(c) }
+      files_for("#{@new_name}/app/controllers").each { |c| fix_controller(c) }
     end
 
-    def fix_controller(file)
-      puts "fixing #{ file }"
-      model = File.basename(file).gsub("_controller.rb", "").singularize
-      return if attr_accessibles[model].empty?
-
-      columns = attr_accessibles[model]
-      params_block = <<-rb
+    def params_block(model, columns)
+      <<-rb
 
   private
   def #{ model }_params
-    params.require(:#{ model }).permit(#{ columns.join(", ") })
+    params.require(:#{ model }).permit(#{ columns.join(', ') })
   end
 rb
+    end
+
+    def model_from_file_name(file_name)
+      File.basename(file).gsub('_controller.rb', '').singularize
+    end
+
+    def fix_controller(file)
+      model = model_from_file_name(file)
+      return if attr_accessibles[model].empty?
+
+      puts "fixing #{ file }"
+      columns = attr_accessibles[model]
 
       # find last occurance of end and replace with params_block
       ctrlr = File.read("#{ @new_name }/app/controllers/#{ file }")
-      end_idx = ctrlr.rindex("end")
-      puts "last index of end is #{ end_idx }"
-      ctrlr.insert(end_idx, params_block)
+      end_idx = ctrlr.rindex('end')
+      ctrlr.insert(end_idx, params_block(model, columns))
       ctrlr.gsub!("params[:#{ model }]", "#{ model }_params")
 
-      File.open("#{ @new_name }/app/controllers/#{ file }.tmp", "w") do |n|
+      File.open("#{ @new_name }/app/controllers/#{ file }.tmp", 'w') do |n|
         n.write(ctrlr)
       end
-      `mv #{ @new_name }/app/controllers/#{ file }.tmp #{ @new_name }/app/controllers/#{ file }`
+      clean_up_tmp(file)
+    end
+
+    def clean_up_tmp(file)
+      from_file = "#{ @new_name }/app/controllers/#{file}.tmp"
+      to_file = "#{ @new_name }/app/controllers/#{file}"
+      `mv #{from_file} #{to_file}`
     end
 
     def fix_models
-      puts "pulling out attr_accessible from models"
+      puts 'pulling out attr_accessible from models'
       files_for("#{ @new_name }/app/models").each { |m| fix_model(m) }
     end
 
     def fix_model(file)
-      lower_model = file.gsub(".rb", "")
+      lower_model = file.gsub('.rb', '')
       puts "fixing #{ lower_model }"
-      File.open("#{ @new_name }/app/models/#{ file }", "r") do |old|
-        File.open("#{ @new_name }/app/models/#{ file }.tmp", "w") do |n|
+      File.open("#{ @new_name }/app/models/#{ file }", 'r') do |old|
+        File.open("#{ @new_name }/app/models/#{ file }.tmp", 'w') do |n|
           old.each_line do |line|
-            if line.strip.start_with?("attr_accessible")
+            if line.strip.start_with?('attr_accessible')
               attr_accessibles[lower_model] += line
                 .strip
-                .gsub("attr_accessible", "")
-                .split(",")
+                .gsub('attr_accessible', '')
+                .split(',')
                 .map(&:strip)
             else
               n.write(line)
@@ -131,16 +140,16 @@ rb
           end
         end
       end
-      puts "Found attr_accessible #{ attr_accessibles[lower_model].join(", ") }"
-      `mv #{ @new_name }/app/models/#{ file }.tmp #{ @new_name }/app/models/#{ file }`
+      puts "Found attr_accessible #{attr_accessibles[lower_model].join(', ')}"
+      `mv #{@new_name}/app/models/#{file}.tmp #{@new_name}/app/models/#{file}`
     end
 
     def files_for(dir)
       puts "Files for: #{ dir }"
       files = []
       Dir.foreach(dir) do |node|
-        next if node.start_with?(".")
-        files << node if node.end_with?(".rb")
+        next if node.start_with?('.')
+        files << node if node.end_with?('.rb')
         if File.directory?("#{ dir }/#{ node }")
           sub_files = files_for("#{ dir }/#{ node }")
           sub_files.map! { |f| "#{ node }/#{ f }" }
@@ -157,7 +166,7 @@ rb
 
     def upgrade_gemfile(from, to)
       # gsub " for '
-      File.open(to, "a") do |f|
+      File.open(to, 'a') do |f|
         unusual_gems(from).each do |gem|
           f.write(gem)
         end
@@ -173,10 +182,10 @@ end
     def unusual_gems(gemfile)
       old_gems = File.readlines(gemfile)
       old_gems.select! do |gem|
-        gem.strip.start_with?("gem") && !COMMON_GEMS.include?(gem.strip)
+        gem.strip.start_with?('gem') && !COMMON_GEMS.include?(gem.strip)
       end
-      puts "CHECK THE GEM GROUPS!!"
-      puts "Special gems: "
+      puts 'CHECK THE GEM GROUPS!!'
+      puts 'Special gems: '
       old_gems
     end
 
@@ -196,22 +205,22 @@ end
     def app_name
       @app_name ||= begin
         lines = File.readlines("#{ @old_name }/config/application.rb")
-        lines.select! { |l| l.start_with?("module") }
-        lines.first.gsub("module", "").strip
+        lines.select! { |l| l.start_with?('module') }
+        lines.first.gsub('module', '').strip
       end
     end
 
     def copy_common_dirs
       SOFT_DIR_MAPPINGS.each do |src, dest|
-        if Dir.exists?("#{ @new_name }/#{ dest }")
+        if Dir.exist?("#{ @new_name }/#{ dest }")
           `cp -rn #{ @old_name }/#{ src }/* #{ @new_name }/#{ dest }/`
         else
           `cp -r #{ @old_name }/#{ src } #{ @new_name }`
         end
       end
       HARD_DIR_MAPPINGS.each do |src, dest|
-        next if src.include?("wrap_parameters")
-        if Dir.exists?("#{ @new_name }/#{ dest }")
+        next if src.include?('wrap_parameters')
+        if Dir.exist?("#{ @new_name }/#{ dest }")
           `cp -r #{ @old_name }/#{ src }/* #{ @new_name }/#{ dest }/`
         else
           `cp -r #{ @old_name }/#{ src } #{ @new_name }`
@@ -226,4 +235,3 @@ end
     end
   end
 end
-

metadata

@@ -1,8 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wize_upgrader
 version: !ruby/object:Gem::Version
-  version: 0.0.5
-  prerelease: 
+  version: 0.0.6
 platform: ruby
 authors:
 - CJ Avilla
@@ -18,32 +17,30 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/wize_upgrader.rb
 - bin/wize_upgrader
+- lib/wize_upgrader.rb
 homepage: https://github.com/w1zeman1p/wize_upgrader
 licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.25
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Upgrades rails apps from 3.2 to 4
 test_files: []
-has_rdoc: