Added dependencies to db and view

Author: cmer

app/helpers/codes_helper.rb

@@ -28,4 +28,16 @@ def author_links(authors)
       link_to(h(author.name), author.permalink)
     end.to_sentence
   end
+  
+  def dependency_link_for(gem)
+    "<a href=\"/#{gem.downcase}\">#{h(gem)}</a>"
+  end
+  
+  def dependency_links_for(code)
+    links = []
+    (code.dependencies || []).each do |d|
+      links << dependency_link_for(d.gem)
+    end
+    links
+  end
 end

app/models/code.rb

@@ -4,6 +4,7 @@ class Code < ActiveRecord::Base
   has_many :comments, :dependent => :destroy, :order => 'created_at desc'
   has_many :working_comments, :class_name => 'Comment', :conditions => { :works_for_me => true }
   has_many :failure_comments, :class_name => 'Comment', :conditions => { :works_for_me => false }
+  has_many :dependencies
 
   named_scope :popular, :order => 'comments_count desc'
   named_scope :unpopular, :order => 'comments_count asc'

app/models/dependency.rb

@@ -0,0 +1,3 @@
+class Dependency < ActiveRecord::Base
+  belongs_to :code
+end

app/views/codes/show.html.erb

@@ -7,6 +7,11 @@
   <dd><%= author_links(@code.authors) %></dd>
   <dt>Summary</dt>
   <dd><%= @code.description_or_summary %></dd>
+<% if @code.dependencies and @code.dependencies.size > 0 %>
+  <dt>Dependencies</dt>
+  <dd><%= dependency_links_for(@code).join(", ") %></dd>
+<% end %>
+
 <% unless @code.c_extension.nil? %>
     <dt class="notes">Notes</dt>
 <% 
@@ -16,7 +21,7 @@ end
 if @code.c_extension == true %>
   <dd><span class="fails stronger normalcase"><%= @code.name.capitalize %> appears to contain C EXTENSIONS, which are not supported by JRuby.</span></dd>
 <% elsif @code.c_extension == false %>
-<dd><span class="works stronger normalcase">Good news! <%= @code.name.capitalize %> does not appear to contain C extensions.</span></dd>
+  <dd><span class="works stronger normalcase">Good news! <%= @code.name.capitalize %> does not appear to contain C extensions.</span></dd>
 <% end %>
 </dl>
 

db/migrate/20090528020846_create_dependencies.rb

@@ -0,0 +1,12 @@
+class CreateDependencies < ActiveRecord::Migration
+  def self.up
+    create_table :dependencies do |t|
+      t.integer :code_id, :null => false
+      t.string  :gem,     :null => false
+    end
+  end
+
+  def self.down
+    drop_table :dependencies
+  end
+end

db/schema.rb

@@ -9,7 +9,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20090527235046) do
+ActiveRecord::Schema.define(:version => 20090528020846) do
 
   create_table "authors", :force => true do |t|
     t.string   "name"
@@ -69,6 +69,11 @@
   add_index "comments", ["code_id"], :name => "index_comments_on_code_id"
   add_index "comments", ["created_at"], :name => "index_comments_on_created_at"
 
+  create_table "dependencies", :force => true do |t|
+    t.integer "code_id", :null => false
+    t.string  "gem",     :null => false
+  end
+
   create_table "jruby_versions", :force => true do |t|
     t.string "version"
   end

lib/gem_property.rb

@@ -1,11 +1,16 @@
 class GemProperty
-  UNSUPPORTED_DEPENDENCIES = %w{ rubyinline eventmachine }
+  UNSUPPORTED_DEPENDENCIES = %w{ rubyinline }
 
   def self.set(code)
     raise(ArgumentError, "Must provide an instance of Code") unless code.is_a?(Code)
     g = GemProperty.new(code.slug_name)
     p = g.properties
     code.c_extension = p[:contains_c_extension]
+    
+    (p[:dependencies] || []).each do |d|
+      code.dependencies << Dependency.new(:gem=>d)
+    end
+    
     code
   end
 
@@ -27,7 +32,8 @@ def properties
       extract_gem
 
       output = {
-        :contains_c_extension     => contains_c_extension?
+        :contains_c_extension     => contains_c_extension?,
+        :dependencies             => dependencies
       }
     end
 
@@ -48,6 +54,7 @@ def clean_up(path)
 
     def download_gem
       output = `gem fetch #{@gem}`
+
       if output.match(/^Downloaded /)
         @file = output.split(" ")[1]
         return @file
@@ -76,4 +83,11 @@ def contains_c_extension?
       false
     end
 
+    def dependencies
+      output = []
+      gem_specs.dependencies.each do |d|
+        output << d.name
+      end
+      output
+    end
 end

public/stylesheets/styles.css

@@ -299,7 +299,7 @@ dt {
 }
 
 dt.notes {
-	padding-bottom: 7px;
+	padding-bottom: 2px;
 }
 
 dd {

test/fixtures/dependencies.yml

@@ -0,0 +1,7 @@
+# Read about fixtures at http://ar.rubyonrails.org/classes/Fixtures.html
+
+# one:
+#   column: value
+#
+# two:
+#   column: value

test/unit/dependency_test.rb

@@ -0,0 +1,8 @@
+require 'test_helper'
+
+class DependencyTest < ActiveSupport::TestCase
+  # Replace this with your real tests.
+  test "the truth" do
+    assert true
+  end
+end

test/unit/gem_property_test.rb

@@ -2,8 +2,8 @@
 
 class GemPropertyTest < Test::Unit::TestCase
   def test_with_real_gems
-    gems = [  [ "eventmachine", {:contains_c_extension=>true} ],
-              [ "tzinfo",       {:contains_c_extension=>false} ] ]
+    gems = [  [ "thin",    { :contains_c_extension=>true,  :dependencies => %w{ rack eventmachine daemons} } ],
+              [ "tzinfo",  { :contains_c_extension=>false, :dependencies => [] } ] ]
 
     gems.each do |gem|
       gem_name = gem[0]