Made fixes suggested by @randx in pull request

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

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
 

db/migrate/20130319214458_create_forked_project_links.rb

 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