Merge pull request #65 from cachethq/elliotfehr-master

Fixed rebase issues

Author: jbrooksuk

app/controllers/HomeController.php

@@ -1,11 +1,20 @@
 <?php
 
 class HomeController extends Controller {
+    /**
+     * @var Component $component
+     */
+    protected $component;
+
+    public function __construct(Component $component) {
+        $this->component = $component;
+    }
+
     /**
      * Returns the rendered Blade templates.
      * @return \Illuminate\View\View
      */
     public function showIndex() {
-        return View::make('index', ['components' => Component::all()]);
+        return View::make('index', ['components' => $this->component->all()]);
     }
 }

app/controllers/SetupController.php

@@ -1,11 +1,15 @@
 <?php
 
 class SetupController extends Controller {
+    public function __construct() {
+        $this->beforeFilter('csrf', ['only' => ['postCachet']]);
+    }
+
     /**
      * Returns the setup page.
      * @return \Illuminate\View\View
      */
-    public function showSetup() {
+    public function getIndex() {
         return View::make('setup')->with([
             'pageTitle' => 'Setup'
         ]);
@@ -15,26 +19,25 @@ public function showSetup() {
      * Handles the actual app setup.
      * @return \Illuminate\Http\RedirectResponse
      */
-    public function setupCachet() {
+    public function postIndex() {
         $postData = Input::get();
         $v = Validator::make($postData, [
             'settings.app_name'     => 'required',
-            'settings.app_domain'   => 'url|required',
+            'settings.app_domain'   => 'required',
             'settings.show_support' => 'boolean',
-            'user.name'             => 'alpha_dash|required',
+            'user.username'         => 'alpha_dash|required',
             'user.email'            => 'email|required',
             'user.password'         => 'required'
         ]);
 
         if ($v->passes()) {
             // Pull the user details out.
-            $userDetails = array_get($postData, 'user');
-            unset($postData['user']);
+            $userDetails = array_pull($postData, 'user');
 
             $user           = new User;
-            $user->username = $userDetails['name'];
+            $user->username = $userDetails['username'];
             $user->email    = $userDetails['email'];
-            $user->password = Hash::make($userDetails['password']);
+            $user->password = $userDetails['password'];
             $user->save();
 
             Auth::login($user);

app/models/User.php

@@ -23,4 +23,15 @@ class User extends Eloquent implements UserInterface, RemindableInterface {
      */
     protected $hidden = ['password', 'remember_token'];
 
+    /**
+     * Hash any password being inserted by default
+     *
+     * @param string @password
+     * @return void
+     */
+    public function setPasswordAttribute($password) {
+        $this->attributes['password'] = Hash::make($password);
+    }
+
+
 }

app/routes/app.php

@@ -10,10 +10,7 @@
 });
 
 Route::group(['before' => 'no_setup:app_name'], function() {
-    Route::get('/setup', 'SetupController@showSetup');
-    Route::group(['before' => 'csrf'], function() {
-        Route::post('/setup', 'SetupController@setupCachet');
-    });
+    Route::controller('/setup', 'SetupController');
 });
 
 Route::group(['before' => 'auth'], function() {

app/views/setup.blade.php

@@ -41,9 +41,9 @@
                             <legend>Administrator Account</legend>
                             <div class='form-group'>
                                 <label class='sr-only'>Username</label>
-                                <input type='text' name='user[name]' class='form-control' placeholder='Username' value='{{ Input::old('user.name', '') }}' required />
-                                @if($errors->has('user.name'))
-                                <span class='text-danger'>{{ $errors->first('user.name') }}</span>
+                                <input type='text' name='user[username]' class='form-control' placeholder='Username' value='{{ Input::old('user.username', '') }}' required />
+                                @if($errors->has('user.username'))
+                                <span class='text-danger'>{{ $errors->first('user.username') }}</span>
                                 @endif
                             </div>
                             <div class='form-group'>

composer.json

@@ -45,7 +45,7 @@
   "scripts": {
     "post-install-cmd": [
       "php artisan optimize",
-      "chmod -R 777 app/storage public",
+      "chmod -R 755 app/storage",
       "php artisan migrate"
     ],
     "post-update-cmd": [