view_mapper 0.2.0 → 0.3.0

data/lib/view_mapper/model_info.rb

@@ -0,0 +1,157 @@
+module ViewMapper
+  class ModelInfo
+
+    attr_reader :model
+    attr_reader :name
+    attr_reader :attributes
+    attr_reader :error
+
+    def initialize(model_name)
+      @model = find_model(model_name)
+      @name = model.to_s unless model.nil?
+    end
+
+    def valid?
+      error.nil?
+    end
+
+    def columns
+      @columns ||= active_record_columns.collect { |col| col.name }
+    end
+
+    def attributes
+      @attributes ||= active_record_columns.collect { |col| Rails::Generator::GeneratedAttribute.new col.name, col.type }
+    end
+
+    def attachments
+      @attachments = find_attachments
+    end
+
+    def has_attachment?(attachment)
+      attachments.include?(attachment)
+    end
+
+    def has_columns_for_attachment?(attachment)
+      !paperclip_columns_for_attachment(attachment).detect { |paperclip_col| !has_column_for_attachment(attachment, paperclip_col) }
+    end
+
+    def child_models
+      model.reflections.select { |key, value| value.macro == :has_many }.collect do |kvpair|
+        kvpair[0].to_s.singularize
+      end.sort.collect do |model_name|
+        ModelInfo.new model_name
+      end
+    end
+
+    def accepts_nested_attributes_for?(child_model)
+      if !model.new.methods.include? "#{child_model.name.underscore.pluralize}_attributes="
+        @error = "Model #{model} does not accept nested attributes for model #{child_model.name}."
+        false
+      else
+        true
+      end
+    end
+
+    def belongs_to?(parent_model_name)
+      has_association_for? :belongs_to, parent_model_name
+    end
+
+    def has_many?(child_model_name)
+      has_association_for? :has_many, child_model_name
+    end
+
+    def has_and_belongs_to_many?(model_name)
+      has_association_for? :has_and_belongs_to_many, model_name
+    end
+
+    def has_foreign_key_for?(parent_model_name)
+      model.columns.detect { |col| is_foreign_key_for?(col, parent_model_name) }
+    end
+
+    private
+
+    def find_model(model_name)
+      model = nil
+      begin
+        model = Object.const_get model_name.camelize
+        if !model.new.kind_of? ActiveRecord::Base
+          @error = "Class '#{model_name}' is not an ActiveRecord::Base."
+          model = nil
+        end
+      rescue NameError
+        @error = "Class '#{model_name}' does not exist or contains a syntax error and could not be loaded."
+      rescue ActiveRecord::StatementInvalid
+        @error = "Table for model '#{model_name}' does not exist - run rake db:migrate first."
+      end
+      model
+    end
+
+    def active_record_columns
+      @active_record_columns ||= inspect_active_record_columns
+    end
+
+    def inspect_active_record_columns
+      model.columns.reject do |col|
+        is_timestamp?(col) || is_primary_key?(col) || is_foreign_key?(col) || is_paperclip_column?(col)
+      end
+    end
+
+    def is_timestamp?(col)
+      %w{ updated_at updated_on created_at created_on }.include? col.name
+    end
+
+    def is_primary_key?(col)
+      col.name == model.primary_key
+    end
+
+    def is_foreign_key?(col)
+      model.reflections.values.detect do |reflection|
+        col.name == reflection.primary_key_name
+      end
+    end
+
+    def is_foreign_key_for?(col, parent_model_name)
+      model.reflections.values.detect do |reflection|
+        col.name == reflection.primary_key_name && reflection.name == parent_model_name.underscore.to_sym
+      end
+    end
+
+    def has_association_for?(association, model_name)
+      !model.reflections.values.detect do |reflection|
+        reflection.name == model_name.underscore.to_sym && reflection.macro == association
+      end.nil?
+    end
+
+    def is_paperclip_column?(col)
+      paperclip_columns.include?(col.name)
+    end
+
+    def find_attachments
+      if model.respond_to?('attachment_definitions') && model.attachment_definitions
+        model.attachment_definitions.keys.collect(&:to_s).sort
+      else
+        []
+      end
+    end
+
+    def has_column_for_attachment(attachment, paperclip_col)
+      has_column = model.columns.map {|col| col.name}.include?(paperclip_col)
+      if !has_column
+        @error = "Column \'#{paperclip_col}\' does not exist. First run script/generate paperclip #{name.downcase} #{attachment}."
+      end
+      has_column
+    end
+
+    def paperclip_columns
+      @paperclip_columns ||= attachments.inject([]) do |result, element|
+        result + paperclip_columns_for_attachment(element)
+      end
+    end
+
+    def paperclip_columns_for_attachment(attachment)
+      %w{ file_name content_type file_size updated_at }.collect do |col|
+        "#{attachment}_#{col}"
+      end
+    end
+  end
+end

data/lib/view_mapper/paperclip_view.rb

@@ -38,7 +38,7 @@ module ViewMapper
       if view_param
         parse_attachments_from_param
       elsif view_only?
-        inspect_model_for_attachments
+        model.attachments
       else
         []
       end
@@ -48,14 +48,6 @@ module ViewMapper
       view_param.split(',')
     end
 
-    def inspect_model_for_attachments
-      if model.respond_to?('attachment_definitions') && model.attachment_definitions
-        model.attachment_definitions.keys.collect { |name| name.to_s }.sort
-      else
-        []
-      end
-    end
-
     def validate
       @valid = validate_attachments
       @valid &&= super
@@ -65,7 +57,7 @@ module ViewMapper
       if !paperclip_installed
         logger.error "The Paperclip plugin does not appear to be installed."
         return false
-      elsif attachments == []
+      elsif attachments.empty?
         if view_only?
           logger.warning "No paperclip attachments exist on the specified class."
         else
@@ -79,44 +71,17 @@ module ViewMapper
 
     def validate_attachment(attachment)
       if view_only?
-        if !has_attachment(attachment.to_sym)
+        if !model.has_attachment?(attachment)
           logger.error "Attachment '#{attachment}' does not exist."
           return false
-        elsif !has_columns_for_attachment(attachment)
+        elsif !model.has_columns_for_attachment?(attachment)
+          logger.error model.error
           return false
         end
       end
       true
     end
 
-    def has_attachment(attachment)
-      model.attachment_definitions && model.attachment_definitions.has_key?(attachment)
-    end
-
-    def has_columns_for_attachment(attachment)
-      !paperclip_columns_for_attachment(attachment).detect { |paperclip_col| !has_column_for_attachment(attachment, paperclip_col) }
-    end
-
-    def has_column_for_attachment(attachment, paperclip_col)
-      has_column = model.columns.collect { |col| col.name }.include?(paperclip_col)
-      if !has_column
-        logger.error "Column \'#{paperclip_col}\' does not exist. First run script/generate paperclip #{name} #{attachment}."
-      end
-      has_column
-    end
-
-    def built_in_columns
-      attachments.inject(super) do |result, element|
-        result + paperclip_columns_for_attachment(element)
-      end
-    end
-
-    def paperclip_columns_for_attachment(attachment)
-      %w{ file_name content_type file_size updated_at }.collect do |col|
-        "#{attachment}_#{col}"
-      end
-    end
-
     def paperclip_installed
       ActiveRecord::Base.methods.include? 'has_attached_file'
     end

data/lib/view_mapper/view_mapper.rb

@@ -6,12 +6,12 @@ module ViewMapper
     Rails::Generator::Commands::Destroy.class_eval { include RouteAction::Destroy }
     super
     if options[:view]
-      self.extend(view_class)
+      self.extend(view_module)
       @source_root = source_root_for_view
     end
   end
 
-  def view_class
+  def view_module
     "ViewMapper::#{view_name.camelize}View".constantize
   end
 

data/lib/view_mapper.rb

@@ -1,5 +1,7 @@
 require 'view_mapper/auto_complete_view'
 require 'view_mapper/paperclip_view'
+require 'view_mapper/has_many_view'
 require 'view_mapper/route_action'
 require 'view_mapper/editable_manifest'
+require 'view_mapper/model_info'
 require 'view_mapper/view_mapper'

data/test/expected_templates/has_many/_form.html.erb

@@ -0,0 +1,26 @@
+<%= f.error_messages %>
+
+<p>
+  <%= f.label :name %><br />
+  <%= f.text_field :name %>
+</p>
+
+<div id='some_other_model_children'>
+  <% f.fields_for :some_other_models do |some_other_model_form| %>
+    <%= render :partial => 'some_other_model', :locals => { :f => some_other_model_form } %>
+  <% end %>
+</div>
+
+<p>
+  <%= add_child_link 'Add a SomeOtherModel', 'some_other_model', f %>
+</p>
+
+<div id='testy_children'>
+  <% f.fields_for :testies do |testy_form| %>
+    <%= render :partial => 'testy', :locals => { :f => testy_form } %>
+  <% end %>
+</div>
+
+<p>
+  <%= add_child_link 'Add a Testy', 'testy', f %>
+</p>

data/test/expected_templates/has_many/_person.html.erb

@@ -0,0 +1,18 @@
+<div class="child">
+  <p>
+    Testy <%= f.label :first_name %><br />
+    <%= f.text_field :first_name %>
+  </p>
+  <p>
+    Testy <%= f.label :last_name %><br />
+    <%= f.text_field :last_name %>
+  </p>
+  <p>
+    Testy <%= f.label :address %><br />
+    <%= f.text_field :address %>
+  </p>
+  <p>
+    <%= f.hidden_field :_delete, :class => 'delete' %>
+    <%= remove_child_link 'remove', f %>
+  </p>
+</div>

data/test/expected_templates/has_many/create_parents.rb

@@ -0,0 +1,15 @@
+class CreateParents < ActiveRecord::Migration
+  def self.up
+    create_table :parents do |t|
+      t.string   :first_name
+      t.string   :last_name
+      t.string   :address
+
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :parents
+  end
+end

data/test/expected_templates/has_many/edit.html.erb

@@ -0,0 +1,11 @@
+<h1>Editing parent</h1>
+
+<% form_for(@parent) do |f| %>
+  <%= render :partial => 'form', :locals => { :f => f } %>
+  <p>
+    <%= f.submit 'Update' %>
+  </p>
+<% end %>
+
+<%= link_to 'Show', @parent %> |
+<%= link_to 'Back', parents_path %>

data/test/expected_templates/has_many/index.html.erb

@@ -0,0 +1,20 @@
+<h1>Listing parents</h1>
+
+<table>
+  <tr>
+    <th>Name</th>
+  </tr>
+
+<% @parents.each do |parent| %>
+  <tr>
+    <td><%=h parent.name %></td>
+    <td><%= link_to 'Show', parent %></td>
+    <td><%= link_to 'Edit', edit_parent_path(parent) %></td>
+    <td><%= link_to 'Destroy', parent, :confirm => 'Are you sure?', :method => :delete %></td>
+  </tr>
+<% end %>
+</table>
+
+<br />
+
+<%= link_to 'New parent', new_parent_path %>

data/test/expected_templates/has_many/new.html.erb

@@ -0,0 +1,10 @@
+<h1>New parent</h1>
+
+<% form_for(@parent) do |f| %>
+  <%= render :partial => 'form', :locals => { :f => f } %>
+  <p>
+    <%= f.submit 'Create' %>
+  </p>
+<% end %>
+
+<%= link_to 'Back', parents_path %>

data/test/expected_templates/has_many/parent.rb

@@ -0,0 +1,14 @@
+class Parent < ActiveRecord::Base
+  has_many :some_other_models
+  has_many :testies
+  accepts_nested_attributes_for :some_other_models,
+                                :allow_destroy => true,
+                                :reject_if => proc { |attrs| attrs['name'].blank? }
+  accepts_nested_attributes_for :testies,
+                                :allow_destroy => true,
+                                :reject_if => proc { |attrs|
+                                  attrs['first_name'].blank? &&
+                                  attrs['last_name'].blank? &&
+                                  attrs['address'].blank?
+                                }
+end

data/test/expected_templates/has_many/show.html.erb

@@ -0,0 +1,33 @@
+<p>
+  <b>Name:</b>
+  <%=h @parent.name %>
+</p>
+
+<% @parent.some_other_models.each do |some_other_model| %>
+  <div class="child">
+    <p>
+      <b>SomeOtherModel Name:</b>
+      <%=h some_other_model.name %>
+    </p>
+  </div>
+<% end %>
+
+<% @parent.testies.each do |testy| %>
+  <div class="child">
+    <p>
+      <b>Testy First name:</b>
+      <%=h testy.first_name %>
+    </p>
+    <p>
+      <b>Testy Last name:</b>
+      <%=h testy.last_name %>
+    </p>
+    <p>
+      <b>Testy Address:</b>
+      <%=h testy.address %>
+    </p>
+  </div>
+<% end %>
+
+<%= link_to 'Edit', edit_parent_path(@parent) %> |
+<%= link_to 'Back', parents_path %>