Properly check for a pre-existing error in `User#namespace_uniq`

Rails adds an empty error to the Errors object even if you just call `#[]` on it: ``` [1] pry(main)> u = User.last [2] pry(main)> u.errors.keys => [] [3] pry(main)> u.errors[:username].present? => false [4] pry(main)> u.errors.keys => [:username] ```

Properly check for a pre-existing error in `User#namespace_uniq`
128757a7 · Robert Speicher · e5276ee6 · 128757a7
Commit 128757a7 authored Feb 29, 2016 by Robert Speicher
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

user.rb app/models/user.rb +2 -1

No files found.
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -372,7 +372,8 @@ class User < ActiveRecord::Base

  def namespace_uniq
    # Return early if username already failed the first uniqueness validation
-    return if self.errors[:username].include?('has already been taken')
+    return if self.errors.key?(:username) &&
+      self.errors[:username].include?('has already been taken')

    namespace_name = self.username
    existing_namespace = Namespace.by_path(namespace_name)