Allow users to update their username

driesvints · driesvints · commit 38af463c5047 · 2014-12-21T15:24:13.000+01:00
diff --git a/app/controllers/UsersController.php b/app/controllers/UsersController.php
@@ -57,7 +57,7 @@ public function putSettings($userName)
             App::abort(403);
         }
 
-        return $this->updater->update($this, $user, Input::only('email'));
+        return $this->updater->update($this, $user, Input::only('name', 'email'));
     }
 
     public function getThreads($userName)
@@ -91,6 +91,6 @@ public function userUpdated($user, $emailChanged = false)
             Session::flash('success', 'Settings updated');
         }
 
-        return Redirect::route('user', $user->name);
+        return Redirect::route('user.settings', $user->name);
     }
 }
diff --git a/app/views/auth/signupconfirm.blade.php b/app/views/auth/signupconfirm.blade.php
@@ -12,7 +12,7 @@
                     <p><img src="{{ $githubUser['image_url'] }}"/></p>
 
                     <p>
-                        {{ Form::label('name') }}
+                        {{ Form::label('name', 'Username') }}
                         {{ Form::text('name', Input::old('name', $githubUser['name'])) }}
                     </p>
 
diff --git a/app/views/users/settings.blade.php b/app/views/users/settings.blade.php
@@ -12,6 +12,15 @@
     </div>
     <div class="user-settings">
         {{ Form::open(['route' => ['user.settings.update', $user->name], 'method' => 'put']) }}
+            <p>
+                {{ Form::label('name', 'Username') }}
+                {{ Form::text('name', $user->name) }}
+            </p>
+
+            @if ($errors->has('name'))
+                <p>{{ $errors->first('name') }}</p>
+            @endif
+
             <p>
                 {{ Form::label('email') }}
                 {{ Form::email('email', $user->email) }}
diff --git a/public/scss/modules/_user.scss b/public/scss/modules/_user.scss
@@ -15,7 +15,7 @@
 .auth {
     label {
         display: inline-block;
-        width: 40px;
+        width: 65px;
     }
     input {
         display: inline-block;
@@ -82,7 +82,7 @@
 
     label {
         display: inline-block;
-        width: 40px;
+        width: 65px;
     }
     input {
         display: inline-block;

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ public function putSettings($userName)`
`57`	`57`	`App::abort(403);`
`58`	`58`	`}`
`59`	`59`
`60`		`- return $this->updater->update($this, $user, Input::only('email'));`
	`60`	`+ return $this->updater->update($this, $user, Input::only('name', 'email'));`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`public function getThreads($userName)`
`@@ -91,6 +91,6 @@ public function userUpdated($user, $emailChanged = false)`
`91`	`91`	`Session::flash('success', 'Settings updated');`
`92`	`92`	`}`
`93`	`93`
`94`		`- return Redirect::route('user', $user->name);`
	`94`	`+ return Redirect::route('user.settings', $user->name);`
`95`	`95`	`}`
`96`	`96`	`}`