validates presence and uniqueness of full_name and github_id

sashazykov · sashazykov · commit 653add9dfa9d · 2014-02-09T11:16:13.000+07:00
diff --git a/app/controllers/users/omniauth_callbacks_controller.rb b/app/controllers/users/omniauth_callbacks_controller.rb
@@ -15,7 +15,7 @@ def github
     @user.name = info['name']
     @user.image = info['image']
     @user.save
-    
+
     sign_in_and_redirect @user, :event => :authentication
     set_flash_message(:notice, :success, :kind => "GitHub") if is_navigational_format?
   end
diff --git a/app/models/project.rb b/app/models/project.rb
@@ -2,6 +2,9 @@ class Project < ActiveRecord::Base
   has_many :deposits # todo: only confirmed deposits that have amount > paid_out
   has_many :tips
 
+  validates :full_name, uniqueness: true, presence: true
+  validates :github_id, uniqueness: true, presence: true
+
   def update_github_info repo
     self.github_id = repo.id
     self.name = repo.name
@@ -69,7 +72,8 @@ def tip_for commit
           user = User.create({
           email: email,
           password: generated_password,
-          name: commit.commit.author.name
+          name: commit.commit.author.name,
+          nickname: (commit.author.login rescue nil)
         })
       end
 
diff --git a/db/migrate/20140209041123_create_indexes_for_projects.rb b/db/migrate/20140209041123_create_indexes_for_projects.rb
@@ -0,0 +1,6 @@
+class CreateIndexesForProjects < ActiveRecord::Migration
+  def change
+    add_index :projects, :full_name,                :unique => true
+    add_index :projects, :github_id,                :unique => true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20140209022632) do
+ActiveRecord::Schema.define(version: 20140209041123) do
 
   create_table "deposits", force: true do |t|
     t.integer  "project_id"
@@ -43,6 +43,9 @@
     t.string   "github_id"
   end
 
+  add_index "projects", ["full_name"], name: "index_projects_on_full_name", unique: true
+  add_index "projects", ["github_id"], name: "index_projects_on_github_id", unique: true
+
   create_table "sendmanies", force: true do |t|
     t.string   "txid"
     t.text     "data"
