Merge pull request #3 from bad6e/add_bd

Add bd

Author: bad6e

app/controllers/application_controller.rb

@@ -2,8 +2,4 @@ class ApplicationController < ActionController::Base
   # Prevent CSRF attacks by raising an exception.
   # For APIs, you may want to use :null_session instead.
   protect_from_forgery with: :exception
-
-  require 'nokogiri'
-  require 'open-uri'
-
 end

app/controllers/stats_controller.rb

@@ -0,0 +1,12 @@
+class StatsController < ApplicationController
+
+  def index
+    @stats = Stat.order(commits: :desc)
+  end
+
+  private
+
+    def commit_params
+      params.require(:record).permit(:name, :commits)
+    end
+end

app/controllers/welcome_controller.rb

@@ -1,7 +1,5 @@
-class Welcome
-
-  def initialize
-    @urls =["https://github.com/bad6e",
+module GlobalConstants
+URLS = ["https://github.com/bad6e",
             "https://github.com/roseak",
             "https://github.com/mcschatz",
             "https://github.com/travishaby",
@@ -21,26 +19,4 @@ def initialize
             "https://github.com/adamki",
             "https://github.com/plato721",
             "https://github.com/Egogre"]
-  end
-
-  def sort
-    join_arrays = get_names.zip(get_count)
-    join_arrays.sort{|a,b| b[1] <=> a[1]}
-  end
-
-  def get_names
-    @urls.map do |url|
-      open_url(url).at_css(".vcard-fullname").text.strip
-    end
-  end
-
-  def get_count
-    @urls.map do |url|
-      open_url(url).at_css(".contrib-number").text.gsub("total","").to_i
-    end
-  end
-
-  def open_url(url)
-    Nokogiri::HTML(open(url))
-  end
 end

app/helpers/welcome_helper.rb

@@ -0,0 +1,4 @@
+class Stat < ActiveRecord::Base
+  validates :name, presence: true
+  validates :commits, :numericality => { :greater_than => 0}, presence: true
+end

app/models/welcome.rb app/models/global_constants.rbapp/models/welcome.rb renamed to app/models/global_constants.rb

@@ -73,17 +73,16 @@
           <h3 id="leaderboard">Leaderboard</h3>
             <table class="table">
               <tr>
-                <td>Place</td>
                 <td>Name</td>
+                <td>Commits</td>
               </tr>
-              <tr>
-                <% n = 0 %>
-                <% @group.each do |css| %>
-                <td><%= n = n + 1 %></td>
-                <td><%= css.join("- ") %> commits</td>
-              </tr>
+              <% @stats.each do |stat| %>
+                <tr>
+                  <td><%= stat.name %></td>
+                  <td><%= stat.commits %></td>
+                </tr>
               <% end %>
-                <table>
+            <table>
         </div>
       </div>
     </div>

app/models/stat.rb

@@ -1,3 +1,3 @@
 Rails.application.routes.draw do
-  root "welcome#index"
+  root "stats#index"
 end

app/views/welcome/index.html.erb app/views/stats/index.html.erbapp/views/welcome/index.html.erb renamed to app/views/stats/index.html.erb

@@ -0,0 +1,10 @@
+class CreateStats < ActiveRecord::Migration
+  def change
+    create_table :stats do |t|
+      t.string :name
+      t.integer :commits
+
+      t.timestamps null: false
+    end
+  end
+end

config/routes.rb

@@ -11,9 +11,16 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 0) do
+ActiveRecord::Schema.define(version: 20151007021403) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
+  create_table "stats", force: :cascade do |t|
+    t.string   "name"
+    t.integer  "commits"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+  end
+
 end