Made fixes suggested by @randx in pull request

See https://github.com/gitlabhq/gitlabhq/pull/3597#discussion-diff-4014638

diff --git a/app/models/ability.rb b/app/models/ability.rb
index 0c5fbc2..eb7f89d 100644 (file)
--- a/app/models/ability.rb
+++ b/app/models/ability.rb
@@ -60,15 +60,16 @@ class Ability
         :read_note,
         :write_project,
         :write_issue,
-        :write_note,
-        :fork_project
+        :write_note
       ]
     end
 
     def project_report_rules
       project_guest_rules + [
         :download_code,
-        :write_snippet
+        :write_snippet,
+        :fork_project
+
       ]
     end
 

diff --git a/db/migrate/20130319214458_create_forked_project_links.rb b/db/migrate/20130319214458_create_forked_project_links.rb
index 55aad12..f91afc2 100644 (file)
--- a/db/migrate/20130319214458_create_forked_project_links.rb
+++ b/db/migrate/20130319214458_create_forked_project_links.rb
@@ -1,11 +1,11 @@
 class CreateForkedProjectLinks < ActiveRecord::Migration
   def change
     create_table :forked_project_links do |t|
-      t.integer :forked_to_project_id, :null => false
-      t.integer :forked_from_project_id, :null => false
+      t.integer :forked_to_project_id, null: false
+      t.integer :forked_from_project_id, null: false
 
       t.timestamps
     end
-    add_index :forked_project_links, :forked_to_project_id, :unique => true
+    add_index :forked_project_links, :forked_to_project_id, unique: true
   end
 end