aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--actionpack/CHANGELOG2
-rwxr-xr-xactionpack/README8
-rw-r--r--actionpack/examples/address_book/index.rhtml33
-rw-r--r--actionpack/examples/address_book/layout.rhtml8
-rw-r--r--actionpack/examples/debate/index.rhtml14
-rw-r--r--actionpack/examples/debate/new_topic.rhtml22
-rw-r--r--actionpack/examples/debate/topic.rhtml32
-rwxr-xr-xactionpack/lib/action_controller/base.rb18
-rw-r--r--actionpack/lib/action_controller/flash.rb2
-rw-r--r--actionpack/lib/action_controller/layout.rb6
-rw-r--r--actionpack/lib/action_controller/rescue.rb2
-rw-r--r--actionpack/lib/action_controller/scaffolding.rb4
-rw-r--r--actionpack/lib/action_controller/templates/rescues/_request_and_response.rhtml44
-rw-r--r--actionpack/lib/action_controller/templates/rescues/_trace.rhtml26
-rw-r--r--actionpack/lib/action_controller/templates/rescues/diagnostics.rhtml11
-rw-r--r--actionpack/lib/action_controller/templates/rescues/layout.rhtml29
-rw-r--r--actionpack/lib/action_controller/templates/rescues/missing_template.rhtml2
-rw-r--r--actionpack/lib/action_controller/templates/rescues/routing_error.rhtml10
-rw-r--r--actionpack/lib/action_controller/templates/rescues/template_error.rhtml21
-rw-r--r--actionpack/lib/action_controller/templates/rescues/unknown_action.rhtml2
-rw-r--r--actionpack/lib/action_controller/templates/scaffolds/edit.rhtml7
-rw-r--r--actionpack/lib/action_controller/templates/scaffolds/layout.rhtml69
-rw-r--r--actionpack/lib/action_controller/templates/scaffolds/list.rhtml27
-rw-r--r--actionpack/lib/action_controller/templates/scaffolds/new.rhtml6
-rw-r--r--actionpack/lib/action_controller/templates/scaffolds/show.rhtml9
-rw-r--r--actionpack/lib/action_view/base.rb41
-rw-r--r--actionpack/lib/action_view/helpers/capture_helper.rb8
-rw-r--r--actionpack/lib/action_view/partials.rb10
-rw-r--r--actionpack/test/controller/layout_test.rb2
-rw-r--r--actionpack/test/controller/new_render_test.rb8
-rw-r--r--actionpack/test/controller/rescue_test.rb2
-rw-r--r--actionpack/test/fixtures/addresses/list.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_cookies_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_cookies_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_flash_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_flash_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_headers_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_headers_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_params_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_params_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_request_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_request_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_response_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_response_method.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_session_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/deprecated_instance_variables/_session_method.rhtml1
-rw-r--r--actionpack/test/fixtures/fun/games/hello_world.rhtml1
-rw-r--r--actionpack/test/fixtures/layouts/builder.rxml3
-rw-r--r--actionpack/test/fixtures/layouts/standard.rhtml1
-rw-r--r--actionpack/test/fixtures/layouts/talk_from_action.rhtml2
-rw-r--r--actionpack/test/fixtures/layouts/yield.rhtml2
-rw-r--r--actionpack/test/fixtures/override/test/hello_world.rhtml1
-rw-r--r--actionpack/test/fixtures/scope/test/modgreet.rhtml1
-rw-r--r--actionpack/test/fixtures/test/_customer.rhtml1
-rw-r--r--actionpack/test/fixtures/test/_customer_greeting.rhtml1
-rw-r--r--actionpack/test/fixtures/test/_hash_object.rhtml1
-rw-r--r--actionpack/test/fixtures/test/_hello.rxml1
-rw-r--r--actionpack/test/fixtures/test/_partial_only.rhtml1
-rw-r--r--actionpack/test/fixtures/test/_person.rhtml2
-rw-r--r--actionpack/test/fixtures/test/action_talk_to_layout.rhtml2
-rw-r--r--actionpack/test/fixtures/test/block_content_for.rhtml2
-rw-r--r--actionpack/test/fixtures/test/capturing.rhtml4
-rw-r--r--actionpack/test/fixtures/test/content_for.rhtml2
-rw-r--r--actionpack/test/fixtures/test/dot.directory/render_file_with_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/test/erb_content_for.rhtml2
-rw-r--r--actionpack/test/fixtures/test/greeting.rhtml1
-rw-r--r--actionpack/test/fixtures/test/hello.rxml4
-rw-r--r--actionpack/test/fixtures/test/hello_world.rhtml1
-rw-r--r--actionpack/test/fixtures/test/hello_world.rxml3
-rw-r--r--actionpack/test/fixtures/test/hello_world_container.rxml3
-rw-r--r--actionpack/test/fixtures/test/hello_world_with_layout_false.rhtml1
-rw-r--r--actionpack/test/fixtures/test/hello_xml_world.rxml11
-rw-r--r--actionpack/test/fixtures/test/list.rhtml1
-rw-r--r--actionpack/test/fixtures/test/non_erb_block_content_for.rxml4
-rw-r--r--actionpack/test/fixtures/test/potential_conflicts.rhtml4
-rw-r--r--actionpack/test/fixtures/test/render_file_with_ivar.rhtml1
-rw-r--r--actionpack/test/fixtures/test/render_file_with_locals.rhtml1
-rw-r--r--actionpack/test/fixtures/test/render_to_string_test.rhtml1
-rw-r--r--actionpack/test/fixtures/test/update_element_with_capture.rhtml9
79 files changed, 61 insertions, 515 deletions
diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG
index 8380af610b..5c61cd9e06 100644
--- a/actionpack/CHANGELOG
+++ b/actionpack/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Added .erb and .builder as preferred aliases to the now deprecated .rhtml and .rxml extensions [Chad Fowler]. This is done to separate the renderer from the mime type. .erb templates are often used to render emails, atom, csv, whatever. So labeling them .rhtml doesn't make too much sense. The same goes for .rxml, which can be used to build everything from HTML to Atom to whatever. .rhtml and .rxml will continue to work until Rails 3.0, though. So this is a slow phasing out. All generators and examples will start using the new aliases, though.
+
* Added caching option to AssetTagHelper#stylesheet_link_tag and AssetTagHelper#javascript_include_tag [DHH]. Examples:
stylesheet_link_tag :all, :cache => true # when ActionController::Base.perform_caching is false =>
diff --git a/actionpack/README b/actionpack/README
index de50e28177..72f0f6241a 100755
--- a/actionpack/README
+++ b/actionpack/README
@@ -391,23 +391,23 @@ request from the web-server (like to be Apache).
And the templates look like this:
- weblog/layout.rhtml:
+ weblog/layout.erb:
<html><body>
<%= yield %>
</body></html>
- weblog/index.rhtml:
+ weblog/index.erb:
<% for post in @posts %>
<p><%= link_to(post.title, :action => "display", :id => post.id %></p>
<% end %>
- weblog/display.rhtml:
+ weblog/display.erb:
<p>
<b><%= post.title %></b><br/>
<b><%= post.content %></b>
</p>
- weblog/new.rhtml:
+ weblog/new.erb:
<%= form "post" %>
This simple setup will list all the posts in the system on the index page,
diff --git a/actionpack/examples/address_book/index.rhtml b/actionpack/examples/address_book/index.rhtml
index 217d39075c..e69de29bb2 100644
--- a/actionpack/examples/address_book/index.rhtml
+++ b/actionpack/examples/address_book/index.rhtml
@@ -1,33 +0,0 @@
-<h1>Address Book</h1>
-
-<% if @people.empty? %>
- <p>No people in the address book yet</p>
-<% else %>
- <table>
- <tr><th>Name</th><th>Email Address</th><th>Phone Number</th></tr>
- <% for person in @people %>
- <tr><td><%= person.name %></td><td><%= person.email_address %></td><td><%= person.phone_number %></td></tr>
- <% end %>
- </table>
-<% end %>
-
-<form action="create_person">
- <p>
- Name:<br />
- <input type="text" name="person[name]">
- </p>
-
- <p>
- Email address:<br />
- <input type="text" name="person[email_address]">
- </p>
-
- <p>
- Phone number:<br />
- <input type="text" name="person[phone_number]">
- </p>
-
- <p>
- <input type="submit" value="Create Person">
- </p>
-</form> \ No newline at end of file
diff --git a/actionpack/examples/address_book/layout.rhtml b/actionpack/examples/address_book/layout.rhtml
index 931e141c01..e69de29bb2 100644
--- a/actionpack/examples/address_book/layout.rhtml
+++ b/actionpack/examples/address_book/layout.rhtml
@@ -1,8 +0,0 @@
-<html>
-<head>
- <title><%= @title || "Untitled" %></title>
-</head>
-<body>
-<%= @content_for_layout %>
-</body>
-</html> \ No newline at end of file
diff --git a/actionpack/examples/debate/index.rhtml b/actionpack/examples/debate/index.rhtml
index ddaa87da57..e69de29bb2 100644
--- a/actionpack/examples/debate/index.rhtml
+++ b/actionpack/examples/debate/index.rhtml
@@ -1,14 +0,0 @@
-<html>
-<body>
-<h1>Topics</h1>
-
-<%= link_to "New topic", :action => "new_topic" %>
-
-<ul>
-<% for topic in @topics %>
- <li><%= link_to "#{topic.title} (#{topic.replies.length} replies)", :action => "topic", :path_params => { "id" => topic.id } %></li>
-<% end %>
-</ul>
-
-</body>
-</html> \ No newline at end of file
diff --git a/actionpack/examples/debate/new_topic.rhtml b/actionpack/examples/debate/new_topic.rhtml
index f52a69cc31..e69de29bb2 100644
--- a/actionpack/examples/debate/new_topic.rhtml
+++ b/actionpack/examples/debate/new_topic.rhtml
@@ -1,22 +0,0 @@
-<html>
-<body>
-<h1>New topic</h1>
-
-<form action="<%= url_for(:action => "create_topic") %>" method="post">
- <p>
- Title:<br>
- <input type="text" name="topic[title]">
- </p>
-
- <p>
- Body:<br>
- <textarea name="topic[body]" style="width: 200px; height: 200px"></textarea>
- </p>
-
- <p>
- <input type="submit" value="Create topic">
- </p>
-</form>
-
-</body>
-</html> \ No newline at end of file
diff --git a/actionpack/examples/debate/topic.rhtml b/actionpack/examples/debate/topic.rhtml
index e247c00f0d..e69de29bb2 100644
--- a/actionpack/examples/debate/topic.rhtml
+++ b/actionpack/examples/debate/topic.rhtml
@@ -1,32 +0,0 @@
-<html>
-<body>
-<h1><%= @topic.title %></h1>
-
-<p><%= @topic.body %></p>
-
-<%= link_to "Back to topics", :action => "index" %>
-
-<% unless @topic.replies.empty? %>
- <h2>Replies</h2>
- <ol>
- <% for reply in @topic.replies %>
- <li><%= reply.body %></li>
- <% end %>
- </ol>
-<% end %>
-
-<h2>Reply to this topic</h2>
-
-<form action="<%= url_for(:action => "create_reply") %>" method="post">
- <input type="hidden" name="reply[topic_id]" value="<%= @topic.id %>">
- <p>
- <textarea name="reply[body]" style="width: 200px; height: 200px"></textarea>
- </p>
-
- <p>
- <input type="submit" value="Create reply">
- </p>
-</form>
-
-</body>
-</html> \ No newline at end of file
diff --git a/actionpack/lib/action_controller/base.rb b/actionpack/lib/action_controller/base.rb
index 360e115f16..39ffb06212 100755
--- a/actionpack/lib/action_controller/base.rb
+++ b/actionpack/lib/action_controller/base.rb
@@ -71,7 +71,7 @@ module ActionController #:nodoc:
#
# Actions, by default, render a template in the <tt>app/views</tt> directory corresponding to the name of the controller and action
# after executing code in the action. For example, the +index+ action of the +GuestBookController+ would render the
- # template <tt>app/views/guestbook/index.rhtml</tt> by default after populating the <tt>@entries</tt> instance variable.
+ # template <tt>app/views/guestbook/index.erb</tt> by default after populating the <tt>@entries</tt> instance variable.
#
# Unlike index, the sign action will not render a template. After performing its main purpose (creating a
# new entry in the guest book), it initiates a redirect instead. This redirect works by returning an external
@@ -662,7 +662,7 @@ module ActionController #:nodoc:
# Template rendering works just like action rendering except that it takes a path relative to the template root.
# The current layout is automatically applied.
#
- # # Renders the template located in [TEMPLATE_ROOT]/weblog/show.r(html|xml) (in Rails, app/views/weblog/show.rhtml)
+ # # Renders the template located in [TEMPLATE_ROOT]/weblog/show.r(html|xml) (in Rails, app/views/weblog/show.erb)
# render :template => "weblog/show"
#
# === Rendering a file
@@ -671,12 +671,12 @@ module ActionController #:nodoc:
# is assumed to be absolute, and the current layout is not applied.
#
# # Renders the template located at the absolute filesystem path
- # render :file => "/path/to/some/template.rhtml"
- # render :file => "c:/path/to/some/template.rhtml"
+ # render :file => "/path/to/some/template.erb"
+ # render :file => "c:/path/to/some/template.erb"
#
# # Renders a template within the current layout, and with a 404 status code
- # render :file => "/path/to/some/template.rhtml", :layout => true, :status => 404
- # render :file => "c:/path/to/some/template.rhtml", :layout => true, :status => 404
+ # render :file => "/path/to/some/template.erb", :layout => true, :status => 404
+ # render :file => "c:/path/to/some/template.erb", :layout => true, :status => 404
#
# # Renders a template relative to the template root and chooses the proper file extension
# render :file => "some/template", :use_full_path => true
@@ -734,7 +734,7 @@ module ActionController #:nodoc:
# render :inline => "<%= 'hello, ' * 3 + 'again' %>"
#
# # Renders "<p>Good seeing you!</p>" using Builder
- # render :inline => "xml.p { 'Good seeing you!' }", :type => :rxml
+ # render :inline => "xml.p { 'Good seeing you!' }", :type => :builder
#
# # Renders "hello david"
# render :inline => "<%= 'hello ' + name %>", :locals => { :name => "david" }
@@ -863,7 +863,7 @@ module ActionController #:nodoc:
render_text(@template.render_file(template_path, use_full_path, locals), status)
end
- def render_template(template, status = nil, type = :rhtml, local_assigns = {}) #:nodoc:
+ def render_template(template, status = nil, type = :erb, local_assigns = {}) #:nodoc:
add_variables_to_assigns
render_text(@template.render_template(type, template, nil, local_assigns), status)
end
@@ -1252,7 +1252,7 @@ module ActionController #:nodoc:
def assert_existence_of_template_file(template_name)
unless template_exists?(template_name) || ignore_missing_templates
- full_template_path = @template.send(:full_template_path, template_name, 'rhtml')
+ full_template_path = @template.send(:full_template_path, template_name, 'erb')
template_type = (template_name =~ /layouts/i) ? 'layout' : 'template'
raise(MissingTemplate, "Missing #{template_type} #{full_template_path}")
end
diff --git a/actionpack/lib/action_controller/flash.rb b/actionpack/lib/action_controller/flash.rb
index 9f50b9571d..0d12503904 100644
--- a/actionpack/lib/action_controller/flash.rb
+++ b/actionpack/lib/action_controller/flash.rb
@@ -16,7 +16,7 @@ module ActionController #:nodoc:
# end
# end
#
- # display.rhtml
+ # display.erb
# <% if flash[:notice] %><div class="notice"><%= flash[:notice] %></div><% end %>
#
# This example just places a string in the flash, but you can put any object in there. And of course, you can put as many
diff --git a/actionpack/lib/action_controller/layout.rb b/actionpack/lib/action_controller/layout.rb
index 6eaf8bb31f..d5d38e2a9a 100644
--- a/actionpack/lib/action_controller/layout.rb
+++ b/actionpack/lib/action_controller/layout.rb
@@ -64,11 +64,11 @@ module ActionController #:nodoc:
#
# If there is a template in <tt>app/views/layouts/</tt> with the same name as the current controller then it will be automatically
# set as that controller's layout unless explicitly told otherwise. Say you have a WeblogController, for example. If a template named
- # <tt>app/views/layouts/weblog.rhtml</tt> or <tt>app/views/layouts/weblog.rxml</tt> exists then it will be automatically set as
- # the layout for your WeblogController. You can create a layout with the name <tt>application.rhtml</tt> or <tt>application.rxml</tt>
+ # <tt>app/views/layouts/weblog.erb</tt> or <tt>app/views/layouts/weblog.builder</tt> exists then it will be automatically set as
+ # the layout for your WeblogController. You can create a layout with the name <tt>application.erb</tt> or <tt>application.builder</tt>
# and this will be set as the default controller if there is no layout with the same name as the current controller and there is
# no layout explicitly assigned with the +layout+ method. Nested controllers use the same folder structure for automatic layout.
- # assignment. So an Admin::WeblogController will look for a template named <tt>app/views/layouts/admin/weblog.rhtml</tt>.
+ # assignment. So an Admin::WeblogController will look for a template named <tt>app/views/layouts/admin/weblog.erb</tt>.
# Setting a layout explicitly will always override the automatic behaviour for the controller where the layout is set.
# Explicitly setting the layout in a parent class, though, will not override the child class's layout assignement if the child
# class has a layout with the same name.
diff --git a/actionpack/lib/action_controller/rescue.rb b/actionpack/lib/action_controller/rescue.rb
index 9f94e076c7..cd3a1b5650 100644
--- a/actionpack/lib/action_controller/rescue.rb
+++ b/actionpack/lib/action_controller/rescue.rb
@@ -152,7 +152,7 @@ module ActionController #:nodoc:
end
def rescues_path(template_name)
- "#{File.dirname(__FILE__)}/templates/rescues/#{template_name}.rhtml"
+ "#{File.dirname(__FILE__)}/templates/rescues/#{template_name}.erb"
end
def template_path_for_local_rescue(exception)
diff --git a/actionpack/lib/action_controller/scaffolding.rb b/actionpack/lib/action_controller/scaffolding.rb
index d02a31e6fc..11ece16f0e 100644
--- a/actionpack/lib/action_controller/scaffolding.rb
+++ b/actionpack/lib/action_controller/scaffolding.rb
@@ -71,7 +71,7 @@ module ActionController
# end
# end
#
- # The <tt>render_scaffold</tt> method will first check to see if you've made your own template (like "weblog/show.rhtml" for
+ # The <tt>render_scaffold</tt> method will first check to see if you've made your own template (like "weblog/show.erb" for
# the show action) and if not, then render the generic template for that action. This gives you the possibility of using the
# scaffold while you're building your specific application. Start out with a totally generic setup, then replace one template
# and one action at a time while relying on the rest of the scaffolded templates and actions.
@@ -176,7 +176,7 @@ module ActionController
end
def scaffold_path(template_name)
- File.dirname(__FILE__) + "/templates/scaffolds/" + template_name + ".rhtml"
+ File.dirname(__FILE__) + "/templates/scaffolds/" + template_name + ".erb"
end
def caller_method_name(caller)
diff --git a/actionpack/lib/action_controller/templates/rescues/_request_and_response.rhtml b/actionpack/lib/action_controller/templates/rescues/_request_and_response.rhtml
index fe60bfdd40..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/_request_and_response.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/_request_and_response.rhtml
@@ -1,44 +0,0 @@
-<% unless @exception.blamed_files.blank? %>
- <% if (hide = @exception.blamed_files.length > 8) %>
- <a href="#" onclick="document.getElementById('blame_trace').style.display='block'; return false;">Show blamed files</a>
- <% end %>
- <pre id="blame_trace" <%='style="display:none"' if hide %>><code><%=h @exception.describe_blame %></code></pre>
-<% end %>
-
-<% if false %>
- <br /><br />
- <% begin %>
- <%= form_tag(request.request_uri, "method" => request.method) %>
- <input type="hidden" name="BP-RETRY" value="1" />
-
- <% for key, values in params %>
- <% next if key == "BP-RETRY" %>
- <% for value in Array(values) %>
- <input type="hidden" name="<%= key %>" value="<%= value %>" />
- <% end %>
- <% end %>
-
- <input type="submit" value="Retry with Breakpoint" />
- </form>
- <% rescue Exception => e %>
- <%=h "Couldn't render breakpoint link due to #{e.class} #{e.message}" %>
- <% end %>
-<% end %>
-
-<%
- clean_params = request.parameters.clone
- clean_params.delete("action")
- clean_params.delete("controller")
-
- request_dump = clean_params.empty? ? 'None' : clean_params.inspect.gsub(',', ",\n")
-%>
-
-<h2 style="margin-top: 30px">Request</h2>
-<p><b>Parameters</b>: <pre><%=h request_dump %></pre></p>
-
-<p><a href="#" onclick="document.getElementById('session_dump').style.display='block'; return false;">Show session dump</a></p>
-<div id="session_dump" style="display:none"><%= debug(request.session.instance_variable_get("@data")) %></div>
-
-
-<h2 style="margin-top: 30px">Response</h2>
-<p><b>Headers</b>: <pre><%=h response ? response.headers.inspect.gsub(',', ",\n") : 'None' %></pre></p>
diff --git a/actionpack/lib/action_controller/templates/rescues/_trace.rhtml b/actionpack/lib/action_controller/templates/rescues/_trace.rhtml
index b322b0aaae..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/_trace.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/_trace.rhtml
@@ -1,26 +0,0 @@
-<%
- traces = [
- ["Application Trace", @exception.application_backtrace],
- ["Framework Trace", @exception.framework_backtrace],
- ["Full Trace", @exception.clean_backtrace]
- ]
- names = traces.collect {|name, trace| name}
-%>
-
-<p><code>RAILS_ROOT: <%= defined?(RAILS_ROOT) ? RAILS_ROOT : "unset" %></code></p>
-
-<div id="traces">
- <% names.each do |name| -%>
- <%
- show = "document.getElementById('#{name.gsub /\s/, '-'}').style.display='block';"
- hide = (names - [name]).collect {|hide_name| "document.getElementById('#{hide_name.gsub /\s/, '-'}').style.display='none';"}
- %>
- <a href="#" onclick="<%= hide %><%= show %>; return false;"><%= name %></a> <%= '|' unless names.last == name %>
- <% end -%>
-
- <% traces.each do |name, trace| -%>
- <div id="<%= name.gsub /\s/, '-' %>" style="display: <%= name == "Application Trace" ? 'block' : 'none' %>;">
- <pre><code><%= trace.join "\n" %></code></pre>
- </div>
- <% end -%>
-</div> \ No newline at end of file
diff --git a/actionpack/lib/action_controller/templates/rescues/diagnostics.rhtml b/actionpack/lib/action_controller/templates/rescues/diagnostics.rhtml
index e9faacef09..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/diagnostics.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/diagnostics.rhtml
@@ -1,11 +0,0 @@
-<h1>
- <%=h @exception.class.to_s %>
- <% if request.parameters['controller'] %>
- in <%=h request.parameters['controller'].humanize %>Controller<% if request.parameters['action'] %>#<%=h request.parameters['action'] %><% end %>
- <% end %>
-</h1>
-<pre><%=h @exception.clean_message %></pre>
-
-<%= render_file(@rescues_path + "/_trace.rhtml", false) %>
-
-<%= render_file(@rescues_path + "/_request_and_response.rhtml", false) %>
diff --git a/actionpack/lib/action_controller/templates/rescues/layout.rhtml b/actionpack/lib/action_controller/templates/rescues/layout.rhtml
index d38f3e67f9..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/layout.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/layout.rhtml
@@ -1,29 +0,0 @@
-<html>
-<head>
- <title>Action Controller: Exception caught</title>
- <style>
- body { background-color: #fff; color: #333; }
-
- body, p, ol, ul, td {
- font-family: verdana, arial, helvetica, sans-serif;
- font-size: 13px;
- line-height: 18px;
- }
-
- pre {
- background-color: #eee;
- padding: 10px;
- font-size: 11px;
- }
-
- a { color: #000; }
- a:visited { color: #666; }
- a:hover { color: #fff; background-color:#000; }
- </style>
-</head>
-<body>
-
-<%= @contents %>
-
-</body>
-</html> \ No newline at end of file
diff --git a/actionpack/lib/action_controller/templates/rescues/missing_template.rhtml b/actionpack/lib/action_controller/templates/rescues/missing_template.rhtml
index dbfdf76947..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/missing_template.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/missing_template.rhtml
@@ -1,2 +0,0 @@
-<h1>Template is missing</h1>
-<p><%=h @exception.message %></p>
diff --git a/actionpack/lib/action_controller/templates/rescues/routing_error.rhtml b/actionpack/lib/action_controller/templates/rescues/routing_error.rhtml
index ccfa858cce..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/routing_error.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/routing_error.rhtml
@@ -1,10 +0,0 @@
-<h1>Routing Error</h1>
-<p><pre><%=h @exception.message %></pre></p>
-<% unless @exception.failures.empty? %><p>
- <h2>Failure reasons:</h2>
- <ol>
- <% @exception.failures.each do |route, reason| %>
- <li><code><%=h route.inspect.gsub('\\', '') %></code> failed because <%=h reason.downcase %></li>
- <% end %>
- </ol>
-</p><% end %>
diff --git a/actionpack/lib/action_controller/templates/rescues/template_error.rhtml b/actionpack/lib/action_controller/templates/rescues/template_error.rhtml
index 827d66467d..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/template_error.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/template_error.rhtml
@@ -1,21 +0,0 @@
-<h1>
- <%=h @exception.original_exception.class.to_s %> in
- <%=h request.parameters["controller"].capitalize if request.parameters["controller"]%>#<%=h request.parameters["action"] %>
-</h1>
-
-<p>
- Showing <i><%=h @exception.file_name %></i> where line <b>#<%=h @exception.line_number %></b> raised:
- <pre><code><%=h @exception.message %></code></pre>
-</p>
-
-<p>Extracted source (around line <b>#<%=h @exception.line_number %></b>):
-<pre><code><%=h @exception.source_extract %></code></pre></p>
-
-<p><%=h @exception.sub_template_message %></p>
-
-<% @real_exception = @exception
- @exception = @exception.original_exception || @exception %>
-<%= render_file(@rescues_path + "/_trace.rhtml", false) %>
-<% @exception = @real_exception %>
-
-<%= render_file(@rescues_path + "/_request_and_response.rhtml", false) %>
diff --git a/actionpack/lib/action_controller/templates/rescues/unknown_action.rhtml b/actionpack/lib/action_controller/templates/rescues/unknown_action.rhtml
index 683379da10..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/rescues/unknown_action.rhtml
+++ b/actionpack/lib/action_controller/templates/rescues/unknown_action.rhtml
@@ -1,2 +0,0 @@
-<h1>Unknown action</h1>
-<p><%=h @exception.message %></p>
diff --git a/actionpack/lib/action_controller/templates/scaffolds/edit.rhtml b/actionpack/lib/action_controller/templates/scaffolds/edit.rhtml
index 63dff602a1..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/scaffolds/edit.rhtml
+++ b/actionpack/lib/action_controller/templates/scaffolds/edit.rhtml
@@ -1,7 +0,0 @@
-<h1>Editing <%= @scaffold_singular_name %></h1>
-
-<%= error_messages_for(@scaffold_singular_name) %>
-<%= form(@scaffold_singular_name, :action => "update#{@scaffold_suffix}") %>
-
-<%= link_to "Show", :action => "show#{@scaffold_suffix}", :id => instance_variable_get("@#{@scaffold_singular_name}") %> |
-<%= link_to "Back", :action => "list#{@scaffold_suffix}" %>
diff --git a/actionpack/lib/action_controller/templates/scaffolds/layout.rhtml b/actionpack/lib/action_controller/templates/scaffolds/layout.rhtml
index 759781e0e7..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/scaffolds/layout.rhtml
+++ b/actionpack/lib/action_controller/templates/scaffolds/layout.rhtml
@@ -1,69 +0,0 @@
-<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
- "http://www.w3.org/TR/html4/loose.dtd">
-<html>
-<head>
- <title>Scaffolding</title>
- <style>
- body { background-color: #fff; color: #333; }
-
- body, p, ol, ul, td {
- font-family: verdana, arial, helvetica, sans-serif;
- font-size: 13px;
- line-height: 18px;
- }
-
- pre {
- background-color: #eee;
- padding: 10px;
- font-size: 11px;
- }
-
- a { color: #000; }
- a:visited { color: #666; }
- a:hover { color: #fff; background-color:#000; }
-
- .fieldWithErrors {
- padding: 2px;
- background-color: red;
- display: table;
- }
-
- #errorExplanation {
- width: 400px;
- border: 2px solid red;
- padding: 7px;
- padding-bottom: 12px;
- margin-bottom: 20px;
- background-color: #f0f0f0;
- }
-
- #errorExplanation h2 {
- text-align: left;
- font-weight: bold;
- padding: 5px 5px 5px 15px;
- font-size: 12px;
- margin: -7px;
- background-color: #c00;
- color: #fff;
- }
-
- #errorExplanation p {
- color: #333;
- margin-bottom: 0;
- padding: 5px;
- }
-
- #errorExplanation ul li {
- font-size: 12px;
- list-style: square;
- }
- </style>
-</head>
-<body>
-
-<p style="color: green"><%= flash[:notice] %></p>
-
-<%= yield %>
-
-</body>
-</html>
diff --git a/actionpack/lib/action_controller/templates/scaffolds/list.rhtml b/actionpack/lib/action_controller/templates/scaffolds/list.rhtml
index fea23dc66f..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/scaffolds/list.rhtml
+++ b/actionpack/lib/action_controller/templates/scaffolds/list.rhtml
@@ -1,27 +0,0 @@
-<h1>Listing <%= @scaffold_plural_name %></h1>
-
-<table>
- <tr>
- <% for column in @scaffold_class.content_columns %>
- <th><%= column.human_name %></th>
- <% end %>
- </tr>
-
-<% for entry in instance_variable_get("@#{@scaffold_plural_name}") %>
- <tr>
- <% for column in @scaffold_class.content_columns %>
- <td><%= entry.send(column.name) %></td>
- <% end %>
- <td><%= link_to "Show", :action => "show#{@scaffold_suffix}", :id => entry %></td>
- <td><%= link_to "Edit", :action => "edit#{@scaffold_suffix}", :id => entry %></td>
- <td><%= link_to "Destroy", {:action => "destroy#{@scaffold_suffix}", :id => entry}, { :confirm => "Are you sure?", :method => :post } %></td>
- </tr>
-<% end %>
-</table>
-
-<%= link_to "Previous page", { :page => instance_variable_get("@#{@scaffold_singular_name}_pages").current.previous } if instance_variable_get("@#{@scaffold_singular_name}_pages").current.previous %>
-<%= link_to "Next page", { :page => instance_variable_get("@#{@scaffold_singular_name}_pages").current.next } if instance_variable_get("@#{@scaffold_singular_name}_pages").current.next %>
-
-<br />
-
-<%= link_to "New #{@scaffold_singular_name}", :action => "new#{@scaffold_suffix}" %>
diff --git a/actionpack/lib/action_controller/templates/scaffolds/new.rhtml b/actionpack/lib/action_controller/templates/scaffolds/new.rhtml
index 66f6626f4e..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/scaffolds/new.rhtml
+++ b/actionpack/lib/action_controller/templates/scaffolds/new.rhtml
@@ -1,6 +0,0 @@
-<h1>New <%= @scaffold_singular_name %></h1>
-
-<%= error_messages_for(@scaffold_singular_name) %>
-<%= form(@scaffold_singular_name, :action => "create#{@scaffold_suffix}") %>
-
-<%= link_to "Back", :action => "list#{@scaffold_suffix}" %>
diff --git a/actionpack/lib/action_controller/templates/scaffolds/show.rhtml b/actionpack/lib/action_controller/templates/scaffolds/show.rhtml
index 46cdfdb493..e69de29bb2 100644
--- a/actionpack/lib/action_controller/templates/scaffolds/show.rhtml
+++ b/actionpack/lib/action_controller/templates/scaffolds/show.rhtml
@@ -1,9 +0,0 @@
-<% for column in @scaffold_class.content_columns %>
- <p>
- <b><%= column.human_name %>:</b>
- <%= instance_variable_get("@#{@scaffold_singular_name}").send(column.name) %>
- </p>
-<% end %>
-
-<%= link_to "Edit", :action => "edit#{@scaffold_suffix}", :id => instance_variable_get("@#{@scaffold_singular_name}") %> |
-<%= link_to "Back", :action => "list#{@scaffold_suffix}" %>
diff --git a/actionpack/lib/action_view/base.rb b/actionpack/lib/action_view/base.rb
index 18d03f5a0b..1b3cf5d95e 100644
--- a/actionpack/lib/action_view/base.rb
+++ b/actionpack/lib/action_view/base.rb
@@ -4,8 +4,8 @@ module ActionView #:nodoc:
class ActionViewError < StandardError #:nodoc:
end
- # Action View templates can be written in three ways. If the template file has a +.rhtml+ extension then it uses a mixture of ERb
- # (included in Ruby) and HTML. If the template file has a +.rxml+ extension then Jim Weirich's Builder::XmlMarkup library is used.
+ # Action View templates can be written in three ways. If the template file has a +.erb+ (or +.rhtml+) extension then it uses a mixture of ERb
+ # (included in Ruby) and HTML. If the template file has a +.builder+ (or +.rxml+) extension then Jim Weirich's Builder::XmlMarkup library is used.
# If the template file has a +.rjs+ extension then it will use ActionView::Helpers::PrototypeHelper::JavaScriptGenerator.
#
# = ERb
@@ -77,7 +77,7 @@ module ActionView #:nodoc:
# == Builder
#
# Builder templates are a more programmatic alternative to ERb. They are especially useful for generating XML content. An +XmlMarkup+ object
- # named +xml+ is automatically made available to templates with a +.rxml+ extension.
+ # named +xml+ is automatically made available to templates with a +.builder+ extension.
#
# Here are some basic examples:
#
@@ -246,7 +246,6 @@ module ActionView #:nodoc:
def render_file(template_path, use_full_path = true, local_assigns = {}) #:nodoc:
@first_render ||= template_path
template_path_without_extension, template_extension = path_and_extension(template_path)
-
if use_full_path
if template_extension
template_file_name = full_template_path(template_path_without_extension, template_extension)
@@ -280,7 +279,7 @@ module ActionView #:nodoc:
elsif options == :update
update_page(&block)
elsif options.is_a?(Hash)
- options = options.reverse_merge(:type => :rhtml, :locals => {}, :use_full_path => true)
+ options = options.reverse_merge(:type => :erb, :locals => {}, :use_full_path => true)
if options[:file]
render_file(options[:file], options[:use_full_path], options[:locals])
@@ -294,7 +293,7 @@ module ActionView #:nodoc:
end
end
- # Renders the +template+ which is given as a string as either rhtml or rxml depending on <tt>template_extension</tt>.
+ # Renders the +template+ which is given as a string as either erb or builder depending on <tt>template_extension</tt>.
# The hash in <tt>local_assigns</tt> is made available as local variables.
def render_template(template_extension, template, file_path = nil, local_assigns = {}) #:nodoc:
if handler = @@template_handlers[template_extension]
@@ -342,15 +341,21 @@ module ActionView #:nodoc:
def delegate_template_exists?(template_path)#:nodoc:
@@template_handlers.find { |k,| template_exists?(template_path, k) }
end
-
+
+ def one_of(template_path, *extensions)#:nodoc:
+ extensions.detect{|ext| template_exists?(template_path, ext)}
+ end
+
def erb_template_exists?(template_path)#:nodoc:
- template_exists?(template_path, :rhtml)
+ one_of(template_path, :erb, :rhtml)
end
-
+ alias :rhtml_template_exists? :erb_template_exists?
+
def builder_template_exists?(template_path)#:nodoc:
- template_exists?(template_path, :rxml)
+ one_of(template_path, :builder, :rxml)
end
-
+ alias :rxml_template_exists? :builder_template_exists?
+
def javascript_template_exists?(template_path)#:nodoc:
template_exists?(template_path, :rjs)
end
@@ -361,7 +366,7 @@ module ActionView #:nodoc:
template_exists?(template_file_name, template_file_extension)
else
cached_template_extension(template_path) ||
- %w(erb builder javascript delegate).any? do |template_type|
+ %w(erb rhtml builder rxml javascript delegate).any? do |template_type|
send("#{template_type}_template_exists?", template_path)
end
end
@@ -401,11 +406,11 @@ module ActionView #:nodoc:
def find_template_extension_for(template_path)
if match = delegate_template_exists?(template_path)
match.first.to_sym
- elsif erb_template_exists?(template_path): :rhtml
- elsif builder_template_exists?(template_path): :rxml
+ elsif extension = erb_template_exists?(template_path): extension
+ elsif extension = builder_template_exists?(template_path): extension
elsif javascript_template_exists?(template_path): :rjs
else
- raise ActionViewError, "No rhtml, rxml, rjs or delegate template found for #{template_path} in #{@view_paths.inspect}"
+ raise ActionViewError, "No erb, builder, rhtml, rxml, rjs or delegate template found for #{template_path} in #{@view_paths.inspect}"
end
end
@@ -464,7 +469,7 @@ module ActionView #:nodoc:
def create_template_source(extension, template, render_symbol, locals)
if template_requires_setup?(extension)
body = case extension.to_sym
- when :rxml
+ when :rxml, :builder
"controller.response.content_type ||= 'application/xml'\n" +
"xml = Builder::XmlMarkup.new(:indent => 2)\n" +
template
@@ -493,7 +498,7 @@ module ActionView #:nodoc:
end
def templates_requiring_setup
- %w(rxml rjs)
+ %w(builder rxml rjs)
end
def assign_method_name(extension, template, file_name)
@@ -523,7 +528,7 @@ module ActionView #:nodoc:
line_offset = @@template_args[render_symbol].size
if extension
case extension.to_sym
- when :rxml, :rjs
+ when :builder, :rxml, :rjs
line_offset += 2
end
end
diff --git a/actionpack/lib/action_view/helpers/capture_helper.rb b/actionpack/lib/action_view/helpers/capture_helper.rb
index 1d6ad56004..91cff4f981 100644
--- a/actionpack/lib/action_view/helpers/capture_helper.rb
+++ b/actionpack/lib/action_view/helpers/capture_helper.rb
@@ -14,7 +14,7 @@ module ActionView
# content_for("name") is a wrapper for capture which will
# make the fragment available by name to a yielding layout or template.
#
- # layout.rhtml:
+ # layout.erb:
#
# <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
# <head>
@@ -28,7 +28,7 @@ module ActionView
# </body>
# </html>
#
- # view.rhtml
+ # view.erb
#
# This page shows an alert box!
#
@@ -42,13 +42,13 @@ module ActionView
# instance variable. You can use this instance variable anywhere
# in your templates and even in your layout.
#
- # Example of capture being used in a .rhtml page:
+ # Example of capture being used in a .erb page:
#
# <% @greeting = capture do %>
# Welcome To my shiny new web page!
# <% end %>
#
- # Example of capture being used in a .rxml page:
+ # Example of capture being used in a .builder page:
#
# @greeting = capture do
# 'Welcome To my shiny new web page!'
diff --git a/actionpack/lib/action_view/partials.rb b/actionpack/lib/action_view/partials.rb
index 063ff5688d..c6049a4683 100644
--- a/actionpack/lib/action_view/partials.rb
+++ b/actionpack/lib/action_view/partials.rb
@@ -7,7 +7,7 @@ module ActionView
#
# <%= render :partial => "account" %>
#
- # This would render "advertiser/_account.rhtml" and pass the instance variable @account in as a local variable +account+ to
+ # This would render "advertiser/_account.erb" and pass the instance variable @account in as a local variable +account+ to
# the template for display.
#
# In another template for Advertiser#buy, we could have:
@@ -18,8 +18,8 @@ module ActionView
# <%= render :partial => "ad", :locals => { :ad => ad } %>
# <% end %>
#
- # This would first render "advertiser/_account.rhtml" with @buyer passed in as the local variable +account+, then render
- # "advertiser/_ad.rhtml" and pass the local variable +ad+ to the template for display.
+ # This would first render "advertiser/_account.erb" with @buyer passed in as the local variable +account+, then render
+ # "advertiser/_ad.erb" and pass the local variable +ad+ to the template for display.
#
# == Rendering a collection of partials
#
@@ -30,7 +30,7 @@ module ActionView
#
# <%= render :partial => "ad", :collection => @advertisements %>
#
- # This will render "advertiser/_ad.rhtml" and pass the local variable +ad+ to the template for display. An iteration counter
+ # This will render "advertiser/_ad.erb" and pass the local variable +ad+ to the template for display. An iteration counter
# will automatically be made available to the template with a name of the form +partial_name_counter+. In the case of the
# example above, the template would be fed +ad_counter+.
#
@@ -43,7 +43,7 @@ module ActionView
#
# <%= render :partial => "advertisement/ad", :locals => { :ad => @advertisement } %>
#
- # This will render the partial "advertisement/_ad.rhtml" regardless of which controller this is being called from.
+ # This will render the partial "advertisement/_ad.erb" regardless of which controller this is being called from.
module Partials
# Deprecated, use render :partial
def render_partial(partial_path, local_assigns = nil, deprecated_local_assigns = nil) #:nodoc:
diff --git a/actionpack/test/controller/layout_test.rb b/actionpack/test/controller/layout_test.rb
index cade6738a7..edf8f44b3a 100644
--- a/actionpack/test/controller/layout_test.rb
+++ b/actionpack/test/controller/layout_test.rb
@@ -110,7 +110,9 @@ class ExemptFromLayoutTest < Test::Unit::TestCase
def test_rhtml_exempt_from_layout_status_should_prevent_layout_render
ActionController::Base.exempt_from_layout :rhtml
+
assert @controller.send(:template_exempt_from_layout?, 'test.rhtml')
+ assert @controller.send(:template_exempt_from_layout?, 'hello.rhtml')
get :hello
assert_equal 'hello.rhtml', @response.body
diff --git a/actionpack/test/controller/new_render_test.rb b/actionpack/test/controller/new_render_test.rb
index d3967791a0..3eb84e6d24 100644
--- a/actionpack/test/controller/new_render_test.rb
+++ b/actionpack/test/controller/new_render_test.rb
@@ -60,12 +60,12 @@ class NewRenderTestController < ActionController::Base
def render_file_with_instance_variables
@secret = 'in the sauce'
- path = File.join(File.dirname(__FILE__), '../fixtures/test/render_file_with_ivar.rhtml')
+ path = File.join(File.dirname(__FILE__), '../fixtures/test/render_file_with_ivar.erb')
render :file => path
end
def render_file_with_locals
- path = File.join(File.dirname(__FILE__), '../fixtures/test/render_file_with_locals.rhtml')
+ path = File.join(File.dirname(__FILE__), '../fixtures/test/render_file_with_locals.erb')
render :file => path, :locals => {:secret => 'in the sauce'}
end
@@ -227,11 +227,11 @@ class NewRenderTestController < ActionController::Base
end
def hello_world_from_rxml_using_action
- render :action => "hello_world.rxml"
+ render :action => "hello_world.builder"
end
def hello_world_from_rxml_using_template
- render :template => "test/hello_world.rxml"
+ render :template => "test/hello_world.builder"
end
def head_with_location_header
diff --git a/actionpack/test/controller/rescue_test.rb b/actionpack/test/controller/rescue_test.rb
index f539546a1a..885ac0b8c0 100644
--- a/actionpack/test/controller/rescue_test.rb
+++ b/actionpack/test/controller/rescue_test.rb
@@ -111,7 +111,7 @@ class RescueTest < Test::Unit::TestCase
def test_rescue_action_locally
get :raises
assert_response :internal_server_error
- assert_template 'diagnostics.rhtml'
+ assert_template 'diagnostics.erb'
assert @response.body.include?('RescueController#raises'), "Response should include controller and action."
assert @response.body.include?("don't panic"), "Response should include exception message."
end
diff --git a/actionpack/test/fixtures/addresses/list.rhtml b/actionpack/test/fixtures/addresses/list.rhtml
index c75e01eece..e69de29bb2 100644
--- a/actionpack/test/fixtures/addresses/list.rhtml
+++ b/actionpack/test/fixtures/addresses/list.rhtml
@@ -1 +0,0 @@
-We only need to get this far!
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_cookies_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_cookies_ivar.rhtml
index 4e8a2d804c..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_cookies_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_cookies_ivar.rhtml
@@ -1 +0,0 @@
-<%= @cookies[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_cookies_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_cookies_method.rhtml
index 68e88bb746..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_cookies_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_cookies_method.rhtml
@@ -1 +0,0 @@
-<%= cookies[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_flash_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_flash_ivar.rhtml
index 4b4782b287..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_flash_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_flash_ivar.rhtml
@@ -1 +0,0 @@
-<%= @flash[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_flash_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_flash_method.rhtml
index f7f9d0913a..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_flash_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_flash_method.rhtml
@@ -1 +0,0 @@
-<%= flash[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_headers_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_headers_ivar.rhtml
index 1176c93ad8..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_headers_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_headers_ivar.rhtml
@@ -1 +0,0 @@
-<%= @headers[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_headers_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_headers_method.rhtml
index 308c4eb6ba..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_headers_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_headers_method.rhtml
@@ -1 +0,0 @@
-<%= headers[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_params_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_params_ivar.rhtml
index 1eea68757f..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_params_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_params_ivar.rhtml
@@ -1 +0,0 @@
-<%= @params[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_params_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_params_method.rhtml
index 7e349b4ca0..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_params_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_params_method.rhtml
@@ -1 +0,0 @@
-<%= params[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_request_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_request_ivar.rhtml
index a1680c23d5..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_request_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_request_ivar.rhtml
@@ -1 +0,0 @@
-<%= @request.method %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_request_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_request_method.rhtml
index 0c74cf1c1b..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_request_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_request_method.rhtml
@@ -1 +0,0 @@
-<%= request.method %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_response_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_response_ivar.rhtml
index 2f12d2ce32..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_response_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_response_ivar.rhtml
@@ -1 +0,0 @@
-<%= @response.body %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_response_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_response_method.rhtml
index 948c75929a..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_response_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_response_method.rhtml
@@ -1 +0,0 @@
-<%= response.body %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_session_ivar.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_session_ivar.rhtml
index 3acc1b8529..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_session_ivar.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_session_ivar.rhtml
@@ -1 +0,0 @@
-<%= @session[:test] %>
diff --git a/actionpack/test/fixtures/deprecated_instance_variables/_session_method.rhtml b/actionpack/test/fixtures/deprecated_instance_variables/_session_method.rhtml
index a899387c47..e69de29bb2 100644
--- a/actionpack/test/fixtures/deprecated_instance_variables/_session_method.rhtml
+++ b/actionpack/test/fixtures/deprecated_instance_variables/_session_method.rhtml
@@ -1 +0,0 @@
-<%= session[:test] %>
diff --git a/actionpack/test/fixtures/fun/games/hello_world.rhtml b/actionpack/test/fixtures/fun/games/hello_world.rhtml
index 1ebfbe2539..e69de29bb2 100644
--- a/actionpack/test/fixtures/fun/games/hello_world.rhtml
+++ b/actionpack/test/fixtures/fun/games/hello_world.rhtml
@@ -1 +0,0 @@
-Living in a nested world \ No newline at end of file
diff --git a/actionpack/test/fixtures/layouts/builder.rxml b/actionpack/test/fixtures/layouts/builder.rxml
index 729af4b8bc..e69de29bb2 100644
--- a/actionpack/test/fixtures/layouts/builder.rxml
+++ b/actionpack/test/fixtures/layouts/builder.rxml
@@ -1,3 +0,0 @@
-xml.wrapper do
- xml << @content_for_layout
-end \ No newline at end of file
diff --git a/actionpack/test/fixtures/layouts/standard.rhtml b/actionpack/test/fixtures/layouts/standard.rhtml
index 368764e6f4..e69de29bb2 100644
--- a/actionpack/test/fixtures/layouts/standard.rhtml
+++ b/actionpack/test/fixtures/layouts/standard.rhtml
@@ -1 +0,0 @@
-<html><%= @content_for_layout %><%= @variable_for_layout %></html> \ No newline at end of file
diff --git a/actionpack/test/fixtures/layouts/talk_from_action.rhtml b/actionpack/test/fixtures/layouts/talk_from_action.rhtml
index 187aab07a2..e69de29bb2 100644
--- a/actionpack/test/fixtures/layouts/talk_from_action.rhtml
+++ b/actionpack/test/fixtures/layouts/talk_from_action.rhtml
@@ -1,2 +0,0 @@
-<title><%= @title || @content_for_title %></title>
-<%= @content_for_layout -%> \ No newline at end of file
diff --git a/actionpack/test/fixtures/layouts/yield.rhtml b/actionpack/test/fixtures/layouts/yield.rhtml
index 482dc9022e..e69de29bb2 100644
--- a/actionpack/test/fixtures/layouts/yield.rhtml
+++ b/actionpack/test/fixtures/layouts/yield.rhtml
@@ -1,2 +0,0 @@
-<title><%= yield :title %></title>
-<%= yield %>
diff --git a/actionpack/test/fixtures/override/test/hello_world.rhtml b/actionpack/test/fixtures/override/test/hello_world.rhtml
index 3e308d3d86..e69de29bb2 100644
--- a/actionpack/test/fixtures/override/test/hello_world.rhtml
+++ b/actionpack/test/fixtures/override/test/hello_world.rhtml
@@ -1 +0,0 @@
-Hello overridden world! \ No newline at end of file
diff --git a/actionpack/test/fixtures/scope/test/modgreet.rhtml b/actionpack/test/fixtures/scope/test/modgreet.rhtml
index 8947726e89..e69de29bb2 100644
--- a/actionpack/test/fixtures/scope/test/modgreet.rhtml
+++ b/actionpack/test/fixtures/scope/test/modgreet.rhtml
@@ -1 +0,0 @@
-<p>Beautiful modules!</p> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_customer.rhtml b/actionpack/test/fixtures/test/_customer.rhtml
index 872d8c44e6..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_customer.rhtml
+++ b/actionpack/test/fixtures/test/_customer.rhtml
@@ -1 +0,0 @@
-Hello: <%= customer.name %> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_customer_greeting.rhtml b/actionpack/test/fixtures/test/_customer_greeting.rhtml
index 6acbcb20c4..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_customer_greeting.rhtml
+++ b/actionpack/test/fixtures/test/_customer_greeting.rhtml
@@ -1 +0,0 @@
-<%= greeting %>: <%= customer_greeting.name %> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_hash_object.rhtml b/actionpack/test/fixtures/test/_hash_object.rhtml
index 037a7368d6..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_hash_object.rhtml
+++ b/actionpack/test/fixtures/test/_hash_object.rhtml
@@ -1 +0,0 @@
-<%= hash_object[:first_name] %> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_hello.rxml b/actionpack/test/fixtures/test/_hello.rxml
index ef52f632d1..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_hello.rxml
+++ b/actionpack/test/fixtures/test/_hello.rxml
@@ -1 +0,0 @@
-xm.hello \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_partial_only.rhtml b/actionpack/test/fixtures/test/_partial_only.rhtml
index a44b3eed40..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_partial_only.rhtml
+++ b/actionpack/test/fixtures/test/_partial_only.rhtml
@@ -1 +0,0 @@
-only partial \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/_person.rhtml b/actionpack/test/fixtures/test/_person.rhtml
index b2e5688956..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/_person.rhtml
+++ b/actionpack/test/fixtures/test/_person.rhtml
@@ -1,2 +0,0 @@
-Second: <%= name %>
-Third: <%= @name %>
diff --git a/actionpack/test/fixtures/test/action_talk_to_layout.rhtml b/actionpack/test/fixtures/test/action_talk_to_layout.rhtml
index 36e896daa8..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/action_talk_to_layout.rhtml
+++ b/actionpack/test/fixtures/test/action_talk_to_layout.rhtml
@@ -1,2 +0,0 @@
-<% @title = "Talking to the layout" -%>
-Action was here! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/block_content_for.rhtml b/actionpack/test/fixtures/test/block_content_for.rhtml
index 9510337365..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/block_content_for.rhtml
+++ b/actionpack/test/fixtures/test/block_content_for.rhtml
@@ -1,2 +0,0 @@
-<% block_content_for :title do 'Putting stuff in the title!' end %>
-Great stuff! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/capturing.rhtml b/actionpack/test/fixtures/test/capturing.rhtml
index 1addaa40d9..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/capturing.rhtml
+++ b/actionpack/test/fixtures/test/capturing.rhtml
@@ -1,4 +0,0 @@
-<% days = capture do %>
- Dreamy days
-<% end %>
-<%= days %> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/content_for.rhtml b/actionpack/test/fixtures/test/content_for.rhtml
index 0e47ca8c3d..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/content_for.rhtml
+++ b/actionpack/test/fixtures/test/content_for.rhtml
@@ -1,2 +0,0 @@
-<% content_for :title do %>Putting stuff in the title!<% end %>
-Great stuff! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/dot.directory/render_file_with_ivar.rhtml b/actionpack/test/fixtures/test/dot.directory/render_file_with_ivar.rhtml
index 8b8a449236..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/dot.directory/render_file_with_ivar.rhtml
+++ b/actionpack/test/fixtures/test/dot.directory/render_file_with_ivar.rhtml
@@ -1 +0,0 @@
-The secret is <%= @secret %>
diff --git a/actionpack/test/fixtures/test/erb_content_for.rhtml b/actionpack/test/fixtures/test/erb_content_for.rhtml
index c3bdd13643..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/erb_content_for.rhtml
+++ b/actionpack/test/fixtures/test/erb_content_for.rhtml
@@ -1,2 +0,0 @@
-<% erb_content_for :title do %>Putting stuff in the title!<% end %>
-Great stuff! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/greeting.rhtml b/actionpack/test/fixtures/test/greeting.rhtml
index 62fb0293f0..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/greeting.rhtml
+++ b/actionpack/test/fixtures/test/greeting.rhtml
@@ -1 +0,0 @@
-<p>This is grand!</p>
diff --git a/actionpack/test/fixtures/test/hello.rxml b/actionpack/test/fixtures/test/hello.rxml
index 82a4a310d3..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello.rxml
+++ b/actionpack/test/fixtures/test/hello.rxml
@@ -1,4 +0,0 @@
-xml.html do
- xml.p "Hello #{@name}"
- xml << render_file("test/greeting")
-end \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/hello_world.rhtml b/actionpack/test/fixtures/test/hello_world.rhtml
index 6769dd60bd..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello_world.rhtml
+++ b/actionpack/test/fixtures/test/hello_world.rhtml
@@ -1 +0,0 @@
-Hello world! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/hello_world.rxml b/actionpack/test/fixtures/test/hello_world.rxml
index bffd2191ba..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello_world.rxml
+++ b/actionpack/test/fixtures/test/hello_world.rxml
@@ -1,3 +0,0 @@
-xml.html do
- xml.p "Hello"
-end \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/hello_world_container.rxml b/actionpack/test/fixtures/test/hello_world_container.rxml
index e48d75c405..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello_world_container.rxml
+++ b/actionpack/test/fixtures/test/hello_world_container.rxml
@@ -1,3 +0,0 @@
-xml.test do
- render :partial => 'hello', :locals => { :xm => xml }
-end \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/hello_world_with_layout_false.rhtml b/actionpack/test/fixtures/test/hello_world_with_layout_false.rhtml
index 6769dd60bd..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello_world_with_layout_false.rhtml
+++ b/actionpack/test/fixtures/test/hello_world_with_layout_false.rhtml
@@ -1 +0,0 @@
-Hello world! \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/hello_xml_world.rxml b/actionpack/test/fixtures/test/hello_xml_world.rxml
index 02b14fe87c..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/hello_xml_world.rxml
+++ b/actionpack/test/fixtures/test/hello_xml_world.rxml
@@ -1,11 +0,0 @@
-xml.html do
- xml.head do
- xml.title "Hello World"
- end
-
- xml.body do
- xml.p "abes"
- xml.p "monks"
- xml.p "wiseguys"
- end
-end \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/list.rhtml b/actionpack/test/fixtures/test/list.rhtml
index cd0ab45d26..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/list.rhtml
+++ b/actionpack/test/fixtures/test/list.rhtml
@@ -1 +0,0 @@
-<%= @test_unchanged = 'goodbye' %><%= render_collection_of_partials "customer", @customers %><%= @test_unchanged %>
diff --git a/actionpack/test/fixtures/test/non_erb_block_content_for.rxml b/actionpack/test/fixtures/test/non_erb_block_content_for.rxml
index 6ff6db0f95..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/non_erb_block_content_for.rxml
+++ b/actionpack/test/fixtures/test/non_erb_block_content_for.rxml
@@ -1,4 +0,0 @@
-content_for :title do
- 'Putting stuff in the title!'
-end
-xml << "\nGreat stuff!" \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/potential_conflicts.rhtml b/actionpack/test/fixtures/test/potential_conflicts.rhtml
index a5e964e359..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/potential_conflicts.rhtml
+++ b/actionpack/test/fixtures/test/potential_conflicts.rhtml
@@ -1,4 +0,0 @@
-First: <%= @name %>
-<%= render :partial => "person", :locals => { :name => "Stephan" } -%>
-Fourth: <%= @name %>
-Fifth: <%= name %> \ No newline at end of file
diff --git a/actionpack/test/fixtures/test/render_file_with_ivar.rhtml b/actionpack/test/fixtures/test/render_file_with_ivar.rhtml
index 8b8a449236..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/render_file_with_ivar.rhtml
+++ b/actionpack/test/fixtures/test/render_file_with_ivar.rhtml
@@ -1 +0,0 @@
-The secret is <%= @secret %>
diff --git a/actionpack/test/fixtures/test/render_file_with_locals.rhtml b/actionpack/test/fixtures/test/render_file_with_locals.rhtml
index ebe09faee6..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/render_file_with_locals.rhtml
+++ b/actionpack/test/fixtures/test/render_file_with_locals.rhtml
@@ -1 +0,0 @@
-The secret is <%= secret %>
diff --git a/actionpack/test/fixtures/test/render_to_string_test.rhtml b/actionpack/test/fixtures/test/render_to_string_test.rhtml
index 6e267e8634..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/render_to_string_test.rhtml
+++ b/actionpack/test/fixtures/test/render_to_string_test.rhtml
@@ -1 +0,0 @@
-The value of foo is: ::<%= @foo %>::
diff --git a/actionpack/test/fixtures/test/update_element_with_capture.rhtml b/actionpack/test/fixtures/test/update_element_with_capture.rhtml
index fa3ef200f9..e69de29bb2 100644
--- a/actionpack/test/fixtures/test/update_element_with_capture.rhtml
+++ b/actionpack/test/fixtures/test/update_element_with_capture.rhtml
@@ -1,9 +0,0 @@
-<% replacement_function = update_element_function("products", :action => :update) do %>
- <p>Product 1</p>
- <p>Product 2</p>
-<% end %>
-<%= javascript_tag(replacement_function) %>
-
-<% update_element_function("status", :action => :update, :binding => binding) do %>
- <b>You bought something!</b>
-<% end %>