From 80cd16372ce1776e5d6af87b2dc7be8205fa1513 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Valim?= <jose.valim@gmail.com>
Date: Wed, 1 Jul 2009 18:33:22 +0200
Subject: Added scaffold views to scaffold controller.

---
 .../generators/erb/scaffold/scaffold_generator.rb  | 42 ++++++++++++++++++++++
 .../erb/scaffold/templates/edit.html.erb           | 18 ++++++++++
 .../erb/scaffold/templates/index.html.erb          | 24 +++++++++++++
 .../erb/scaffold/templates/layout.html.erb         | 17 +++++++++
 .../generators/erb/scaffold/templates/new.html.erb | 17 +++++++++
 .../erb/scaffold/templates/show.html.erb           | 10 ++++++
 railties/lib/generators/generated_attribute.rb     |  4 +++
 railties/lib/generators/named_base.rb              |  5 +--
 .../rails/scaffold/scaffold_generator.rb           |  2 ++
 .../scaffold_controller_generator.rb               | 10 ++++--
 .../scaffold_controller/templates/controller.rb    |  2 ++
 11 files changed, 147 insertions(+), 4 deletions(-)
 create mode 100644 railties/lib/generators/erb/scaffold/scaffold_generator.rb
 create mode 100644 railties/lib/generators/erb/scaffold/templates/edit.html.erb
 create mode 100644 railties/lib/generators/erb/scaffold/templates/index.html.erb
 create mode 100644 railties/lib/generators/erb/scaffold/templates/layout.html.erb
 create mode 100644 railties/lib/generators/erb/scaffold/templates/new.html.erb
 create mode 100644 railties/lib/generators/erb/scaffold/templates/show.html.erb

(limited to 'railties')

diff --git a/railties/lib/generators/erb/scaffold/scaffold_generator.rb b/railties/lib/generators/erb/scaffold/scaffold_generator.rb
new file mode 100644
index 0000000000..c89af042b7
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/scaffold_generator.rb
@@ -0,0 +1,42 @@
+require 'generators/erb'
+
+module Erb
+  module Generators
+    class ScaffoldGenerator < Base
+      include Rails::Generators::ControllerNamedBase
+
+      argument :attributes, :type => :hash, :default => {}, :banner => "field:type field:type"
+      class_option :singleton, :type => :boolean, :desc => "Supply to skip index action"
+
+      # TODO Spec me
+      def copy_index_file
+        return if options[:singleton]
+        copy_view :index
+      end
+
+      def copy_edit_file
+        copy_view :edit
+      end
+
+      def copy_show_file
+        copy_view :show
+      end
+
+      def copy_new_file
+        copy_view :new
+      end
+
+      # TODO invoke_if?
+      def copy_layout_file
+        template "layout.html.erb", File.join("app", "views", "layouts", controller_class_path, "#{controller_file_name}.html.erb")
+      end
+
+      protected
+
+        def copy_view(view)
+          template "#{view}.html.erb", File.join("app", "views", controller_file_path, "#{view}.html.erb")
+        end
+
+    end
+  end
+end
diff --git a/railties/lib/generators/erb/scaffold/templates/edit.html.erb b/railties/lib/generators/erb/scaffold/templates/edit.html.erb
new file mode 100644
index 0000000000..cca1d61c68
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/templates/edit.html.erb
@@ -0,0 +1,18 @@
+<h1>Editing <%= singular_name %></h1>
+
+<%% form_for(@<%= singular_name %>) do |f| %>
+  <%%= f.error_messages %>
+
+<% for attribute in attributes -%>
+  <p>
+    <%%= f.label :<%= attribute.name %> %><br />
+    <%%= f.<%= attribute.field_type %> :<%= attribute.name %> %>
+  </p>
+<% end -%>
+  <p>
+    <%%= f.submit 'Update' %>
+  </p>
+<%% end %>
+
+<%%= link_to 'Show', @<%= singular_name %> %> |
+<%%= link_to 'Back', <%= plural_name %>_path %>
\ No newline at end of file
diff --git a/railties/lib/generators/erb/scaffold/templates/index.html.erb b/railties/lib/generators/erb/scaffold/templates/index.html.erb
new file mode 100644
index 0000000000..427ff81e03
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/templates/index.html.erb
@@ -0,0 +1,24 @@
+<h1>Listing <%= plural_name %></h1>
+
+<table>
+  <tr>
+<% for attribute in attributes -%>
+    <th><%= attribute.human_name %></th>
+<% end -%>
+  </tr>
+
+<%% @<%= plural_name %>.each do |<%= singular_name %>| %>
+  <tr>
+<% for attribute in attributes -%>
+    <td><%%=h <%= singular_name %>.<%= attribute.name %> %></td>
+<% end -%>
+    <td><%%= link_to 'Show', <%= singular_name %> %></td>
+    <td><%%= link_to 'Edit', edit_<%= singular_name %>_path(<%= singular_name %>) %></td>
+    <td><%%= link_to 'Destroy', <%= singular_name %>, :confirm => 'Are you sure?', :method => :delete %></td>
+  </tr>
+<%% end %>
+</table>
+
+<br />
+
+<%%= link_to 'New <%= singular_name %>', new_<%= singular_name %>_path %>
diff --git a/railties/lib/generators/erb/scaffold/templates/layout.html.erb b/railties/lib/generators/erb/scaffold/templates/layout.html.erb
new file mode 100644
index 0000000000..ebc97f8130
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/templates/layout.html.erb
@@ -0,0 +1,17 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
+       "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+  <meta http-equiv="content-type" content="text/html;charset=UTF-8" />
+  <title><%= controller_class_name %>: <%%= controller.action_name %></title>
+  <%%= stylesheet_link_tag 'scaffold' %>
+</head>
+<body>
+
+<p style="color: green"><%%= flash[:notice] %></p>
+
+<%%= yield %>
+
+</body>
+</html>
diff --git a/railties/lib/generators/erb/scaffold/templates/new.html.erb b/railties/lib/generators/erb/scaffold/templates/new.html.erb
new file mode 100644
index 0000000000..96c89fc50e
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/templates/new.html.erb
@@ -0,0 +1,17 @@
+<h1>New <%= singular_name %></h1>
+
+<%% form_for(@<%= singular_name %>) do |f| %>
+  <%%= f.error_messages %>
+
+<% for attribute in attributes -%>
+  <p>
+    <%%= f.label :<%= attribute.name %> %><br />
+    <%%= f.<%= attribute.field_type %> :<%= attribute.name %> %>
+  </p>
+<% end -%>
+  <p>
+    <%%= f.submit 'Create' %>
+  </p>
+<%% end %>
+
+<%%= link_to 'Back', <%= plural_name %>_path %>
\ No newline at end of file
diff --git a/railties/lib/generators/erb/scaffold/templates/show.html.erb b/railties/lib/generators/erb/scaffold/templates/show.html.erb
new file mode 100644
index 0000000000..adecaf70c6
--- /dev/null
+++ b/railties/lib/generators/erb/scaffold/templates/show.html.erb
@@ -0,0 +1,10 @@
+<% for attribute in attributes -%>
+<p>
+  <b><%= attribute.column.human_name %>:</b>
+  <%%=h @<%= singular_name %>.<%= attribute.name %> %>
+</p>
+
+<% end -%>
+
+<%%= link_to 'Edit', edit_<%= singular_name %>_path(@<%= singular_name %>) %> |
+<%%= link_to 'Back', <%= plural_name %>_path %>
\ No newline at end of file
diff --git a/railties/lib/generators/generated_attribute.rb b/railties/lib/generators/generated_attribute.rb
index 7cd1fed8a4..35dac99dbe 100644
--- a/railties/lib/generators/generated_attribute.rb
+++ b/railties/lib/generators/generated_attribute.rb
@@ -35,6 +35,10 @@ module Rails
         end
       end
 
+      def human_name
+        name.to_s.humanize
+      end
+
       def reference?
         [ :references, :belongs_to ].include?(self.type)
       end
diff --git a/railties/lib/generators/named_base.rb b/railties/lib/generators/named_base.rb
index 54f832a9bd..fd7b8369c4 100644
--- a/railties/lib/generators/named_base.rb
+++ b/railties/lib/generators/named_base.rb
@@ -96,7 +96,8 @@ module Rails
     #
     module ControllerNamedBase
       def self.included(base) #:nodoc:
-        base.send :attr_reader, :controller_name, :controller_class_name, :controller_file_name
+        base.send :attr_reader, :controller_name, :controller_class_name, :controller_file_name,
+                                :controller_class_path, :controller_file_path
       end
 
       # Set controller variables on initialization.
@@ -105,7 +106,7 @@ module Rails
         super
         @controller_name = name.pluralize
 
-        base_name, class_path, file_path, class_nesting, class_nesting_depth = extract_modules(@controller_name)
+        base_name, @controller_class_path, @controller_file_path, class_nesting, class_nesting_depth = extract_modules(@controller_name)
         class_name_without_nesting, @controller_file_name, controller_plural_name = inflect_names(base_name)
 
         @controller_class_name = if class_nesting.empty?
diff --git a/railties/lib/generators/rails/scaffold/scaffold_generator.rb b/railties/lib/generators/rails/scaffold/scaffold_generator.rb
index 874ef98057..be86de7572 100644
--- a/railties/lib/generators/rails/scaffold/scaffold_generator.rb
+++ b/railties/lib/generators/rails/scaffold/scaffold_generator.rb
@@ -7,6 +7,8 @@ module Rails
 
       remove_hook_for :actions, :resource_controller
       hook_for :scaffold_controller, :required => true
+
+      # invoke_if stylesheets
     end
   end
 end
diff --git a/railties/lib/generators/rails/scaffold_controller/scaffold_controller_generator.rb b/railties/lib/generators/rails/scaffold_controller/scaffold_controller_generator.rb
index 181d21ba35..7fbbf90105 100644
--- a/railties/lib/generators/rails/scaffold_controller/scaffold_controller_generator.rb
+++ b/railties/lib/generators/rails/scaffold_controller/scaffold_controller_generator.rb
@@ -5,14 +5,19 @@ module Rails
 
       check_class_collision :suffix => "Controller"
       class_option :orm, :desc => "ORM to generate the controller for", :banner => "NAME", :type => :string
-      class_option :singleton, :type => :boolean, :desc => "Supply to create a singleton controller"
+      class_option :singleton, :type => :boolean, :desc => "Supply to create a singleton controller" # TODO Spec me
 
       def create_controller_files
         template 'controller.rb', File.join('app/controllers', class_path, "#{file_name}_controller.rb")
       end
 
       hook_for :template_engine, :test_framework, :as => :scaffold
-      invoke_if :helper
+
+      # Invoke the helper using the controller (pluralized) name.
+      #
+      invoke_if :helper do |base, invoked|
+        base.invoke invoked, [ base.controller_name ]
+      end
 
       protected
 
@@ -23,6 +28,7 @@ module Rails
         def orm_instance
           @orm_instance ||= @orm_class.new(file_name)
         end
+
     end
   end
 end
diff --git a/railties/lib/generators/rails/scaffold_controller/templates/controller.rb b/railties/lib/generators/rails/scaffold_controller/templates/controller.rb
index 307638acf8..3cc8bbf8e7 100644
--- a/railties/lib/generators/rails/scaffold_controller/templates/controller.rb
+++ b/railties/lib/generators/rails/scaffold_controller/templates/controller.rb
@@ -1,4 +1,5 @@
 class <%= controller_class_name %>Controller < ApplicationController
+<% unless options[:singleton] -%>
   # GET /<%= table_name %>
   # GET /<%= table_name %>.xml
   def index
@@ -9,6 +10,7 @@ class <%= controller_class_name %>Controller < ApplicationController
       format.xml  { render :xml => @<%= table_name %> }
     end
   end
+<% end -%>
 
   # GET /<%= table_name %>/1
   # GET /<%= table_name %>/1.xml
-- 
cgit v1.2.3