Refactored pdf generation with newest prawn.
This commit is contained in:
parent
78cc423a46
commit
eb532babd6
31 changed files with 311 additions and 2118 deletions
3
Gemfile
3
Gemfile
|
@ -19,8 +19,7 @@ gem 'jquery-rails'
|
||||||
|
|
||||||
gem 'mysql2'
|
gem 'mysql2'
|
||||||
gem "fastercsv"
|
gem "fastercsv"
|
||||||
gem "prawn", '<=0.6.3'
|
gem 'prawn'
|
||||||
gem 'prawnto_2', :require => "prawnto" # Used for prawn view templates
|
|
||||||
gem 'haml-rails'
|
gem 'haml-rails'
|
||||||
gem "will_paginate", "~> 3.0.pre2"
|
gem "will_paginate", "~> 3.0.pre2"
|
||||||
gem 'client_side_validations'
|
gem 'client_side_validations'
|
||||||
|
|
27
Gemfile.lock
27
Gemfile.lock
|
@ -7,6 +7,7 @@ GIT
|
||||||
GEM
|
GEM
|
||||||
remote: https://rubygems.org/
|
remote: https://rubygems.org/
|
||||||
specs:
|
specs:
|
||||||
|
Ascii85 (1.0.2)
|
||||||
actionmailer (3.2.8)
|
actionmailer (3.2.8)
|
||||||
actionpack (= 3.2.8)
|
actionpack (= 3.2.8)
|
||||||
mail (~> 2.4.4)
|
mail (~> 2.4.4)
|
||||||
|
@ -64,6 +65,7 @@ GEM
|
||||||
haml (~> 3.1)
|
haml (~> 3.1)
|
||||||
railties (>= 3.1, < 4.1)
|
railties (>= 3.1, < 4.1)
|
||||||
has_scope (0.5.1)
|
has_scope (0.5.1)
|
||||||
|
hashery (2.0.1)
|
||||||
hike (1.2.1)
|
hike (1.2.1)
|
||||||
i18n (0.6.1)
|
i18n (0.6.1)
|
||||||
inherited_resources (1.3.1)
|
inherited_resources (1.3.1)
|
||||||
|
@ -87,22 +89,16 @@ GEM
|
||||||
mime-types (1.19)
|
mime-types (1.19)
|
||||||
multi_json (1.3.6)
|
multi_json (1.3.6)
|
||||||
mysql2 (0.3.11)
|
mysql2 (0.3.11)
|
||||||
|
pdf-reader (1.2.0)
|
||||||
|
Ascii85 (~> 1.0.0)
|
||||||
|
hashery (~> 2.0)
|
||||||
|
ruby-rc4
|
||||||
polyamorous (0.5.0)
|
polyamorous (0.5.0)
|
||||||
activerecord (~> 3.0)
|
activerecord (~> 3.0)
|
||||||
polyglot (0.3.3)
|
polyglot (0.3.3)
|
||||||
prawn (0.6.3)
|
prawn (0.12.0)
|
||||||
prawn-core (>= 0.6.3, < 0.7)
|
pdf-reader (>= 0.9.0)
|
||||||
prawn-format (>= 0.2.3, < 0.3)
|
ttfunk (~> 1.0.2)
|
||||||
prawn-layout (>= 0.3.2, < 0.4)
|
|
||||||
prawn-security (>= 0.1.1, < 0.2)
|
|
||||||
prawn-core (0.6.3)
|
|
||||||
prawn-format (0.2.3)
|
|
||||||
prawn-core
|
|
||||||
prawn-layout (0.3.2)
|
|
||||||
prawn-security (0.1.1)
|
|
||||||
prawnto_2 (0.2.5)
|
|
||||||
prawn (>= 0.6.0)
|
|
||||||
rails (>= 3.1)
|
|
||||||
rack (1.4.1)
|
rack (1.4.1)
|
||||||
rack-cache (1.2)
|
rack-cache (1.2)
|
||||||
rack (>= 0.4)
|
rack (>= 0.4)
|
||||||
|
@ -132,6 +128,7 @@ GEM
|
||||||
json (~> 1.4)
|
json (~> 1.4)
|
||||||
responders (0.9.3)
|
responders (0.9.3)
|
||||||
railties (~> 3.1)
|
railties (~> 3.1)
|
||||||
|
ruby-rc4 (0.1.5)
|
||||||
sass (3.2.1)
|
sass (3.2.1)
|
||||||
sass-rails (3.2.5)
|
sass-rails (3.2.5)
|
||||||
railties (~> 3.2.0)
|
railties (~> 3.2.0)
|
||||||
|
@ -151,6 +148,7 @@ GEM
|
||||||
treetop (1.4.10)
|
treetop (1.4.10)
|
||||||
polyglot
|
polyglot
|
||||||
polyglot (>= 0.3.1)
|
polyglot (>= 0.3.1)
|
||||||
|
ttfunk (1.0.3)
|
||||||
tzinfo (0.3.33)
|
tzinfo (0.3.33)
|
||||||
uglifier (1.3.0)
|
uglifier (1.3.0)
|
||||||
execjs (>= 0.3.0)
|
execjs (>= 0.3.0)
|
||||||
|
@ -176,8 +174,7 @@ DEPENDENCIES
|
||||||
localize_input!
|
localize_input!
|
||||||
meta_search
|
meta_search
|
||||||
mysql2
|
mysql2
|
||||||
prawn (<= 0.6.3)
|
prawn
|
||||||
prawnto_2
|
|
||||||
rails (= 3.2.8)
|
rails (= 3.2.8)
|
||||||
rails3_acts_as_paranoid (~> 0.1.4)
|
rails3_acts_as_paranoid (~> 0.1.4)
|
||||||
sass-rails (~> 3.2.3)
|
sass-rails (~> 3.2.3)
|
||||||
|
|
|
@ -6,10 +6,6 @@ class OrdersController < ApplicationController
|
||||||
|
|
||||||
before_filter :authenticate_orders
|
before_filter :authenticate_orders
|
||||||
|
|
||||||
# Define layout exceptions for PDF actions:
|
|
||||||
layout "application", :except => [:faxPdf, :matrixPdf, :articlesPdf, :groupsPdf]
|
|
||||||
prawnto :prawn => { :page_size => 'A4' }
|
|
||||||
|
|
||||||
# List orders
|
# List orders
|
||||||
def index
|
def index
|
||||||
@open_orders = Order.open
|
@open_orders = Order.open
|
||||||
|
@ -43,13 +39,26 @@ class OrdersController < ApplicationController
|
||||||
def show
|
def show
|
||||||
@order= Order.find(params[:id])
|
@order= Order.find(params[:id])
|
||||||
|
|
||||||
if params[:view] # Articles-list will be replaced
|
respond_to do |format|
|
||||||
@partial = case params[:view]
|
format.html
|
||||||
when 'normal' then "articles"
|
format.js do
|
||||||
when 'groups'then 'shared/articles_by_groups'
|
@partial = case params[:view]
|
||||||
when 'articles'then 'shared/articles_by_articles'
|
when 'normal' then "articles"
|
||||||
end
|
when 'groups'then 'shared/articles_by_groups'
|
||||||
render :layout => false
|
when 'articles'then 'shared/articles_by_articles'
|
||||||
|
else 'articles'
|
||||||
|
end
|
||||||
|
render :layout => false
|
||||||
|
end
|
||||||
|
format.pdf do
|
||||||
|
pdf = case params[:document]
|
||||||
|
when 'groups' then OrderByGroups.new(@order)
|
||||||
|
when 'articles' then OrderByArticles.new(@order)
|
||||||
|
when 'fax' then OrderFax.new(@order)
|
||||||
|
when 'matrix' then OrderMatrix.new(@order)
|
||||||
|
end
|
||||||
|
send_data pdf.to_pdf, filename: pdf.filename, type: 'application/pdf'
|
||||||
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
@ -100,28 +109,6 @@ class OrdersController < ApplicationController
|
||||||
redirect_to order, notice: "Die Bestellung wurde beendet."
|
redirect_to order, notice: "Die Bestellung wurde beendet."
|
||||||
end
|
end
|
||||||
|
|
||||||
# Renders the groups-orderd PDF.
|
|
||||||
def groupsPdf
|
|
||||||
@order = Order.find(params[:id])
|
|
||||||
prawnto :filename => "#{Date.today}_#{@order.name}_GruppenSortierung.pdf"
|
|
||||||
end
|
|
||||||
|
|
||||||
# Renders the articles-orderd PDF.
|
|
||||||
def articlesPdf
|
|
||||||
@order = Order.find(params[:id])
|
|
||||||
prawnto :filename => "#{Date.today}_#{@order.name}_ArtikelSortierung.pdf",
|
|
||||||
:prawn => { :left_margin => 48,
|
|
||||||
:right_margin => 48,
|
|
||||||
:top_margin => 48,
|
|
||||||
:bottom_margin => 48 }
|
|
||||||
end
|
|
||||||
|
|
||||||
# Renders the fax PDF.
|
|
||||||
def faxPdf
|
|
||||||
@order = Order.find(params[:id])
|
|
||||||
prawnto :filename => "#{Date.today}_#{@order.name}_FAX.pdf"
|
|
||||||
end
|
|
||||||
|
|
||||||
# Renders the fax-text-file
|
# Renders the fax-text-file
|
||||||
# e.g. for easier use with online-fax-software, which don't accept pdf-files
|
# e.g. for easier use with online-fax-software, which don't accept pdf-files
|
||||||
def text_fax_template
|
def text_fax_template
|
||||||
|
@ -148,15 +135,4 @@ class OrdersController < ApplicationController
|
||||||
:type => 'text/plain; charset=utf-8; header=present',
|
:type => 'text/plain; charset=utf-8; header=present',
|
||||||
:disposition => "attachment; filename=#{order.name}"
|
:disposition => "attachment; filename=#{order.name}"
|
||||||
end
|
end
|
||||||
|
|
||||||
# Renders the matrix PDF.
|
|
||||||
def matrixPdf
|
|
||||||
@order = Order.find(params[:id])
|
|
||||||
unless @order.order_articles.ordered.empty?
|
|
||||||
prawnto :filename => "#{Date.today}_#{@order.name}_Matrix.pdf"
|
|
||||||
else
|
|
||||||
flash[:error] = "Es sind keine Artikel bestellt worden."
|
|
||||||
redirect_to @order
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
33
app/documents/order_by_articles.rb
Normal file
33
app/documents/order_by_articles.rb
Normal file
|
@ -0,0 +1,33 @@
|
||||||
|
# encoding: utf-8
|
||||||
|
class OrderByArticles < OrderPdf
|
||||||
|
|
||||||
|
def filename
|
||||||
|
"Bestellung #{@order.name}-#{@order.ends.to_date} - Artikelsortierung.pdf"
|
||||||
|
end
|
||||||
|
|
||||||
|
def title
|
||||||
|
"Artikelsortierung der Bestellung: #{@order.name}, beendet am #{@order.ends.strftime('%d.%m.%Y')}"
|
||||||
|
end
|
||||||
|
|
||||||
|
def body
|
||||||
|
@order.order_articles.ordered.each do |order_article|
|
||||||
|
text "#{order_article.article.name} (#{order_article.article.unit} | #{order_article.price.unit_quantity.to_s} | #{number_with_precision(order_article.price.fc_price, precision: 2)})",
|
||||||
|
style: :bold, size: 10
|
||||||
|
rows = []
|
||||||
|
rows << %w(Bestellgruppe Menge Preis)
|
||||||
|
for goa in order_article.group_order_articles
|
||||||
|
rows << [goa.group_order.ordergroup.name,
|
||||||
|
goa.result,
|
||||||
|
number_with_precision(order_article.price.fc_price * goa.result, precision: 2)]
|
||||||
|
end
|
||||||
|
|
||||||
|
table rows, column_widths: [200,40,40], cell_style: {size: 8, overflow: :shrink_to_fit} do |table|
|
||||||
|
table.columns(1..2).align = :right
|
||||||
|
table.cells.border_width = 1
|
||||||
|
table.cells.border_color = '666666'
|
||||||
|
end
|
||||||
|
move_down 10
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
51
app/documents/order_by_groups.rb
Normal file
51
app/documents/order_by_groups.rb
Normal file
|
@ -0,0 +1,51 @@
|
||||||
|
# encoding: utf-8
|
||||||
|
class OrderByGroups < OrderPdf
|
||||||
|
|
||||||
|
def filename
|
||||||
|
"Bestellung #{@order.name}-#{@order.ends.to_date} - Gruppensortierung.pdf"
|
||||||
|
end
|
||||||
|
|
||||||
|
def title
|
||||||
|
"Gruppensortierung der Bestellung: #{@order.name}, beendet am #{@order.ends.strftime('%d.%m.%Y')}"
|
||||||
|
end
|
||||||
|
|
||||||
|
def body
|
||||||
|
# Start rendering
|
||||||
|
@order.group_orders.each do |group_order|
|
||||||
|
text group_order.ordergroup.name, size: 9, style: :bold
|
||||||
|
|
||||||
|
total = 0
|
||||||
|
rows = []
|
||||||
|
rows << %w(Artikel Menge Preis GebGr Einheit Summe) # Table Header
|
||||||
|
|
||||||
|
group_order_articles = group_order.group_order_articles.ordered
|
||||||
|
group_order_articles.each do |goa|
|
||||||
|
price = goa.order_article.price.fc_price
|
||||||
|
sub_total = price * goa.result
|
||||||
|
total += sub_total
|
||||||
|
rows << [goa.order_article.article.name,
|
||||||
|
goa.result,
|
||||||
|
number_with_precision(price, precision: 2),
|
||||||
|
goa.order_article.price.unit_quantity,
|
||||||
|
goa.order_article.article.unit,
|
||||||
|
number_with_precision(sub_total, precision: 2)]
|
||||||
|
end
|
||||||
|
rows << [ "Summe", nil, nil, nil, nil, number_with_precision(total, precision: 2)]
|
||||||
|
|
||||||
|
table rows, column_widths: [250,50,50,50,50,50], cell_style: {size: 8, overflow: :shrink_to_fit} do |table|
|
||||||
|
# borders
|
||||||
|
table.cells.borders = []
|
||||||
|
table.row(0).borders = [:bottom]
|
||||||
|
table.row(group_order_articles.size).borders = [:bottom]
|
||||||
|
table.cells.border_width = 1
|
||||||
|
table.cells.border_color = '666666'
|
||||||
|
|
||||||
|
table.columns(1..3).align = :right
|
||||||
|
table.columns(5).align = :right
|
||||||
|
end
|
||||||
|
|
||||||
|
move_down 15
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
||||||
|
end
|
69
app/documents/order_fax.rb
Normal file
69
app/documents/order_fax.rb
Normal file
|
@ -0,0 +1,69 @@
|
||||||
|
# encoding: utf-8
|
||||||
|
class OrderFax < OrderPdf
|
||||||
|
|
||||||
|
def filename
|
||||||
|
"Bestellung #{@order.name}-#{@order.ends.to_date} - Fax.pdf"
|
||||||
|
end
|
||||||
|
|
||||||
|
def title
|
||||||
|
false
|
||||||
|
end
|
||||||
|
|
||||||
|
def body
|
||||||
|
contact = FoodsoftConfig[:contact].symbolize_keys
|
||||||
|
|
||||||
|
# From paragraph
|
||||||
|
bounding_box [margin_box.right-200,margin_box.top], width: 200 do
|
||||||
|
text FoodsoftConfig[:name], align: :right
|
||||||
|
move_down 5
|
||||||
|
text contact[:street], align: :right
|
||||||
|
move_down 5
|
||||||
|
text "#{contact[:zip_code]} #{contact[:city]}", align: :right
|
||||||
|
move_down 10
|
||||||
|
text contact[:phone], size: 9, align: :right
|
||||||
|
move_down 5
|
||||||
|
text contact[:email], size: 9, align: :right
|
||||||
|
end
|
||||||
|
|
||||||
|
# Recipient
|
||||||
|
bounding_box [margin_box.left,margin_box.top-60], width: 200 do
|
||||||
|
text @order.name
|
||||||
|
move_down 5
|
||||||
|
text @order.supplier.address
|
||||||
|
move_down 5
|
||||||
|
text "Fax: " + @order.supplier.fax
|
||||||
|
end
|
||||||
|
|
||||||
|
text Date.today.strftime('%d.%m.%Y'), align: :right
|
||||||
|
|
||||||
|
move_down 10
|
||||||
|
text "Lieferdatum:"
|
||||||
|
move_down 10
|
||||||
|
text "Ansprechpartner: " + @order.supplier.contact_person
|
||||||
|
move_down 10
|
||||||
|
|
||||||
|
# Articles
|
||||||
|
data = [["BestellNr.", "Menge","Name", "Gebinde", "Einheit","Preis/Einheit"]]
|
||||||
|
data = @order.order_articles.ordered.all(include: :article).collect do |a|
|
||||||
|
[a.article.order_number,
|
||||||
|
a.units_to_order,
|
||||||
|
a.article.name,
|
||||||
|
a.price.unit_quantity,
|
||||||
|
a.article.unit,
|
||||||
|
a.price.price]
|
||||||
|
end
|
||||||
|
table data, cell_style: {size: 8, overflow: :shrink_to_fit} do |table|
|
||||||
|
table.cells.border_width = 1
|
||||||
|
table.cells.border_color = '666666'
|
||||||
|
|
||||||
|
table.columns(1).align = :right
|
||||||
|
table.columns(3..5).align = :right
|
||||||
|
end
|
||||||
|
#font_size: 8,
|
||||||
|
#vertical_padding: 3,
|
||||||
|
#border_style: :grid,
|
||||||
|
#headers: ["BestellNr.", "Menge","Name", "Gebinde", "Einheit","Preis/Einheit"],
|
||||||
|
#align: {0 => :left}
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
86
app/documents/order_matrix.rb
Normal file
86
app/documents/order_matrix.rb
Normal file
|
@ -0,0 +1,86 @@
|
||||||
|
# encoding: utf-8
|
||||||
|
class OrderMatrix < OrderPdf
|
||||||
|
|
||||||
|
MAX_ARTICLES_PER_PAGE = 16 # How many order_articles shoud written on a page
|
||||||
|
|
||||||
|
def filename
|
||||||
|
"Bestellung #{@order.name}-#{@order.ends.to_date} - Sortiermatrix.pdf"
|
||||||
|
end
|
||||||
|
|
||||||
|
def title
|
||||||
|
"Sortiermatrix der Bestellung: #{@order.name}, beendet am #{@order.ends.strftime('%d.%m.%Y')}"
|
||||||
|
end
|
||||||
|
|
||||||
|
def body
|
||||||
|
order_articles = @order.order_articles.ordered
|
||||||
|
|
||||||
|
text "Artikelübersicht", style: :bold
|
||||||
|
move_down 5
|
||||||
|
text "Insgesamt #{order_articles.size} Artikel", size: 8
|
||||||
|
move_down 10
|
||||||
|
|
||||||
|
order_articles_data = [%w(Artikel Einheit Gebinde FC-Preis Menge)]
|
||||||
|
|
||||||
|
order_articles.each do |a|
|
||||||
|
order_articles_data << [a.article.name,
|
||||||
|
a.article.unit,
|
||||||
|
a.price.unit_quantity,
|
||||||
|
number_with_precision(a.price.fc_price, precision: 2),
|
||||||
|
a.units_to_order]
|
||||||
|
end
|
||||||
|
|
||||||
|
table order_articles_data, cell_style: {size: 8, overflow: :shrink_to_fit} do |table|
|
||||||
|
table.cells.border_width = 1
|
||||||
|
table.cells.border_color = '666666'
|
||||||
|
end
|
||||||
|
|
||||||
|
page_number = 0
|
||||||
|
total_num_order_articles = order_articles.size
|
||||||
|
|
||||||
|
while page_number * MAX_ARTICLES_PER_PAGE < total_num_order_articles do # Start page generating
|
||||||
|
|
||||||
|
page_number += 1
|
||||||
|
start_new_page(layout: :landscape)
|
||||||
|
|
||||||
|
# Collect order_articles for this page
|
||||||
|
current_order_articles = order_articles.select do |a|
|
||||||
|
order_articles.index(a) >= (page_number-1) * MAX_ARTICLES_PER_PAGE and
|
||||||
|
order_articles.index(a) < page_number * MAX_ARTICLES_PER_PAGE
|
||||||
|
end
|
||||||
|
|
||||||
|
# Make order_articles header
|
||||||
|
header = [""]
|
||||||
|
for header_article in current_order_articles
|
||||||
|
name = header_article.article.name.gsub(/[-\/]/, " ").gsub(".", ". ")
|
||||||
|
name = name.split.collect { |w| w.truncate(8) }.join(" ")
|
||||||
|
header << name.truncate(30)
|
||||||
|
end
|
||||||
|
|
||||||
|
# Collect group results
|
||||||
|
groups_data = [header]
|
||||||
|
|
||||||
|
@order.group_orders.includes(:ordergroup).all.each do |group_order|
|
||||||
|
|
||||||
|
group_result = [group_order.ordergroup.name.truncate(20)]
|
||||||
|
|
||||||
|
for order_article in current_order_articles
|
||||||
|
# get the Ordergroup result for this order_article
|
||||||
|
goa = order_article.group_order_articles.first conditions: { group_order_id: group_order.id }
|
||||||
|
group_result << ((goa.nil? || goa.result == 0) ? "" : goa.result.to_i)
|
||||||
|
end
|
||||||
|
groups_data << group_result
|
||||||
|
end
|
||||||
|
|
||||||
|
# Make table
|
||||||
|
column_widths = [85]
|
||||||
|
(MAX_ARTICLES_PER_PAGE + 1).times { |i| column_widths << 41 unless i == 0 }
|
||||||
|
table groups_data, column_widths: column_widths, cell_style: {size: 8, overflow: :shrink_to_fit} do |table|
|
||||||
|
table.cells.border_width = 1
|
||||||
|
table.cells.border_color = '666666'
|
||||||
|
table.row_colors = ['ffffff','ececec']
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
|
end
|
|
@ -5,9 +5,9 @@ module OrdersHelper
|
||||||
link_to text, order_path(order, :view => view), :remote => true
|
link_to text, order_path(order, :view => view), :remote => true
|
||||||
end
|
end
|
||||||
|
|
||||||
def link_to_pdf(order, action)
|
def order_pdf(order, document)
|
||||||
link_to image_tag("save_pdf.png", :size => "16x16", :border => "0", :alt => "PDF erstellen"),
|
link_to image_tag("save_pdf.png", :size => "16x16", :border => "0", :alt => "PDF erstellen"),
|
||||||
{ :action => action, :id => order, :format => :pdf }, { :title => "PDF erstellen" }
|
order_path(order, :document => document, :format => :pdf), { :title => "PDF erstellen" }
|
||||||
end
|
end
|
||||||
|
|
||||||
def options_for_suppliers_to_select
|
def options_for_suppliers_to_select
|
||||||
|
|
|
@ -1,36 +0,0 @@
|
||||||
end_date = @order.ends.strftime('%d.%m.%Y')
|
|
||||||
title = "#{@order.name} | beendet am #{end_date}"
|
|
||||||
|
|
||||||
# Define header and footer
|
|
||||||
pdf.header [pdf.margin_box.left,pdf.margin_box.top+30] do
|
|
||||||
pdf.text title, :size => 10, :align => :center
|
|
||||||
end
|
|
||||||
pdf.footer [pdf.margin_box.left, pdf.margin_box.bottom-5] do
|
|
||||||
pdf.stroke_horizontal_rule
|
|
||||||
pdf.text "Seite #{pdf.page_count}", :size => 8
|
|
||||||
end
|
|
||||||
|
|
||||||
# Start rendering
|
|
||||||
|
|
||||||
for order_article in @order.order_articles.ordered
|
|
||||||
pdf.text "#{order_article.article.name} (#{order_article.article.unit} |\
|
|
||||||
#{order_article.price.unit_quantity.to_s} | #{number_to_currency(order_article.price.fc_price)})",
|
|
||||||
:style => :bold, :size => 10
|
|
||||||
pdf.move_down 5
|
|
||||||
data = []
|
|
||||||
for goa in order_article.group_order_articles
|
|
||||||
data << [goa.group_order.ordergroup.name,
|
|
||||||
goa.result,
|
|
||||||
number_with_precision(order_article.price.fc_price * goa.result)]
|
|
||||||
end
|
|
||||||
|
|
||||||
pdf.table data,
|
|
||||||
:font_size => 8,
|
|
||||||
:headers => ["Bestellgruppe", "Menge", "Preis"],
|
|
||||||
:widths => { 0 => 200, 1 => 40, 2 => 40 },
|
|
||||||
:border_style => :grid,
|
|
||||||
:row_colors => ['ffffff','ececec'],
|
|
||||||
:vertical_padding => 3,
|
|
||||||
:align => { 2 => :right }
|
|
||||||
pdf.move_down 10
|
|
||||||
end
|
|
|
@ -1,53 +0,0 @@
|
||||||
# Get ActiveRecord objects
|
|
||||||
contact = FoodsoftConfig[:contact].symbolize_keys
|
|
||||||
|
|
||||||
# Define header and footer
|
|
||||||
#pdf.header [pdf.margin_box.left,pdf.margin_box.top+30] do
|
|
||||||
# pdf.text title, :size => 10, :align => :center
|
|
||||||
#end
|
|
||||||
pdf.footer [pdf.margin_box.left, pdf.margin_box.bottom-5] do
|
|
||||||
pdf.stroke_horizontal_rule
|
|
||||||
pdf.text "Seite #{pdf.page_count}", :size => 8
|
|
||||||
end
|
|
||||||
|
|
||||||
# From paragraph
|
|
||||||
pdf.bounding_box [pdf.margin_box.right-200,pdf.margin_box.top], :width => 200 do
|
|
||||||
pdf.text FoodsoftConfig[:name], :align => :right
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text contact[:street], :align => :right
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text contact[:zip_code] + " " + contact[:city], :align => :right
|
|
||||||
pdf.move_down 10
|
|
||||||
pdf.text contact[:phone], :size => 9, :align => :right
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text contact[:email], :size => 9, :align => :right
|
|
||||||
end
|
|
||||||
|
|
||||||
# Recipient
|
|
||||||
pdf.bounding_box [pdf.margin_box.left,pdf.margin_box.top-60], :width => 200 do
|
|
||||||
pdf.text @order.name
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text @order.supplier.address
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text "Fax: " + @order.supplier.fax
|
|
||||||
end
|
|
||||||
|
|
||||||
pdf.text Date.today.strftime('%d.%m.%Y'), :align => :right
|
|
||||||
|
|
||||||
pdf.move_down 10
|
|
||||||
pdf.text "Lieferdatum:"
|
|
||||||
pdf.move_down 10
|
|
||||||
pdf.text "Ansprechpartner: " + @order.supplier.contact_person
|
|
||||||
pdf.move_down 10
|
|
||||||
|
|
||||||
# Articles
|
|
||||||
data = @order.order_articles.ordered.all(:include => :article).collect do |a|
|
|
||||||
[a.article.order_number, a.units_to_order, a.article.name,
|
|
||||||
a.price.unit_quantity, a.article.unit, a.price.price]
|
|
||||||
end
|
|
||||||
pdf.table data,
|
|
||||||
:font_size => 8,
|
|
||||||
:vertical_padding => 3,
|
|
||||||
:border_style => :grid,
|
|
||||||
:headers => ["BestellNr.", "Menge","Name", "Gebinde", "Einheit","Preis/Einheit"],
|
|
||||||
:align => {0 => :left}
|
|
|
@ -1,48 +0,0 @@
|
||||||
end_date = @order.ends.strftime('%d.%m.%Y')
|
|
||||||
title = "Gruppensortierung für #{@order.name}, beendet am #{end_date}"
|
|
||||||
|
|
||||||
# Define header and footer
|
|
||||||
pdf.header [pdf.margin_box.left,pdf.margin_box.top+20] do
|
|
||||||
pdf.text title, :size => 10, :align => :center
|
|
||||||
end
|
|
||||||
pdf.footer [pdf.margin_box.left, pdf.margin_box.bottom-5] do
|
|
||||||
pdf.stroke_horizontal_rule
|
|
||||||
pdf.text "Seite #{pdf.page_count}", :size => 8
|
|
||||||
end
|
|
||||||
|
|
||||||
|
|
||||||
# Start rendering
|
|
||||||
groups = @order.group_orders.size
|
|
||||||
counter = 1
|
|
||||||
for group_order in @order.group_orders
|
|
||||||
pdf.text group_order.ordergroup.name, :style => :bold
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text "Gruppe #{counter.to_s}/#{groups.to_s}", :size => 8
|
|
||||||
pdf.move_down 5
|
|
||||||
|
|
||||||
total = 0
|
|
||||||
data = []
|
|
||||||
group_order.group_order_articles.ordered.each do |goa|
|
|
||||||
price = goa.order_article.price.fc_price
|
|
||||||
sub_total = price * goa.result
|
|
||||||
total += sub_total
|
|
||||||
data << [goa.order_article.article.name,
|
|
||||||
goa.result, number_with_precision(price),
|
|
||||||
goa.order_article.price.unit_quantity,
|
|
||||||
goa.order_article.article.unit,
|
|
||||||
number_with_precision(sub_total)]
|
|
||||||
end
|
|
||||||
data << [ {:text => "Summe", :colspan => 5}, number_with_precision(total)]
|
|
||||||
|
|
||||||
pdf.table data,
|
|
||||||
:font_size => 8,
|
|
||||||
:border_style => :grid,
|
|
||||||
:vertical_padding => 3,
|
|
||||||
:headers => ["Artikel", "Menge", "Preis", "GebGr", "Einheit", "Summe"],
|
|
||||||
:widths => { 0 => 250 },
|
|
||||||
:row_colors => ['ffffff','ececec'],
|
|
||||||
:align => { 0 => :right, 5 => :right }
|
|
||||||
|
|
||||||
counter += 1
|
|
||||||
pdf.move_down 10
|
|
||||||
end
|
|
|
@ -1,83 +0,0 @@
|
||||||
title = "#{@order.name}, beendet am #{@order.ends.strftime('%d.%m.%Y')}"
|
|
||||||
|
|
||||||
# Define header and footer
|
|
||||||
pdf.header [pdf.margin_box.left,pdf.margin_box.top+20] do
|
|
||||||
pdf.text title, :size => 10, :align => :center
|
|
||||||
end
|
|
||||||
pdf.footer [pdf.margin_box.left, pdf.margin_box.bottom-5] do
|
|
||||||
pdf.stroke_horizontal_rule
|
|
||||||
pdf.move_down 2
|
|
||||||
pdf.text "Seite #{pdf.page_count}", :size => 8
|
|
||||||
end
|
|
||||||
|
|
||||||
max_order_articles_per_page = 16 # How many order_articles shoud written on a page
|
|
||||||
order_articles = @order.order_articles.ordered
|
|
||||||
|
|
||||||
pdf.text "Artikelübersicht", :style => :bold
|
|
||||||
pdf.move_down 5
|
|
||||||
pdf.text "Insgesamt #{order_articles.size} Artikel", :size => 8
|
|
||||||
pdf.move_down 10
|
|
||||||
|
|
||||||
order_articles_data = order_articles.collect do |a|
|
|
||||||
[a.article.name, a.article.unit, a.price.unit_quantity, number_with_precision(a.price.fc_price), a.units_to_order]
|
|
||||||
end
|
|
||||||
pdf.table order_articles_data,
|
|
||||||
:font_size => 8,
|
|
||||||
:border_style => :grid,
|
|
||||||
:vertical_padding => 3,
|
|
||||||
:headers => ["Artikel", "Einheit", "Gebinde", "FC-Preis", "Menge"],
|
|
||||||
:align => { 3 => :right }
|
|
||||||
|
|
||||||
|
|
||||||
page_number = 0
|
|
||||||
total_num_order_articles = order_articles.size
|
|
||||||
|
|
||||||
while (page_number * max_order_articles_per_page < total_num_order_articles) do # Start page generating
|
|
||||||
|
|
||||||
page_number += 1
|
|
||||||
pdf.start_new_page(:layout => :landscape)
|
|
||||||
pdf.header [pdf.margin_box.left,pdf.margin_box.top+20] do
|
|
||||||
pdf.text title, :size => 10, :align => :center
|
|
||||||
end
|
|
||||||
|
|
||||||
# Collect order_articles for this page
|
|
||||||
current_order_articles = order_articles.select do |a|
|
|
||||||
order_articles.index(a) >= (page_number-1) * max_order_articles_per_page and
|
|
||||||
order_articles.index(a) < page_number * max_order_articles_per_page
|
|
||||||
end
|
|
||||||
|
|
||||||
# Make order_articles header
|
|
||||||
header = [""]
|
|
||||||
for header_article in current_order_articles
|
|
||||||
name = header_article.article.name.gsub(/[-\/]/, " ").gsub(".", ". ")
|
|
||||||
name = name.split.collect { |w| truncate(w, :length => 8, :omission => "..") }.join(" ")
|
|
||||||
header << truncate(name, :length => 30, :omission => " ..")
|
|
||||||
end
|
|
||||||
|
|
||||||
# Collect group results
|
|
||||||
groups_data = []
|
|
||||||
for group_order in @order.group_orders.all(:include => :ordergroup)
|
|
||||||
|
|
||||||
group_result = [truncate(group_order.ordergroup.name, :length => 20)]
|
|
||||||
|
|
||||||
for order_article in current_order_articles
|
|
||||||
# get the Ordergroup result for this order_article
|
|
||||||
goa = order_article.group_order_articles.first :conditions => { :group_order_id => group_order.id }
|
|
||||||
group_result << ((goa.nil? || goa.result == 0) ? "" : goa.result.to_i)
|
|
||||||
end
|
|
||||||
groups_data << group_result
|
|
||||||
end
|
|
||||||
|
|
||||||
# Make table
|
|
||||||
widths = {0 => 85} # Generate widths-hash for table layout
|
|
||||||
(max_order_articles_per_page + 1).times { |i| widths.merge!({ i => 41 }) unless i == 0 }
|
|
||||||
logger.debug "Spaltenbreiten: #{widths.inspect}"
|
|
||||||
pdf.table groups_data,
|
|
||||||
:font_size => 8,
|
|
||||||
:border_style => :grid,
|
|
||||||
:vertical_padding => 3,
|
|
||||||
:headers => header,
|
|
||||||
:column_widths => widths,
|
|
||||||
:row_colors => ['ffffff','ececec']
|
|
||||||
|
|
||||||
end
|
|
|
@ -30,7 +30,7 @@
|
||||||
%p
|
%p
|
||||||
Gruppenbestellungen:
|
Gruppenbestellungen:
|
||||||
%b= @order.group_orders.count
|
%b= @order.group_orders.count
|
||||||
= "[#{@order.group_orders.find(:all, :include => :ordergroup).collect{|g| g.ordergroup.name}.join(', ')}]"
|
= "[#{@order.group_orders.includes(:ordergroup).all.collect{|g| g.ordergroup.name}.join(', ')}]"
|
||||||
|
|
||||||
%p
|
%p
|
||||||
Netto/Bruttosumme aller Artikel:
|
Netto/Bruttosumme aller Artikel:
|
||||||
|
@ -56,16 +56,16 @@
|
||||||
= update_articles_link(@order, "Artikelübersicht", :normal)
|
= update_articles_link(@order, "Artikelübersicht", :normal)
|
||||||
|
|
|
|
||||||
= update_articles_link(@order, "Sortiert nach Gruppen", :groups)
|
= update_articles_link(@order, "Sortiert nach Gruppen", :groups)
|
||||||
= link_to_pdf(@order, 'groupsPdf')
|
= order_pdf(@order, :groups)
|
||||||
|
|
|
|
||||||
= update_articles_link(@order, "Sortiert nach Artikeln", :articles)
|
= update_articles_link(@order, "Sortiert nach Artikeln", :articles)
|
||||||
= link_to_pdf(@order, 'articlesPdf')
|
= order_pdf(@order, :articles)
|
||||||
|
|
|
|
||||||
Matrix:
|
Matrix:
|
||||||
= link_to_pdf(@order, 'matrixPdf')
|
= order_pdf(@order, :matrix)
|
||||||
|
|
|
|
||||||
Faxvorlage:
|
Faxvorlage:
|
||||||
= link_to_pdf(@order, 'faxPdf')
|
= order_pdf(@order, :fax)
|
||||||
= link_to image_tag("text_file.png", :size => "16x16", :border => "0", :alt => "Textdatei erstellen"), { :action => 'text_fax_template', :id => @order }, { :title => "Download file" }
|
= link_to image_tag("text_file.png", :size => "16x16", :border => "0", :alt => "Textdatei erstellen"), { :action => 'text_fax_template', :id => @order }, { :title => "Download file" }
|
||||||
|
|
|
|
||||||
= link_to 'Kommentare', '#comments'
|
= link_to 'Kommentare', '#comments'
|
||||||
|
|
32
lib/order_pdf.rb
Normal file
32
lib/order_pdf.rb
Normal file
|
@ -0,0 +1,32 @@
|
||||||
|
require "prawn/measurement_extensions"
|
||||||
|
|
||||||
|
class OrderPdf < Prawn::Document
|
||||||
|
include ActionView::Helpers::NumberHelper
|
||||||
|
|
||||||
|
def initialize(order, options = {})
|
||||||
|
options[:page_size] ||= "A4"
|
||||||
|
#options[:left_margin] ||= 40
|
||||||
|
#options[:right_margin] ||= 40
|
||||||
|
options[:top_margin] ||= 50
|
||||||
|
#options[:bottom_margin] ||= 40
|
||||||
|
super(options)
|
||||||
|
@order = order
|
||||||
|
end
|
||||||
|
|
||||||
|
def to_pdf
|
||||||
|
# Define header
|
||||||
|
repeat :all, dynamic: true do
|
||||||
|
draw_text title, size: 10, style: :bold, at: [bounds.left, bounds.top+20] if title # Header
|
||||||
|
draw_text "Seite #{page_number}", size: 8, at: [bounds.left, bounds.bottom-10] # Footer
|
||||||
|
end
|
||||||
|
|
||||||
|
body # Add content, which is defined in subclasses
|
||||||
|
|
||||||
|
render # Render pdf
|
||||||
|
end
|
||||||
|
|
||||||
|
# Helper method to test pdf via rails console: OrderByGroups.new(order).save_tmp
|
||||||
|
def save_tmp
|
||||||
|
File.open("#{Rails.root}/tmp/#{self.class.to_s.underscore}.pdf", 'w') {|f| f.write(to_pdf.force_encoding("UTF-8")) }
|
||||||
|
end
|
||||||
|
end
|
20
vendor/plugins/rails_upgrade/MIT-LICENSE
vendored
20
vendor/plugins/rails_upgrade/MIT-LICENSE
vendored
|
@ -1,20 +0,0 @@
|
||||||
Copyright (c) 2010 Jeremy McAnally
|
|
||||||
|
|
||||||
Permission is hereby granted, free of charge, to any person obtaining
|
|
||||||
a copy of this software and associated documentation files (the
|
|
||||||
"Software"), to deal in the Software without restriction, including
|
|
||||||
without limitation the rights to use, copy, modify, merge, publish,
|
|
||||||
distribute, sublicense, and/or sell copies of the Software, and to
|
|
||||||
permit persons to whom the Software is furnished to do so, subject to
|
|
||||||
the following conditions:
|
|
||||||
|
|
||||||
The above copyright notice and this permission notice shall be
|
|
||||||
included in all copies or substantial portions of the Software.
|
|
||||||
|
|
||||||
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
|
|
||||||
EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
|
|
||||||
MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
|
|
||||||
NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
|
|
||||||
LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
|
|
||||||
OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
|
|
||||||
WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
|
|
26
vendor/plugins/rails_upgrade/README
vendored
26
vendor/plugins/rails_upgrade/README
vendored
|
@ -1,26 +0,0 @@
|
||||||
= rails-upgrade
|
|
||||||
|
|
||||||
A simple battery of scripts for upgrading Rails app/checking them for required updates. This application should work on Rails 2.x and 3.0, with a focus on upgrading to 3.0.
|
|
||||||
|
|
||||||
== Usage
|
|
||||||
|
|
||||||
You need to install this plugin first:
|
|
||||||
|
|
||||||
script/plugin install git://github.com/rails/rails_upgrade.git
|
|
||||||
|
|
||||||
Then you can run its rake tasks to check your application:
|
|
||||||
|
|
||||||
# Check your app for required upgrades
|
|
||||||
rake rails:upgrade:check
|
|
||||||
|
|
||||||
# Backup your likely modified files that might be overwritten by the generator
|
|
||||||
rake rails:upgrade:backup
|
|
||||||
|
|
||||||
# Generate a new route file
|
|
||||||
rake rails:upgrade:routes
|
|
||||||
|
|
||||||
# Generate a Gemfile from your config.gem directives
|
|
||||||
rake rails:upgrade:gems
|
|
||||||
|
|
||||||
# Generate code for a new config/application.rb from your environment.rb
|
|
||||||
rake rails:upgrade:configuration
|
|
22
vendor/plugins/rails_upgrade/Rakefile
vendored
22
vendor/plugins/rails_upgrade/Rakefile
vendored
|
@ -1,22 +0,0 @@
|
||||||
require 'rake'
|
|
||||||
require 'rake/testtask'
|
|
||||||
require 'rake/rdoctask'
|
|
||||||
|
|
||||||
desc 'Default: run unit tests.'
|
|
||||||
task :default => :test
|
|
||||||
|
|
||||||
Rake::TestTask.new do |t|
|
|
||||||
t.libs << 'lib'
|
|
||||||
t.libs << 'test'
|
|
||||||
t.test_files = FileList['test/*_test.rb']
|
|
||||||
t.verbose = true
|
|
||||||
end
|
|
||||||
|
|
||||||
desc 'Generate documentation for the rails_upgrade plugin.'
|
|
||||||
Rake::RDocTask.new(:rdoc) do |rdoc|
|
|
||||||
rdoc.rdoc_dir = 'rdoc'
|
|
||||||
rdoc.title = 'Rails-upgrade'
|
|
||||||
rdoc.options << '--line-numbers' << '--inline-source'
|
|
||||||
rdoc.rdoc_files.include('README')
|
|
||||||
rdoc.rdoc_files.include('lib/**/*.rb')
|
|
||||||
end
|
|
2
vendor/plugins/rails_upgrade/init.rb
vendored
2
vendor/plugins/rails_upgrade/init.rb
vendored
|
@ -1,2 +0,0 @@
|
||||||
# Get long stack traces for easier debugging; you'll thank me later.
|
|
||||||
Rails.backtrace_cleaner.remove_silencers!
|
|
38
vendor/plugins/rails_upgrade/install.rb
vendored
38
vendor/plugins/rails_upgrade/install.rb
vendored
|
@ -1,38 +0,0 @@
|
||||||
puts "Thanks for installing the Rails upgrade plugin. This is a set of generators and analysis tools to help you upgrade your application to Rails 3. It consists of three tasks...
|
|
||||||
|
|
||||||
To get a feel for what you'll need to change to get your app running, run the application analysis:
|
|
||||||
|
|
||||||
rake rails:upgrade:check
|
|
||||||
|
|
||||||
This should give you an idea of the manual changes that need to be done, but you'll probably want to upgrade some of those automatically. The fastest way to do this is to run 'rails .', which will simply generate a new app on top of your existing code. But this generation also has the effect of replacing some existing files, some of which you might not want to replace. To back those up, first run:
|
|
||||||
|
|
||||||
rake rails:upgrade:backup
|
|
||||||
|
|
||||||
That will backup files you've probably edited that will be replaced in the upgrade; if you finish the upgrade and find that you don't need the old copies, just delete them. Otherwise, copy their contents back into the new files or run one of the following upgraders...
|
|
||||||
|
|
||||||
Routes upgrader
|
|
||||||
===============
|
|
||||||
|
|
||||||
To generate a new routes file from your existing routes file, simply run the following Rake task:
|
|
||||||
|
|
||||||
rake rails:upgrade:routes
|
|
||||||
|
|
||||||
This will output a new routes file that you can copy and paste or pipe into a new, Rails 3 compatible config/routes.rb.
|
|
||||||
|
|
||||||
Gemfile generator
|
|
||||||
=================
|
|
||||||
|
|
||||||
Creating a new Gemfile is as simple as running:
|
|
||||||
|
|
||||||
rake rails:upgrade:gems
|
|
||||||
|
|
||||||
This task will extract your config.gem calls and generate code you can put into a bundler compatible Gemfile.
|
|
||||||
|
|
||||||
Configuration generator
|
|
||||||
=======================
|
|
||||||
|
|
||||||
Much of the configuration information that lived in environment.rb now belongs in a new file named config/application.rb; use the following task to generate code you can put into config/application.rb from your existing config/environment.rb:
|
|
||||||
|
|
||||||
rake rails:upgrade:configuration
|
|
||||||
|
|
||||||
"
|
|
|
@ -1,477 +0,0 @@
|
||||||
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)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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")
|
|
||||||
lines += grep_for("config.gem ", "config/**/*.rb")
|
|
||||||
files = extract_filenames(lines)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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) ", "(?<!:)subject ", "(?<!:)from "].each do |v|
|
|
||||||
lines = grep_for_with_perl_regex(v, "app/models/")
|
|
||||||
files += extract_filenames(lines) || []
|
|
||||||
end
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
alert(
|
|
||||||
"Old ActionMailer class API",
|
|
||||||
"You're using the old API in a mailer class.",
|
|
||||||
"http://lindsaar.net/2010/1/26/new-actionmailer-api-in-rails-3",
|
|
||||||
files
|
|
||||||
)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
# Checks for old-style generators
|
|
||||||
def check_generators
|
|
||||||
generators = Dir.glob(base_path + "vendor/plugins/**/generators/**/")
|
|
||||||
|
|
||||||
unless generators.empty?
|
|
||||||
files = generators.reject do |g|
|
|
||||||
grep_for("def manifest", g).empty?
|
|
||||||
end.compact
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
alert(
|
|
||||||
"Old Rails generator API",
|
|
||||||
"A plugin in the app is using the old generator API (a new one may be available at http://github.com/trydionel/rails3-generators).",
|
|
||||||
"http://blog.plataformatec.com.br/2010/01/discovering-rails-3-generators/",
|
|
||||||
files
|
|
||||||
)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
# Checks a list of known broken plugins and gems
|
|
||||||
def check_plugins
|
|
||||||
# This list is off the wiki; will need to be updated often, esp. since RSpec is working on it
|
|
||||||
bad_plugins = ["rspec", "rspec-rails", "hoptoad", "authlogic", "nifty-generators",
|
|
||||||
"restful_authentication", "searchlogic", "cucumber", "cucumber-rails", "devise",
|
|
||||||
"inherited_resources"]
|
|
||||||
|
|
||||||
bad_plugins = bad_plugins.map do |p|
|
|
||||||
p if File.exist?("#{base_path}vendor/plugins/#{p}") || !Dir.glob("#{base_path}vendor/gems/#{p}-*").empty?
|
|
||||||
end.compact
|
|
||||||
|
|
||||||
unless bad_plugins.empty?
|
|
||||||
alert(
|
|
||||||
"Known broken plugins",
|
|
||||||
"At least one plugin in your app is broken (according to the wiki). Most of project maintainers are rapidly working towards compatability, but do be aware you may encounter issues.",
|
|
||||||
"http://wiki.rubyonrails.org/rails/version3/plugins_and_gems",
|
|
||||||
bad_plugins
|
|
||||||
)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
# Checks for old-style ERb helpers
|
|
||||||
def check_old_helpers
|
|
||||||
|
|
||||||
lines = grep_for("<% .*content_tag.* do.*%>", "app/views/**/*")
|
|
||||||
lines += grep_for("<% .*javascript_tag.* do.*%>", "app/views/**/*")
|
|
||||||
lines += grep_for("<% .*form_for.* do.*%>", "app/views/**/*")
|
|
||||||
lines += grep_for("<% .*form_tag.* do.*%>", "app/views/**/*")
|
|
||||||
lines += grep_for("<% .*fields_for.* do.*%>", "app/views/**/*")
|
|
||||||
lines += grep_for("<% .*field_set_tag.* do.*%>", "app/views/**/*")
|
|
||||||
|
|
||||||
files = extract_filenames(lines)
|
|
||||||
|
|
||||||
if !files.blank?
|
|
||||||
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)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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)
|
|
||||||
|
|
||||||
unless files.empty?
|
|
||||||
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 -rH #{"-P" if perl_regex} \"#{text}\" #{where} | grep -v \.svn"
|
|
||||||
else
|
|
||||||
"grep -rH #{"-P" if perl_regex} '#{text}' #{where} | grep -v \.svn"
|
|
||||||
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
|
|
||||||
filenames = extract_filenames_from_grep(output)
|
|
||||||
else
|
|
||||||
filenames = extract_filenames_from_rak(output)
|
|
||||||
end
|
|
||||||
|
|
||||||
filenames.compact.map do |f|
|
|
||||||
f.gsub(base_path, "")
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def extract_filenames_from_grep(output)
|
|
||||||
return [] 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 [] 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
|
|
|
@ -1,95 +0,0 @@
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class GemfileGenerator
|
|
||||||
def generate_new_gemfile
|
|
||||||
if has_environment?
|
|
||||||
generate_gemfile
|
|
||||||
else
|
|
||||||
raise FileNotFoundError, "Can't find environment.rb [config/environment.rb]!"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def has_environment?
|
|
||||||
File.exists?("config/environment.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def environment_code
|
|
||||||
File.open("config/environment.rb").read
|
|
||||||
end
|
|
||||||
|
|
||||||
def generate_gemfile
|
|
||||||
environment_file = environment_code
|
|
||||||
|
|
||||||
# Get each line that starts with config.gem
|
|
||||||
gem_lines = environment_file.split("\n").select {|l| l =~ /^\s*config\.gem/}
|
|
||||||
|
|
||||||
# Toss those lines to a generator class; the lines are evaluated in the
|
|
||||||
# context of that instance.
|
|
||||||
config = GemfileGenerator.new
|
|
||||||
config.instance_eval(gem_lines.join("\n"))
|
|
||||||
|
|
||||||
config.output
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class GemfileGenerator
|
|
||||||
# Creates a target for the config.gem calls
|
|
||||||
def config
|
|
||||||
self
|
|
||||||
end
|
|
||||||
|
|
||||||
def initialize
|
|
||||||
@gems = []
|
|
||||||
end
|
|
||||||
|
|
||||||
# Receive a call to add a gem to the list
|
|
||||||
def gem(name, options={})
|
|
||||||
data = {}
|
|
||||||
|
|
||||||
# Add new keys from old keys
|
|
||||||
data[:require] = options[:lib] if options[:lib]
|
|
||||||
data[:source] = options[:source] if options[:source]
|
|
||||||
|
|
||||||
version = options[:version]
|
|
||||||
@gems << [name, version, data]
|
|
||||||
end
|
|
||||||
|
|
||||||
# Generate the Gemfile output
|
|
||||||
def output
|
|
||||||
# Generic preamble, taken from Yehuda Katz's blog
|
|
||||||
preamble = <<STR
|
|
||||||
# Edit this Gemfile to bundle your application's dependencies.
|
|
||||||
# This preamble is the current preamble for Rails 3 apps; edit as needed.
|
|
||||||
source 'http://rubygems.org'
|
|
||||||
|
|
||||||
gem 'rails', '3.0.6'
|
|
||||||
|
|
||||||
STR
|
|
||||||
preamble + generate_upgraded_code
|
|
||||||
end
|
|
||||||
|
|
||||||
# Get Gemfile call for all the gems
|
|
||||||
def generate_upgraded_code
|
|
||||||
code = @gems.map do |name, version, data|
|
|
||||||
version_string = (version ? "'#{version}'" : nil)
|
|
||||||
source = data.delete(:source)
|
|
||||||
|
|
||||||
data_string = nil
|
|
||||||
unless data.empty?
|
|
||||||
data_string = data.to_a.map {|k, v| ":#{k} => '#{v}'"}.join(", ")
|
|
||||||
end
|
|
||||||
|
|
||||||
# If we have a source, generate a call to +source+ then output the
|
|
||||||
# gem call. Otherwise, just generate the gem requirement.
|
|
||||||
if source
|
|
||||||
str = ["'#{name}'", version_string, data_string].compact.join(", ")
|
|
||||||
"source '#{source}'\ngem #{str}"
|
|
||||||
else
|
|
||||||
str = ["'#{name}'", version_string, data_string].compact.join(", ")
|
|
||||||
"gem #{str}"
|
|
||||||
end
|
|
||||||
end.join("\n")
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,51 +0,0 @@
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class NewConfigurationGenerator
|
|
||||||
def generate_new_configurations
|
|
||||||
if has_environment?
|
|
||||||
generate_new_application_rb
|
|
||||||
else
|
|
||||||
raise FileNotFoundError, "Can't find environment.rb [config/environment.rb]!"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def has_environment?
|
|
||||||
File.exists?("config/environment.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def environment_code
|
|
||||||
File.open("config/environment.rb").read
|
|
||||||
end
|
|
||||||
|
|
||||||
def generate_new_application_rb
|
|
||||||
environment_file = environment_code
|
|
||||||
|
|
||||||
initializer_code = ""
|
|
||||||
if matches = environment_file.match(/Rails\:\:Initializer\.run do \|config\|\n(.*)\nend/m)
|
|
||||||
initializer_code = matches[1]
|
|
||||||
else
|
|
||||||
raise "There doesn't seem to be a real environment.rb in your app. Are you sure config/environment.rb has the right contents?"
|
|
||||||
end
|
|
||||||
|
|
||||||
frame = "# Put this in config/application.rb
|
|
||||||
require File.expand_path('../boot', __FILE__)
|
|
||||||
|
|
||||||
module #{app_name.classify}
|
|
||||||
class Application < Rails::Application
|
|
||||||
%s
|
|
||||||
end
|
|
||||||
end"
|
|
||||||
|
|
||||||
frame % [indent(initializer_code)]
|
|
||||||
end
|
|
||||||
|
|
||||||
def indent(text)
|
|
||||||
text.split("\n").map {|l| " #{l}"}.join("\n")
|
|
||||||
end
|
|
||||||
|
|
||||||
def app_name
|
|
||||||
File.basename(Dir.pwd)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
366
vendor/plugins/rails_upgrade/lib/routes_upgrader.rb
vendored
366
vendor/plugins/rails_upgrade/lib/routes_upgrader.rb
vendored
|
@ -1,366 +0,0 @@
|
||||||
# TODO: Fix formatting on member/collection methods
|
|
||||||
|
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
module FakeRouter
|
|
||||||
module ActionController
|
|
||||||
module Routing
|
|
||||||
class Routes
|
|
||||||
def self.setup
|
|
||||||
@redrawer = Rails::Upgrading::RouteRedrawer.new
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.redrawer
|
|
||||||
@redrawer
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.draw
|
|
||||||
yield @redrawer
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RoutesUpgrader
|
|
||||||
def generate_new_routes
|
|
||||||
if has_routes_file?
|
|
||||||
upgrade_routes
|
|
||||||
else
|
|
||||||
raise FileNotFoundError, "Can't find your routes file [config/routes.rb]!"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def has_routes_file?
|
|
||||||
File.exists?("config/routes.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def routes_code
|
|
||||||
File.read("config/routes.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def upgrade_routes
|
|
||||||
FakeRouter::ActionController::Routing::Routes.setup
|
|
||||||
|
|
||||||
# Read and eval the file; our fake route mapper will capture
|
|
||||||
# the calls to draw routes and generate new route code
|
|
||||||
FakeRouter.module_eval(routes_code)
|
|
||||||
|
|
||||||
# Give the route set to the code generator and get its output
|
|
||||||
generator = RouteGenerator.new(FakeRouter::ActionController::Routing::Routes.redrawer.routes)
|
|
||||||
generator.generate
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RouteRedrawer
|
|
||||||
attr_accessor :routes
|
|
||||||
|
|
||||||
def self.stack
|
|
||||||
@stack
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.stack=(val)
|
|
||||||
@stack = val
|
|
||||||
end
|
|
||||||
|
|
||||||
def initialize
|
|
||||||
@routes = []
|
|
||||||
|
|
||||||
# The old default route was actually two routes; we generate the new style
|
|
||||||
# one only if we haven't generated it for the first old default route.
|
|
||||||
@default_route_generated = false
|
|
||||||
|
|
||||||
# Setup the stack for parents; used use proper indentation
|
|
||||||
self.class.stack = [@routes]
|
|
||||||
end
|
|
||||||
|
|
||||||
def root(options)
|
|
||||||
debug "mapping root"
|
|
||||||
@routes << FakeRoute.new("/", options)
|
|
||||||
end
|
|
||||||
|
|
||||||
def connect(path, options={})
|
|
||||||
debug "connecting #{path}"
|
|
||||||
|
|
||||||
if (path == ":controller/:action/:id.:format" || path == ":controller/:action/:id")
|
|
||||||
if !@default_route_generated
|
|
||||||
current_parent << FakeRoute.new("/:controller(/:action(/:id))", {:default_route => true})
|
|
||||||
|
|
||||||
@default_route_generated = true
|
|
||||||
end
|
|
||||||
else
|
|
||||||
current_parent << FakeRoute.new(path, options)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def resources(*args)
|
|
||||||
if block_given?
|
|
||||||
parent = FakeResourceRoute.new(args.shift)
|
|
||||||
debug "mapping resources #{parent.name} with block"
|
|
||||||
|
|
||||||
parent = stack(parent) do
|
|
||||||
yield(self)
|
|
||||||
end
|
|
||||||
|
|
||||||
current_parent << parent
|
|
||||||
else
|
|
||||||
if args.last.is_a?(Hash)
|
|
||||||
current_parent << FakeResourceRoute.new(args.shift, args.pop)
|
|
||||||
debug "mapping resources #{current_parent.last.name} w/o block with args"
|
|
||||||
else
|
|
||||||
args.each do |a|
|
|
||||||
current_parent << FakeResourceRoute.new(a)
|
|
||||||
debug "mapping resources #{current_parent.last.name}"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def resource(*args)
|
|
||||||
if block_given?
|
|
||||||
parent = FakeSingletonResourceRoute.new(args.shift)
|
|
||||||
debug "mapping resource #{parent.name} with block"
|
|
||||||
|
|
||||||
parent = stack(parent) do
|
|
||||||
yield(self)
|
|
||||||
end
|
|
||||||
|
|
||||||
current_parent << parent
|
|
||||||
else
|
|
||||||
if args.last.is_a?(Hash)
|
|
||||||
current_parent << FakeSingletonResourceRoute.new(args.shift, args.pop)
|
|
||||||
debug "mapping resources #{current_parent.last.name} w/o block with args"
|
|
||||||
else
|
|
||||||
args.each do |a|
|
|
||||||
current_parent << FakeSingletonResourceRoute.new(a)
|
|
||||||
debug "mapping resources #{current_parent.last.name}"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def namespace(name, options = {})
|
|
||||||
debug "mapping namespace #{name}"
|
|
||||||
namespace = FakeNamespace.new(name, options)
|
|
||||||
|
|
||||||
namespace = stack(namespace) do
|
|
||||||
yield(self)
|
|
||||||
end
|
|
||||||
|
|
||||||
current_parent << namespace
|
|
||||||
end
|
|
||||||
|
|
||||||
def method_missing(m, *args)
|
|
||||||
debug "named route: #{m}"
|
|
||||||
current_parent << FakeRoute.new(args.shift, args.pop, m.to_s)
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.indent
|
|
||||||
' ' * ((stack.length) * 2)
|
|
||||||
end
|
|
||||||
|
|
||||||
private
|
|
||||||
def debug(txt)
|
|
||||||
puts txt if ENV['DEBUG']
|
|
||||||
end
|
|
||||||
|
|
||||||
def stack(obj)
|
|
||||||
self.class.stack << obj
|
|
||||||
yield
|
|
||||||
self.class.stack.pop
|
|
||||||
end
|
|
||||||
|
|
||||||
def current_parent
|
|
||||||
self.class.stack.last
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RouteObject
|
|
||||||
def indent_lines(code_lines)
|
|
||||||
if code_lines.length > 1
|
|
||||||
code_lines.flatten.map {|l| "#{@indent}#{l.chomp}"}.join("\n") + "\n"
|
|
||||||
else
|
|
||||||
"#{@indent}#{code_lines.shift}"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def opts_to_string(opts)
|
|
||||||
opts.is_a?(Hash) ? opts.map {|k, v|
|
|
||||||
":#{k} => " + (v.is_a?(Hash) ? ('{ ' + opts_to_string(v) + ' }') : "#{value_to_string(v)}")
|
|
||||||
}.join(", ") : opts.to_s
|
|
||||||
end
|
|
||||||
|
|
||||||
def value_to_string(value)
|
|
||||||
case value
|
|
||||||
when Regexp then value.inspect
|
|
||||||
when String then "'" + value.to_s + "'"
|
|
||||||
else value.to_s
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class FakeNamespace < RouteObject
|
|
||||||
attr_accessor :routes, :name, :options
|
|
||||||
|
|
||||||
def initialize(name, options = {})
|
|
||||||
@routes = []
|
|
||||||
@name, @options = name, options
|
|
||||||
@indent = RouteRedrawer.indent
|
|
||||||
end
|
|
||||||
|
|
||||||
def to_route_code
|
|
||||||
if !@options.empty?
|
|
||||||
options = ', ' + opts_to_string(@options)
|
|
||||||
else
|
|
||||||
options = ''
|
|
||||||
end
|
|
||||||
|
|
||||||
lines = ["namespace :#{@name}#{options} do", @routes.map {|r| r.to_route_code}, "end"]
|
|
||||||
|
|
||||||
indent_lines(lines)
|
|
||||||
end
|
|
||||||
|
|
||||||
def <<(val)
|
|
||||||
@routes << val
|
|
||||||
end
|
|
||||||
|
|
||||||
def last
|
|
||||||
@routes.last
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class FakeRoute < RouteObject
|
|
||||||
attr_accessor :name, :path, :options
|
|
||||||
|
|
||||||
def initialize(path, options, name = "")
|
|
||||||
@path = path
|
|
||||||
@options = options || {}
|
|
||||||
@name = name
|
|
||||||
@indent = RouteRedrawer.indent
|
|
||||||
end
|
|
||||||
|
|
||||||
def to_route_code
|
|
||||||
if @options[:default_route]
|
|
||||||
indent_lines ["match '#{@path}'"]
|
|
||||||
else
|
|
||||||
base = "match '%s' => '%s#%s'"
|
|
||||||
extra_options = []
|
|
||||||
|
|
||||||
if not name.empty?
|
|
||||||
extra_options << ":as => :#{name}"
|
|
||||||
end
|
|
||||||
|
|
||||||
if @options[:requirements]
|
|
||||||
@options[:constraints] = @options.delete(:requirements)
|
|
||||||
end
|
|
||||||
|
|
||||||
if @options[:conditions]
|
|
||||||
@options[:via] = @options.delete(:conditions).delete(:method)
|
|
||||||
end
|
|
||||||
|
|
||||||
if @options[:method]
|
|
||||||
@options[:via] = @options.delete(:method).to_s
|
|
||||||
end
|
|
||||||
|
|
||||||
@options ||= {}
|
|
||||||
base = (base % [@path, @options.delete(:controller), (@options.delete(:action) || "index")])
|
|
||||||
opts = opts_to_string(@options)
|
|
||||||
|
|
||||||
route_pieces = ([base] + extra_options + [opts])
|
|
||||||
route_pieces.delete("")
|
|
||||||
|
|
||||||
indent_lines [route_pieces.join(", ")]
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class FakeResourceRoute < RouteObject
|
|
||||||
attr_accessor :name, :children
|
|
||||||
|
|
||||||
def initialize(name, options = {})
|
|
||||||
@name = name
|
|
||||||
@children = []
|
|
||||||
@options = options
|
|
||||||
@indent = RouteRedrawer.indent
|
|
||||||
end
|
|
||||||
|
|
||||||
def to_route_code
|
|
||||||
# preserve :only & :except options
|
|
||||||
copied_options = @options.reject { |k,v| ![:only, :except].member?(k) }
|
|
||||||
unless copied_options.empty?
|
|
||||||
copied_options_str = ", " + copied_options.map { |k, v| "#{k.inspect} => #{v.inspect}" }.join(",")
|
|
||||||
end
|
|
||||||
|
|
||||||
if !@children.empty? || @options.has_key?(:collection) || @options.has_key?(:member)
|
|
||||||
prefix = ["#{route_method} :#{@name}#{copied_options_str} do"]
|
|
||||||
lines = prefix + custom_methods + [@children.map {|r| r.to_route_code}.join("\n"), "end"]
|
|
||||||
|
|
||||||
indent_lines(lines)
|
|
||||||
else
|
|
||||||
base = "#{route_method} :%s#{copied_options_str}"
|
|
||||||
indent_lines [base % [@name]]
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def custom_methods
|
|
||||||
collection_code = generate_custom_methods_for(:collection)
|
|
||||||
member_code = generate_custom_methods_for(:member)
|
|
||||||
[collection_code, member_code]
|
|
||||||
end
|
|
||||||
|
|
||||||
def generate_custom_methods_for(group)
|
|
||||||
return "" unless @options[group]
|
|
||||||
|
|
||||||
method_code = []
|
|
||||||
|
|
||||||
RouteRedrawer.stack << self
|
|
||||||
@options[group].each do |name, methods|
|
|
||||||
[*methods].each do |method|
|
|
||||||
method_code << "#{method} :#{name}"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
RouteRedrawer.stack.pop
|
|
||||||
|
|
||||||
indent_lines ["#{group} do", method_code, "end"].flatten
|
|
||||||
end
|
|
||||||
|
|
||||||
def route_method
|
|
||||||
"resources"
|
|
||||||
end
|
|
||||||
|
|
||||||
def <<(val)
|
|
||||||
@children << val
|
|
||||||
end
|
|
||||||
|
|
||||||
def last
|
|
||||||
@children.last
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class FakeSingletonResourceRoute < FakeResourceRoute
|
|
||||||
def route_method
|
|
||||||
"resource"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RouteGenerator
|
|
||||||
def initialize(routes)
|
|
||||||
@routes = routes
|
|
||||||
@new_code = ""
|
|
||||||
end
|
|
||||||
|
|
||||||
def generate
|
|
||||||
@new_code = @routes.map do |r|
|
|
||||||
r.to_route_code
|
|
||||||
end.join("\n")
|
|
||||||
|
|
||||||
"#{app_name.underscore.classify}::Application.routes.draw do\n#{@new_code}\nend\n"
|
|
||||||
end
|
|
||||||
|
|
||||||
private
|
|
||||||
def app_name
|
|
||||||
File.basename(Dir.pwd)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,78 +0,0 @@
|
||||||
$:.unshift(File.dirname(__FILE__) + "/../../lib")
|
|
||||||
require 'routes_upgrader'
|
|
||||||
require 'gemfile_generator'
|
|
||||||
require 'application_checker'
|
|
||||||
require 'new_configuration_generator'
|
|
||||||
|
|
||||||
require 'fileutils'
|
|
||||||
|
|
||||||
namespace :rails do
|
|
||||||
namespace :upgrade do
|
|
||||||
desc "Runs a battery of checks on your Rails 2.x app and generates a report on required upgrades for Rails 3"
|
|
||||||
task :check do
|
|
||||||
checker = Rails::Upgrading::ApplicationChecker.new
|
|
||||||
checker.run
|
|
||||||
end
|
|
||||||
|
|
||||||
desc "Generates a Gemfile for your Rails 3 app out of your config.gem directives"
|
|
||||||
task :gems do
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
new_gemfile = generator.generate_new_gemfile
|
|
||||||
|
|
||||||
puts new_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
desc "Create a new, upgraded route file from your current routes.rb"
|
|
||||||
task :routes do
|
|
||||||
upgrader = Rails::Upgrading::RoutesUpgrader.new
|
|
||||||
new_routes = upgrader.generate_new_routes
|
|
||||||
|
|
||||||
puts new_routes
|
|
||||||
end
|
|
||||||
|
|
||||||
desc "Extracts your configuration code so you can create a new config/application.rb"
|
|
||||||
task :configuration do
|
|
||||||
upgrader = Rails::Upgrading::NewConfigurationGenerator.new
|
|
||||||
new_config = upgrader.generate_new_application_rb
|
|
||||||
|
|
||||||
puts new_config
|
|
||||||
end
|
|
||||||
|
|
||||||
CLEAR = "\e[0m"
|
|
||||||
CYAN = "\e[36m"
|
|
||||||
WHITE = "\e[37m"
|
|
||||||
|
|
||||||
desc "Backs up your likely modified files so you can run the Rails 3 generator on your app with little risk"
|
|
||||||
task :backup do
|
|
||||||
files = [".gitignore",
|
|
||||||
"app/controllers/application_controller.rb",
|
|
||||||
"app/helpers/application_helper.rb",
|
|
||||||
"config/routes.rb",
|
|
||||||
"config/environment.rb",
|
|
||||||
"config/environments/development.rb",
|
|
||||||
"config/environments/production.rb",
|
|
||||||
"config/environments/staging.rb",
|
|
||||||
"config/database.yml",
|
|
||||||
"config.ru",
|
|
||||||
"doc/README_FOR_APP",
|
|
||||||
"test/test_helper.rb"]
|
|
||||||
|
|
||||||
puts
|
|
||||||
files.each do |f|
|
|
||||||
if File.exist?(f)
|
|
||||||
puts "#{CYAN}* #{CLEAR}backing up #{WHITE}#{f}#{CLEAR} to #{WHITE}#{f}.rails2#{CLEAR}"
|
|
||||||
FileUtils.cp(f, "#{f}.rails2")
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
puts
|
|
||||||
puts "This is a list of the files analyzed and backed up (if they existed);\nyou will probably not want the generator to replace them since\nyou probably modified them (but now they're safe if you accidentally do!)."
|
|
||||||
puts
|
|
||||||
|
|
||||||
files.each do |f|
|
|
||||||
puts "#{CYAN}- #{CLEAR}#{f}"
|
|
||||||
end
|
|
||||||
puts
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,330 +0,0 @@
|
||||||
require 'test_helper'
|
|
||||||
require 'application_checker'
|
|
||||||
require 'fileutils'
|
|
||||||
|
|
||||||
tmp_dir = File.expand_path("#{File.dirname(__FILE__)}/fixtures/tmp")
|
|
||||||
|
|
||||||
if defined? BASE_ROOT
|
|
||||||
BASE_ROOT.replace tmp_dir
|
|
||||||
else
|
|
||||||
BASE_ROOT = tmp_dir
|
|
||||||
end
|
|
||||||
FileUtils.mkdir_p BASE_ROOT
|
|
||||||
|
|
||||||
# Stub out methods on upgrader class
|
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class ApplicationChecker
|
|
||||||
attr_reader :alerts, :culprits
|
|
||||||
|
|
||||||
def base_path
|
|
||||||
BASE_ROOT + "/"
|
|
||||||
end
|
|
||||||
|
|
||||||
def in_rails_app?
|
|
||||||
true
|
|
||||||
end
|
|
||||||
|
|
||||||
def initialize
|
|
||||||
@alerts = {}
|
|
||||||
@culprits = {}
|
|
||||||
end
|
|
||||||
|
|
||||||
def alert(title, text, more_info_url, culprits)
|
|
||||||
@alerts[title] = [text, more_info_url]
|
|
||||||
@culprits[title] = culprits
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class ApplicationCheckerTest < ActiveSupport::TestCase
|
|
||||||
def setup
|
|
||||||
@checker = Rails::Upgrading::ApplicationChecker.new
|
|
||||||
@old_dir = Dir.pwd
|
|
||||||
|
|
||||||
Dir.chdir(BASE_ROOT)
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_ar_methods_in_controller
|
|
||||||
make_file("app/controllers", "post_controller.rb", "Post.find(:all)")
|
|
||||||
@checker.check_ar_methods
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Soon-to-be-deprecated ActiveRecord calls")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_ar_methods_in_models
|
|
||||||
make_file("app/models", "post.rb", "Post.find(:all)")
|
|
||||||
@checker.check_ar_methods
|
|
||||||
|
|
||||||
key = "Soon-to-be-deprecated ActiveRecord calls"
|
|
||||||
assert @checker.alerts.has_key?(key)
|
|
||||||
assert_equal "app/models/post.rb", @checker.culprits[key].first
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_svn_subdirs_are_not_included
|
|
||||||
make_file("app/models/.svn/text-base", "foo.rb.tmp", "Post.find(:all)")
|
|
||||||
@checker.check_ar_methods
|
|
||||||
assert @checker.alerts.empty?
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_validation_on_methods
|
|
||||||
make_file("app/models", "post.rb", "validate_on_create :comments_valid?")
|
|
||||||
@checker.check_validation_on_methods
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Updated syntax for validate_on_* methods")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_before_validation_on_methods
|
|
||||||
make_file("app/models", "post.rb", "before_validation_on_create :comments_valid?")
|
|
||||||
@checker.check_before_validation_on_methods
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Updated syntax for before_validation_on_* methods")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_named_scope_left_over
|
|
||||||
make_file("app/models", "post.rb", "scope :failure")
|
|
||||||
@checker.check_ar_methods
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("scope is now just scope")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_routes
|
|
||||||
make_file("config/", "routes.rb", " map.connect 'fail'")
|
|
||||||
@checker.check_routes
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old router API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_for_old_test_help
|
|
||||||
make_file("test/", "test_helper.rb", " require 'test_help'")
|
|
||||||
@checker.check_test_help
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated test_help path")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_for_old_test_help_with_double_quotes
|
|
||||||
make_file("test/", "test_helper.rb", " require \"test_help\"")
|
|
||||||
@checker.check_test_help
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated test_help path")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_for_old_test_help_doesnt_see_test_helper
|
|
||||||
make_file("test/", "test_helper.rb", " require 'test_helper'")
|
|
||||||
@checker.check_test_help
|
|
||||||
|
|
||||||
assert !@checker.alerts.has_key?("Deprecated test_help path")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_lack_of_app_dot_rb
|
|
||||||
@checker.check_environment
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("New file needed: config/application.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_environment_syntax
|
|
||||||
make_file("config/", "environment.rb", "config.frameworks = []")
|
|
||||||
@checker.check_environment
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old environment.rb")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_gems
|
|
||||||
make_file("config/", "environment.rb", "config.gem 'rails'")
|
|
||||||
@checker.check_gems
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old gem bundling (config.gems)")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_gems_finds_nothing
|
|
||||||
@checker.check_gems
|
|
||||||
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Old gem bundling (config.gems)")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_finds_nothing
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_syntax
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\nrecipients @users\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_syntax_from
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\nfrom @user\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_syntax_subject
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\nsubject @subject\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_syntax_attachment
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\nattachment 'application/pdf' do |a|\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_new_check_mailer_syntax_from
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\n:from => @users\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_new_check_mailer_syntax_subject
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\n:subject => @users\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_new_check_mailer_syntax_attachments
|
|
||||||
make_file("app/models/", "notifications.rb", "def signup\nattachments['an-image.jp'] = File.read('an-image.jpg')\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert ! @checker.alerts.has_key?("Old ActionMailer class API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_mailer_api
|
|
||||||
make_file("app/controllers/", "thing_controller.rb", "def signup\n Notifications.deliver_signup\n end")
|
|
||||||
@checker.check_mailers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated ActionMailer API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_generators
|
|
||||||
make_file("vendor/plugins/thing/generators/thing/", "thing_generator.rb", "def manifest\n m.whatever\n end")
|
|
||||||
@checker.check_generators
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old Rails generator API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_plugins
|
|
||||||
make_file("vendor/plugins/rspec-rails/", "whatever.rb", "def rspec; end")
|
|
||||||
@checker.check_plugins
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Known broken plugins")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_ignoring_comments
|
|
||||||
make_file("config/", "routes.rb", "# map.connect 'fail'")
|
|
||||||
@checker.check_routes
|
|
||||||
|
|
||||||
assert !@checker.alerts.has_key?("Old router API")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_constants_in_app_code
|
|
||||||
make_file("app/controllers/", "thing_controller.rb", "class ThingController; THING = RAILS_ENV; end;")
|
|
||||||
@checker.check_deprecated_constants
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated constant(s)")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_constants_in_lib
|
|
||||||
make_file("lib/", "extra_thing.rb", "class ExtraThing; THING = RAILS_ENV; end;")
|
|
||||||
@checker.check_deprecated_constants
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated constant(s)")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_cookie_finds_nothing
|
|
||||||
@checker.check_old_cookie_secret
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Deprecated cookie secret setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_cookie_settings
|
|
||||||
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.cookie_verifier_secret = 'OMG'")
|
|
||||||
@checker.check_old_cookie_secret
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated cookie secret setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_session_finds_nothing
|
|
||||||
@checker.check_old_session_secret
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Deprecated session secret setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_session_secret
|
|
||||||
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.session = {\n:whatever => 'woot'\n}")
|
|
||||||
@checker.check_old_session_secret
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated session secret setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_old_session_setting_finds_nothing
|
|
||||||
@checker.check_old_session_setting
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Old session store setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_deprecated_session_settings
|
|
||||||
make_file("config/initializers/", "more_settings.rb", "ActionController::Base.session_store = :cookie\nthings.awesome(:whatever)")
|
|
||||||
@checker.check_old_session_setting
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Old session store setting")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_helpers
|
|
||||||
make_file("app/views/users/", "test.html.erb", "<b>blah blah blah</b><% form_for(:thing) do |f| %> <label>doo dah</label> <%= f.whatever %> <% end %>")
|
|
||||||
@checker.check_old_helpers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated ERb helper calls")
|
|
||||||
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_old_helpers_lets_regular_blocks_pass
|
|
||||||
make_file("app/views/users/", "another_test.html.erb", "<b>blah blah blah</b><% @some_items.each do |item| %> <label>doo dah</label> <%= item %> <% end %>")
|
|
||||||
@checker.check_old_helpers
|
|
||||||
|
|
||||||
assert_equal @checker.alerts.has_key?("Deprecated ERb helper calls"), false
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_old_helpers_lets_regular_blocks_pass
|
|
||||||
make_file("app/views/users/", "another_test.html.erb", "<b>blah blah blah</b><% @some_items.each do |item| %> <label>doo dah</label> <%= item %> <% end %>")
|
|
||||||
@checker.check_old_helpers
|
|
||||||
|
|
||||||
assert_equal false, @checker.alerts.has_key?("Deprecated ERb helper calls")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_old_ajax_helpers
|
|
||||||
make_file("app/views/sections", "section.js", "<%= link_to_remote 'section-', :update => 'sections', :url => {:action => :destroy, :controller => 'sections', :id => @section.id } %>")
|
|
||||||
@checker.check_old_ajax_helpers
|
|
||||||
|
|
||||||
assert @checker.alerts.has_key?("Deprecated AJAX helper calls")
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_check_old_ajax_helpers_empty
|
|
||||||
@checker.check_old_ajax_helpers
|
|
||||||
|
|
||||||
assert ! @checker.alerts.has_key?("Deprecated AJAX helper calls")
|
|
||||||
end
|
|
||||||
|
|
||||||
def teardown
|
|
||||||
clear_files
|
|
||||||
|
|
||||||
Dir.chdir(@old_dir)
|
|
||||||
end
|
|
||||||
|
|
||||||
def make_file(where, name=nil, contents=nil)
|
|
||||||
FileUtils.mkdir_p "#{BASE_ROOT}/#{where}"
|
|
||||||
File.open("#{BASE_ROOT}/#{where}/#{name}", "w+") do |f|
|
|
||||||
f.write(contents)
|
|
||||||
end if name
|
|
||||||
end
|
|
||||||
|
|
||||||
def clear_files
|
|
||||||
FileUtils.rm_rf(Dir.glob("#{BASE_ROOT}/*"))
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,72 +0,0 @@
|
||||||
require 'test_helper'
|
|
||||||
require 'gemfile_generator'
|
|
||||||
|
|
||||||
# Stub out methods on upgrader class
|
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class GemfileGenerator
|
|
||||||
attr_writer :environment_code
|
|
||||||
|
|
||||||
def has_environment?
|
|
||||||
true
|
|
||||||
end
|
|
||||||
|
|
||||||
def environment_code
|
|
||||||
@environment_code
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class GemfileGeneratorTest < ActiveSupport::TestCase
|
|
||||||
PREAMBLE = <<STR
|
|
||||||
# Edit this Gemfile to bundle your application's dependencies.
|
|
||||||
# This preamble is the current preamble for Rails 3 apps; edit as needed.
|
|
||||||
source 'http://rubygems.org'
|
|
||||||
|
|
||||||
gem 'rails', '3.0.6'
|
|
||||||
|
|
||||||
STR
|
|
||||||
|
|
||||||
def test_generates_with_no_gems
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = ""
|
|
||||||
|
|
||||||
assert_equal PREAMBLE, generator.generate_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_with_gem
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = "config.gem 'camping'"
|
|
||||||
|
|
||||||
assert_equal PREAMBLE + "gem 'camping'", generator.generate_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_with_version
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = "config.gem 'camping', :version => '2.1.1'"
|
|
||||||
|
|
||||||
assert_equal PREAMBLE + "gem 'camping', '2.1.1'", generator.generate_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_can_add_sources
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = "config.gem 'camping', :source => 'http://code.whytheluckystiff.net'"
|
|
||||||
|
|
||||||
assert_equal PREAMBLE + "source 'http://code.whytheluckystiff.net'\ngem 'camping'", generator.generate_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_changes_lib_to_new_key
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = "config.gem 'camping', :lib => 'kamping'"
|
|
||||||
|
|
||||||
assert_equal PREAMBLE + "gem 'camping', :require => 'kamping'", generator.generate_gemfile
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_with_all_options
|
|
||||||
generator = Rails::Upgrading::GemfileGenerator.new
|
|
||||||
generator.environment_code = "config.gem 'camping', :lib => 'kamping', :source => 'http://code.whytheluckystiff.net', :version => '2.1.1'"
|
|
||||||
|
|
||||||
assert_equal PREAMBLE + "source 'http://code.whytheluckystiff.net'\ngem 'camping', '2.1.1', :require => 'kamping'", generator.generate_gemfile
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,63 +0,0 @@
|
||||||
require 'test_helper'
|
|
||||||
require 'new_configuration_generator'
|
|
||||||
|
|
||||||
# Stub out methods on upgrader class
|
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class NewConfigurationGenerator
|
|
||||||
attr_writer :environment_code
|
|
||||||
|
|
||||||
def has_environment?
|
|
||||||
true
|
|
||||||
end
|
|
||||||
|
|
||||||
def environment_code
|
|
||||||
@environment_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def app_name
|
|
||||||
"my_application"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class NewConfigurationGeneratorTest < ActiveSupport::TestCase
|
|
||||||
FRAME = "# Put this in config/application.rb
|
|
||||||
require File.expand_path('../boot', __FILE__)
|
|
||||||
|
|
||||||
module MyApplication
|
|
||||||
class Application < Rails::Application
|
|
||||||
%s
|
|
||||||
end
|
|
||||||
end"
|
|
||||||
|
|
||||||
CONFIG = " config.what_have_you = 'thing'
|
|
||||||
config.action_controller = 'what'"
|
|
||||||
|
|
||||||
CODE = "require 'w/e'
|
|
||||||
|
|
||||||
this_happens_before_the(code)
|
|
||||||
more_before_the_code!
|
|
||||||
|
|
||||||
Rails::Initializer.run do |config|
|
|
||||||
%s
|
|
||||||
end
|
|
||||||
|
|
||||||
this_is_after_the_code
|
|
||||||
"
|
|
||||||
|
|
||||||
def test_raises_error_with_no_code
|
|
||||||
generator = Rails::Upgrading::NewConfigurationGenerator.new
|
|
||||||
generator.environment_code = ""
|
|
||||||
|
|
||||||
assert_raises(RuntimeError) { generator.generate_new_application_rb }
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_with_code
|
|
||||||
generator = Rails::Upgrading::NewConfigurationGenerator.new
|
|
||||||
generator.environment_code = CODE % [CONFIG]
|
|
||||||
|
|
||||||
assert_equal FRAME % [generator.indent(CONFIG)], generator.generate_new_application_rb
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,184 +0,0 @@
|
||||||
require 'test_helper'
|
|
||||||
require 'routes_upgrader'
|
|
||||||
|
|
||||||
# Stub out methods on upgrader class
|
|
||||||
module Rails
|
|
||||||
module Upgrading
|
|
||||||
class RoutesUpgrader
|
|
||||||
attr_writer :routes_code
|
|
||||||
|
|
||||||
def has_routes_file?
|
|
||||||
true
|
|
||||||
end
|
|
||||||
|
|
||||||
def routes_code
|
|
||||||
@routes_code
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RouteGenerator
|
|
||||||
def app_name
|
|
||||||
"MyApplication"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
class RoutesUpgraderTest < ActiveSupport::TestCase
|
|
||||||
def setup
|
|
||||||
Rails::Upgrading::RouteRedrawer.stack = []
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_routes_file
|
|
||||||
routes_code = "
|
|
||||||
ActionController::Routing::Routes.draw do |map|
|
|
||||||
map.connect '/home', :controller => 'home', :action => 'index'
|
|
||||||
map.login '/login', :controller => 'sessions', :action => 'new'
|
|
||||||
|
|
||||||
map.resources :hats
|
|
||||||
map.resource :store
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
new_routes_code = "MyApplication::Application.routes.draw do
|
|
||||||
match '/home' => 'home#index'
|
|
||||||
match '/login' => 'sessions#new', :as => :login
|
|
||||||
resources :hats
|
|
||||||
resource :store
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
upgrader = Rails::Upgrading::RoutesUpgrader.new
|
|
||||||
upgrader.routes_code = routes_code
|
|
||||||
|
|
||||||
result = upgrader.generate_new_routes
|
|
||||||
|
|
||||||
assert_equal new_routes_code, result
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_regular_route
|
|
||||||
route = Rails::Upgrading::FakeRoute.new("/about", {:controller => 'static', :action => 'about'})
|
|
||||||
assert_equal "match '/about' => 'static#about'", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_named_route
|
|
||||||
route = Rails::Upgrading::FakeRoute.new("/about", {:controller => 'static', :action => 'about'}, "about")
|
|
||||||
assert_equal "match '/about' => 'static#about', :as => :about", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_namespace
|
|
||||||
ns = Rails::Upgrading::FakeNamespace.new("static")
|
|
||||||
# Add a route to the namespace
|
|
||||||
ns << Rails::Upgrading::FakeRoute.new("/about", {:controller => 'static', :action => 'about'})
|
|
||||||
|
|
||||||
assert_equal "namespace :static do\nmatch '/about' => 'static#about'\nend\n", ns.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_namespace_with_options
|
|
||||||
ns = Rails::Upgrading::FakeNamespace.new("static", { :path_prefix => 'prefix' })
|
|
||||||
# Add a route to the namespace
|
|
||||||
ns << Rails::Upgrading::FakeRoute.new("/about", {:controller => 'static', :action => 'about'})
|
|
||||||
|
|
||||||
assert_equal "namespace :static, :path_prefix => 'prefix' do\nmatch '/about' => 'static#about'\nend\n", ns.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_resources
|
|
||||||
route = Rails::Upgrading::FakeResourceRoute.new("hats")
|
|
||||||
assert_equal "resources :hats", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_resources
|
|
||||||
route = Rails::Upgrading::FakeSingletonResourceRoute.new("hat")
|
|
||||||
assert_equal "resource :hat", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_resources_with_special_methods
|
|
||||||
route = Rails::Upgrading::FakeResourceRoute.new("hats", {:member => {:wear => :get}, :collection => {:toss => :post}})
|
|
||||||
assert_equal "resources :hats do\ncollection do\npost :toss\nend\nmember do\nget :wear\nend\n\nend\n", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_resources_with_multiple_special_methods_per_name
|
|
||||||
route = Rails::Upgrading::FakeResourceRoute.new("hats", {:member => {:wear => [:get, :put]}, :collection => {:toss => [:get, :post]}})
|
|
||||||
assert_equal "resources :hats do\ncollection do\nget :toss\npost :toss\nend\nmember do\nget :wear\nput :wear\nend\n\nend\n", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_route_with_extra_params
|
|
||||||
route = Rails::Upgrading::FakeRoute.new("/about", {:controller => 'static', :action => 'about', :something => 'extra'})
|
|
||||||
assert_equal "match '/about' => 'static#about', :something => 'extra'", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_route_with_requirements
|
|
||||||
route = Rails::Upgrading::FakeRoute.new("/foo", {:controller => 'foo', :action => 'bar', :requirements => {:digit => /%d/}})
|
|
||||||
assert_equal "match '/foo' => 'foo#bar', :constraints => { :digit => /%d/ }", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_root
|
|
||||||
routes_code = "
|
|
||||||
ActionController::Routing::Routes.draw do |map|
|
|
||||||
map.root :controller => 'home', :action => 'index'
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
new_routes_code = "MyApplication::Application.routes.draw do
|
|
||||||
match '/' => 'home#index'
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
upgrader = Rails::Upgrading::RoutesUpgrader.new
|
|
||||||
upgrader.routes_code = routes_code
|
|
||||||
|
|
||||||
result = upgrader.generate_new_routes
|
|
||||||
|
|
||||||
assert_equal new_routes_code, result
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_default_route
|
|
||||||
routes_code = "
|
|
||||||
ActionController::Routing::Routes.draw do |map|
|
|
||||||
map.connect ':controller/:action/:id.:format'
|
|
||||||
map.connect ':controller/:action/:id'
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
new_routes_code = "MyApplication::Application.routes.draw do
|
|
||||||
match '/:controller(/:action(/:id))'
|
|
||||||
end
|
|
||||||
"
|
|
||||||
|
|
||||||
upgrader = Rails::Upgrading::RoutesUpgrader.new
|
|
||||||
upgrader.routes_code = routes_code
|
|
||||||
|
|
||||||
result = upgrader.generate_new_routes
|
|
||||||
|
|
||||||
assert_equal new_routes_code, result
|
|
||||||
end
|
|
||||||
|
|
||||||
|
|
||||||
def test_preserves_resources_except_option
|
|
||||||
route = Rails::Upgrading::FakeResourceRoute.new("hats", :except => [:index])
|
|
||||||
assert_equal "resources :hats, :except => [:index]", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_preserves_resources_only_option
|
|
||||||
route = Rails::Upgrading::FakeResourceRoute.new("hats", :only => :show)
|
|
||||||
assert_equal "resources :hats, :only => :show", route.to_route_code
|
|
||||||
end
|
|
||||||
|
|
||||||
def test_generates_code_for_delete_route
|
|
||||||
routes_code = %Q{
|
|
||||||
ActionController::Routing::Routes.draw do |map|
|
|
||||||
map.sign_out '/sign_out', :controller => 'sessions', :action => 'destroy', :method => :delete
|
|
||||||
end
|
|
||||||
}
|
|
||||||
|
|
||||||
new_routes_code = %Q{
|
|
||||||
MyApplication::Application.routes.draw do
|
|
||||||
match '/sign_out' => 'sessions#destroy', :as => :sign_out, :via => 'delete'
|
|
||||||
end
|
|
||||||
}
|
|
||||||
|
|
||||||
upgrader = Rails::Upgrading::RoutesUpgrader.new
|
|
||||||
upgrader.routes_code = routes_code
|
|
||||||
assert_equal new_routes_code.strip, upgrader.generate_new_routes.strip
|
|
||||||
end
|
|
||||||
end
|
|
|
@ -1,5 +0,0 @@
|
||||||
require 'test/unit'
|
|
||||||
|
|
||||||
require 'rubygems'
|
|
||||||
require 'active_support'
|
|
||||||
require 'active_support/test_case'
|
|
1
vendor/plugins/rails_upgrade/uninstall.rb
vendored
1
vendor/plugins/rails_upgrade/uninstall.rb
vendored
|
@ -1 +0,0 @@
|
||||||
# Uninstall hook code here
|
|
Loading…
Reference in a new issue