Refactor

2 files changed, 9 insertions, 9 deletions

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index 6b6b3b20a8d..d7c1241698a 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -94,11 +94,9 @@ class UsersController < ApplicationController
   private
 
   def authorize_read_user!
-    if can?(current_user, :read_user, user)
-      ensure_canonical_path(user.namespace, params[:username])
-    else
-      render_404
-    end
+    render_404 unless can?(current_user, :read_user, user)
+    
+    ensure_canonical_path(user.namespace, params[:username])
   end
 
   def user
diff --git a/app/models/route.rb b/app/models/route.rb
index e0d85ff7db7..accc423ae46 100644
--- a/app/models/route.rb
+++ b/app/models/route.rb
@@ -43,11 +43,13 @@ class Route < ActiveRecord::Base
     RedirectRoute.matching_path_and_descendants(path)
   end
 
-  def create_redirect_for_old_path
-    create_redirect(path_was) if path_changed?
-  end
-
   def create_redirect(path)
     RedirectRoute.create(source: source, path: path)
   end
+
+  private
+
+  def create_redirect_for_old_path
+    create_redirect(path_was) if path_changed?
+  end
 end