require 'open3' module Rails module Upgrading class ApplicationChecker def initialize @issues = [] raise NotInRailsAppError unless in_rails_app? end def in_rails_app? File.exist?("config/environment.rb") end # Run all the check methods def run # Ruby 1.8 returns method names as strings whereas 1.9 uses symbols the_methods = (self.public_methods - Object.methods) - [:run, :initialize, "run", "initialize"] the_methods.each {|m| send m } end # Check for deprecated ActiveRecord calls def check_ar_methods files = [] ["find(:all", "find(:first", "find.*:conditions =>", ":joins =>"].each do |v| lines = grep_for(v, "app/") files += extract_filenames(lines) || [] end unless files.empty? alert( "Soon-to-be-deprecated ActiveRecord calls", "Methods such as find(:all), find(:first), finds with conditions, and the :joins option will soon be deprecated.", "http://m.onkey.org/2010/1/22/active-record-query-interface", files ) end lines = grep_for("named_scope", "app/models/") files = extract_filenames(lines) if files alert( "named_scope is now just scope", "The named_scope method has been renamed to just scope.", "http://github.com/rails/rails/commit/d60bb0a9e4be2ac0a9de9a69041a4ddc2e0cc914", files ) end end def check_validation_on_methods files = [] ["validate_on_create", "validate_on_update"].each do |v| lines = grep_for(v, "app/models/") files += extract_filenames(lines) || [] end unless files.empty? alert( "Updated syntax for validate_on_* methods", "Validate-on-callback methods (validate_on_create/validate_on_destroy) have been changed to validate :x, :on => :create", "https://rails.lighthouseapp.com/projects/8994/tickets/3880-validate_on_create-and-validate_on_update-no-longer-seem-to-exist", files ) end end def check_before_validation_on_methods files = [] %w(before_validation_on_create before_validation_on_update).each do |v| lines = grep_for(v, "app/models/") files += extract_filenames(lines) || [] end unless files.empty? alert( "Updated syntax for before_validation_on_* methods", "before_validation_on_* methods have been changed to before_validation(:on => :create/:update) { ... }", "https://rails.lighthouseapp.com/projects/8994/tickets/4699-before_validation_on_create-and-before_validation_on_update-doesnt-exist", files ) end end # Check for deprecated router syntax def check_routes lines = ["map\\.", "ActionController::Routing::Routes", "\\.resources"].map do |v| grep_for(v, "config/routes.rb").empty? ? nil : true end.compact unless lines.empty? alert( "Old router API", "The router API has totally changed.", "http://yehudakatz.com/2009/12/26/the-rails-3-router-rack-it-up/", "config/routes.rb" ) end end # Check for deprecated test_help require def check_test_help files = [] # Hate to duplicate code, but we have to double quote this one... lines = grep_for("\'test_help\'", "test/", true) files += extract_filenames(lines) || [] lines = grep_for("\"test_help\"", "test/") files += extract_filenames(lines) || [] files.uniq! unless files.empty? alert( "Deprecated test_help path", "You now must require 'rails/test_help' not just 'test_help'.", "http://weblog.rubyonrails.org/2009/9/1/gem-packaging-best-practices", files ) end end # Check for old (pre-application.rb) environment.rb file def check_environment unless File.exist?("config/application.rb") alert( "New file needed: config/application.rb", "You need to add a config/application.rb.", "http://omgbloglol.com/post/353978923/the-path-to-rails-3-approaching-the-upgrade", "config/application.rb" ) end lines = grep_for("config.", "config/environment.rb") unless lines.empty? alert( "Old environment.rb", "environment.rb doesn't do what it used to; you'll need to move some of that into application.rb.", "http://omgbloglol.com/post/353978923/the-path-to-rails-3-approaching-the-upgrade", "config/environment.rb" ) end end # Check for deprecated constants def check_deprecated_constants files = [] ["RAILS_ENV", "RAILS_ROOT", "RAILS_DEFAULT_LOGGER"].each do |v| lines = grep_for(v, "app/") files += extract_filenames(lines) || [] lines = grep_for(v, "lib/") files += extract_filenames(lines) || [] end unless files.empty? alert( "Deprecated constant(s)", "Constants like RAILS_ENV, RAILS_ROOT, and RAILS_DEFAULT_LOGGER are now deprecated.", "http://litanyagainstfear.com/blog/2010/02/03/the-rails-module/", files.uniq ) end end # Check for old-style config.gem calls def check_gems lines = grep_for("config.gem ", "config/*.rb") files = extract_filenames(lines) if files alert( "Old gem bundling (config.gems)", "The old way of bundling is gone now. You need a Gemfile for bundler.", "http://omgbloglol.com/post/353978923/the-path-to-rails-3-approaching-the-upgrade", files ) end end # Checks for old mailer syntax in both mailer classes and those # classes utilizing the mailers def check_mailers lines = grep_for("deliver_", "app/models/ #{base_path}app/controllers/ #{base_path}app/observers/") files = extract_filenames(lines) if files alert( "Deprecated ActionMailer API", "You're using the old ActionMailer API to send e-mails in a controller, model, or observer.", "http://lindsaar.net/2010/1/26/new-actionmailer-api-in-rails-3", files ) end files = [] ["recipients ", "attachment(?!s) ", "(?", "app/views/**/*") files = extract_filenames(lines) if files alert( "Deprecated ERb helper calls", "Block helpers that use concat (e.g., form_for) should use <%= instead of <%. The current form will continue to work for now, but you will get deprecation warnings since this form will go away in the future.", "http://weblog.rubyonrails.org/", files ) end end # Checks for old-style AJAX helpers def check_old_ajax_helpers files = [] ['link_to_remote','form_remote_tag','remote_form_for'].each do |type| lines = grep_for(type, "app/views/**/*") inner_files = extract_filenames(lines) files += inner_files unless inner_files.nil? end unless files.empty? alert( "Deprecated AJAX helper calls", "AJAX javascript helpers have been switched to be unobtrusive and use :remote => true instead of having a seperate function to handle remote requests.", "http://www.themodestrubyist.com/2010/02/24/rails-3-ujs-and-csrf-meta-tags/", files ) end end # Checks for old cookie secret settings def check_old_cookie_secret lines = grep_for("ActionController::Base.cookie_verifier_secret = ", "config/**/*") files = extract_filenames(lines) if files alert( "Deprecated cookie secret setting", "Previously, cookie secret was set directly on ActionController::Base; it's now config.secret_token.", "http://lindsaar.net/2010/4/7/rails_3_session_secret_and_session_store", files ) end end def check_old_session_secret lines = grep_for("ActionController::Base.session = {", "config/**/*") files = extract_filenames(lines) if files alert( "Deprecated session secret setting", "Previously, session secret was set directly on ActionController::Base; it's now config.secret_token.", "http://lindsaar.net/2010/4/7/rails_3_session_secret_and_session_store", files ) end end # Checks for old session settings def check_old_session_setting lines = grep_for("ActionController::Base.session_store", "config/**/*") files = extract_filenames(lines) if files alert( "Old session store setting", "Previously, session store was set directly on ActionController::Base; it's now config.session_store :whatever.", "http://lindsaar.net/2010/4/7/rails_3_session_secret_and_session_store", files ) end end private def grep_for_with_perl_regex(text, where = "./", double_quote = false) grep_for(text, where, double_quote, true) end # Find a string in a set of files; calls +find_with_grep+ and +find_with_rak+ # depending on platform. # # TODO: Figure out if this works on Windows. def grep_for(text, where = "./", double_quote = false, perl_regex = false) # If they're on Windows, they probably don't have grep. @probably_has_grep ||= (Config::CONFIG['host_os'].downcase =~ /mswin|windows|mingw/).nil? # protect against double root paths in Rails 3 where.gsub!(Regexp.new(base_path),'') lines = if @probably_has_grep find_with_grep(text, base_path + where, double_quote, perl_regex) else find_with_rak(text, base_path + where, double_quote) end # ignore comments lines.gsub /^(\/[^:]+:)?\s*#.+$/m, "" end # Sets a base path for finding files; mostly for testing def base_path Dir.pwd + "/" end # Use the grep utility to find a string in a set of files def find_with_grep(text, where, double_quote, perl_regex = false) value = "" # Specifically double quote for finding 'test_help' command = if double_quote "grep -r #{"-P" if perl_regex} --exclude=\*.svn\* \"#{text}\" #{where}" else "grep -r #{"-P" if perl_regex} --exclude=\*.svn\* '#{text}' #{where}" end Open3.popen3(command) do |stdin, stdout, stderr| value = stdout.read end value end # Use the rak gem to grep the files (not yet implemented) def find_with_rak(text, where, double_quote) value = "" Open3.popen3("rak --nogroup -l '#{Regexp.escape(text)}' #{where}") do |stdin, stdout, stderr| value = stdout.read end value end # Extract the filenames from the grep output def extract_filenames(output) if @probably_has_grep extract_filenames_from_grep(output) else extract_filenames_from_rak(output) end end def extract_filenames_from_grep(output) return nil if output.empty? output.split("\n").map do |fn| if m = fn.match(/^(.+?):/) m[1] end end.compact.uniq end def extract_filenames_from_rak(output) return nil if output.empty? output.split("\n").uniq end # Terminal colors, borrowed from Thor CLEAR = "\e[0m" BOLD = "\e[1m" RED = "\e[31m" YELLOW = "\e[33m" CYAN = "\e[36m" WHITE = "\e[37m" # Show an upgrade alert to the user def alert(title, text, more_info_url, culprits) if Config::CONFIG['host_os'].downcase =~ /mswin|windows|mingw/ basic_alert(title, text, more_info_url, culprits) else color_alert(title, text, more_info_url, culprits) end end # Show an upgrade alert to the user. If we're on Windows, we can't # use terminal colors, hence this method. def basic_alert(title, text, more_info_url, culprits) puts "** " + title puts text puts "More information: #{more_info_url}" puts puts "The culprits: " Array(culprits).each do |c| puts "\t- #{c}" end puts end # Show a colorful alert to the user def color_alert(title, text, more_info_url, culprits) puts "#{RED}#{BOLD}#{title}#{CLEAR}" puts "#{WHITE}#{text}" puts "#{BOLD}More information:#{CLEAR} #{CYAN}#{more_info_url}" puts puts "#{WHITE}The culprits: " Array(culprits).each do |c| puts "#{YELLOW}\t- #{c}" end ensure puts "#{CLEAR}" end end end end