unicodeveloper / laravel-hackathon-starter

app/Http/Controllers/ContactController.php

Spacing +2 added lines, -2 removed lines

@@ -29,13 +29,13 @@
         $emailToSendTo = $request->input('email');
         $body = $request->input('message');
 
-        Mail::send('emails.contact', ['body' => $body], function ($message) use ($name,$emailToSendTo) {
+        Mail::send('emails.contact', [ 'body' => $body ], function($message) use ($name, $emailToSendTo) {
             $message->from('unicodeveloper@hackathon-starter.com', "From: {$name}");
 
             $message->to($emailToSendTo)->subject("Message From Laravel Hackathon Starter Contact Form");
         });
 
-        return redirect()->route('contact')->with('info','Your Message has been dispatched successfully');
+        return redirect()->route('contact')->with('info', 'Your Message has been dispatched successfully');
     }
 
 }

app/Providers/RouteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function map(Router $router)
     {
-        $router->group(['namespace' => $this->namespace], function ($router) {
+        $router->group([ 'namespace' => $this->namespace ], function($router) {
             require app_path('Http/routes.php');
         });
     }

app/Http/Controllers/OauthController.php

Doc Comments +5 added lines, -1 removed lines

@@ -26,6 +26,9 @@ 
         return $this->execute(($request->has('code') || $request->has('oauth_token')), $provider);
     }
 
+    /**
+     * @param boolean $request
+     */
     public function execute($request, $provider)
     {
         if (! $request) {
@@ -41,6 +44,7 @@ 
     /**
      * Find a user by username or create a new user
      * @param
+     * @param \Laravel\Socialite\Contracts\User $userData
      * @return
      */
     public function findByProviderIdOrCreate($userData, $provider)
@@ -103,7 +107,7 @@ 
     /**
      * Get Data from Social Media Account
      * @param  string $provider
-     * @return collection
+     * @return \Laravel\Socialite\Contracts\User
      */
     private function getSocialUser($provider)
     {

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace App\Http\Controllers;
 
 use Illuminate\Http\Request;
-
-use App\Http\Requests;
 use App\Http\Controllers\Controller;
 use Illuminate\Contracts\Auth\Guard;
 use Laravel\Socialite\Contracts\Factory as Socialite;

Spacing +7 added lines, -7 removed lines

@@ -28,7 +28,7 @@ 
 
     public function execute($request, $provider)
     {
-        if (! $request) {
+        if (!$request) {
             return $this->getAuthorizationFirst($provider);
         }
 
@@ -51,7 +51,7 @@ 
 
         $username = $this->isEmailExists($userData->getNickName()) ? null : $userData->getEmail();
 
-        if (empty($user))  {
+        if (empty($user)) {
             $user = User::create([
                 'fullname'      => $userData->getName(),
                 'username'      => $username,
@@ -69,16 +69,16 @@ 
 
     private function isUsernameExists($username = null)
     {
-        $username = User::whereUsername($username)->first()['username'];
+        $username = User::whereUsername($username)->first()[ 'username' ];
 
-        return (! is_null($username)) ? true : false;
+        return (!is_null($username)) ? true : false;
     }
 
     private function isEmailExists($email = null)
     {
-        $email = User::whereEmail($email)->first()['email'];
+        $email = User::whereEmail($email)->first()[ 'email' ];
 
-        return (! is_null($email)) ? true : false;
+        return (!is_null($email)) ? true : false;
     }
 
     /**
@@ -100,7 +100,7 @@ 
             'username' => $user->username,
         ];
 
-        if (! empty(array_diff($dbData, $socialData))) {
+        if (!empty(array_diff($dbData, $socialData))) {
             $user->avatar = $userData->getAvatar();
             $user->fullname = $userData->getName();
             $user->username = $userData->getNickName();

app/Http/Controllers/Auth/AuthController.php

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
      */
     public function __construct()
     {
-        $this->middleware('guest', ['except' => 'logout']);
+        $this->middleware('guest', [ 'except' => 'logout' ]);
     }
 
     /**
@@ -64,9 +64,9 @@ 
     protected function create(array $data)
     {
         return User::create([
-            'fullname' => $data['name'],
-            'email' => $data['email'],
-            'password' => bcrypt($data['password']),
+            'fullname' => $data[ 'name' ],
+            'email' => $data[ 'email' ],
+            'password' => bcrypt($data[ 'password' ]),
         ]);
     }
 

app/Http/Controllers/NytController.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace App\Http\Controllers;
 
 use GuzzleHttp\Client;
-use App\Http\Requests;
-use Illuminate\Http\Request;
 use App\Http\Controllers\Controller;
 
 class NytController extends Controller

Indentation +4 added lines, -4 removed lines

@@ -27,11 +27,11 @@
      */
     public function __construct()
     {
-         $this->baseUrl = 'http://api.nytimes.com/svc';
-         $this->client = new Client(['base_uri' => $this->baseUrl]);
+            $this->baseUrl = 'http://api.nytimes.com/svc';
+            $this->client = new Client(['base_uri' => $this->baseUrl]);
 
-         $relativeUrl = '/books/v3/lists/overview.json?api-key=' . env('NYT_BOOKS_API_KEY');
-         $this->setGetResponse($relativeUrl);
+            $relativeUrl = '/books/v3/lists/overview.json?api-key=' . env('NYT_BOOKS_API_KEY');
+            $this->setGetResponse($relativeUrl);
     }
 
     /**

Spacing +4 added lines, -4 removed lines

@@ -28,9 +28,9 @@ 
     public function __construct()
     {
          $this->baseUrl = 'http://api.nytimes.com/svc';
-         $this->client = new Client(['base_uri' => $this->baseUrl]);
+         $this->client = new Client([ 'base_uri' => $this->baseUrl ]);
 
-         $relativeUrl = '/books/v3/lists/overview.json?api-key=' . env('NYT_BOOKS_API_KEY');
+         $relativeUrl = '/books/v3/lists/overview.json?api-key='.env('NYT_BOOKS_API_KEY');
          $this->setGetResponse($relativeUrl);
     }
 
@@ -40,7 +40,7 @@ 
      */
     private function setGetResponse($relativeUrl)
     {
-        $this->response = $this->client->get($this->baseUrl . $relativeUrl, []);
+        $this->response = $this->client->get($this->baseUrl.$relativeUrl, [ ]);
     }
 
     /**
@@ -58,7 +58,7 @@ 
      */
     private function getData()
     {
-        return $this->getResponse()['results']['lists'][0]['books'];
+        return $this->getResponse()[ 'results' ][ 'lists' ][ 0 ][ 'books' ];
     }
 
     /**

app/Http/Controllers/SteamController.php

Indentation +4 added lines, -4 removed lines

@@ -9,10 +9,10 @@
 
 class SteamController extends Controller
 {
-     /**
-     * Return all data to the Steam API dashboard
-     * @return mixed
-     */
+        /**
+         * Return all data to the Steam API dashboard
+         * @return mixed
+         */
     public function getPage()
     {
         return view('api.steam');

app/Http/Controllers/WebScrapingController.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
 
     /**
      * Scrape the Links
-     * @param $siteToCrawl
+     * @param string $siteToCrawl
      * @return array
      */
     public function getData($siteToCrawl)

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use Illuminate\Http\Request;
-
-use App\Http\Requests;
 use Goutte\Client;
 use App\Http\Controllers\Controller;
 

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $crawler = $this->client->request('GET', $siteToCrawl);
 
         $arr = $crawler->filter('.title a[href^="http"], a[href^="https"]')->each(function($element) {
-            $links = [];
+            $links = [ ];
 
             array_push($links, $element->text());
 

app/Http/Controllers/YahooController.php

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use Illuminate\Http\Request;
-
-use App\Http\Requests;
 use Goutte\Client;
 use App\Http\Controllers\Controller;
 

Indentation +13 added lines, -13 removed lines

@@ -10,10 +10,10 @@ 
 
 class YahooController extends Controller
 {
-     /**
-     * Instance of Guzzle Client
-     * @var object
-     */
+        /**
+         * Instance of Guzzle Client
+         * @var object
+         */
     protected $client;
 
     /**
@@ -27,13 +27,13 @@ 
      */
     public function __construct()
     {
-         $this->baseUrl = 'https://query.yahooapis.com/v1/public/yql';
-         $this->client = new Client(['base_uri' => $this->baseUrl]);
+            $this->baseUrl = 'https://query.yahooapis.com/v1/public/yql';
+            $this->client = new Client(['base_uri' => $this->baseUrl]);
 
-         $query = "SELECT * FROM weather.forecast WHERE (location = 10007)";
+            $query = "SELECT * FROM weather.forecast WHERE (location = 10007)";
 
-         $relativeUrl = '?q=' . $query . '&format=json';
-         $this->setGetResponse($relativeUrl);
+            $relativeUrl = '?q=' . $query . '&format=json';
+            $this->setGetResponse($relativeUrl);
     }
 
     /**
@@ -64,10 +64,10 @@ 
     }
 
 
-     /**
-     * Return all data to the Yahoo API dashboard
-     * @return mixed
-     */
+        /**
+         * Return all data to the Yahoo API dashboard
+         * @return mixed
+         */
     public function getPage()
     {
         $data = $this->getData();

Spacing +4 added lines, -4 removed lines

@@ -28,11 +28,11 @@ 
     public function __construct()
     {
          $this->baseUrl = 'https://query.yahooapis.com/v1/public/yql';
-         $this->client = new Client(['base_uri' => $this->baseUrl]);
+         $this->client = new Client([ 'base_uri' => $this->baseUrl ]);
 
          $query = "SELECT * FROM weather.forecast WHERE (location = 10007)";
 
-         $relativeUrl = '?q=' . $query . '&format=json';
+         $relativeUrl = '?q='.$query.'&format=json';
          $this->setGetResponse($relativeUrl);
     }
 
@@ -42,7 +42,7 @@ 
      */
     private function setGetResponse($relativeUrl)
     {
-        $this->response = $this->client->get($this->baseUrl . $relativeUrl, []);
+        $this->response = $this->client->get($this->baseUrl.$relativeUrl, [ ]);
     }
 
     /**
@@ -60,7 +60,7 @@ 
      */
     private function getData()
     {
-        return $this->getResponse()['query']['results']['channel'];
+        return $this->getResponse()[ 'query' ][ 'results' ][ 'channel' ];
     }
 
 

app/Http/Controllers/ClockworkController.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace App\Http\Controllers;
 
 use Illuminate\Http\Request;
-
-use App\Http\Requests;
 use App\Http\Controllers\Controller;
 use Illuminate\Contracts\Auth\Guard;
 use Laravel\Socialite\Contracts\Factory as Socialite;

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     /**
      * Send a Text Message
      * @param  Request $request
-     * @return string
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function sendTextMessage(Request $request)
     {

Spacing +1 added lines, -1 removed lines

@@ -36,6 +36,6 @@
 
         Twilio::message($number, $message);
 
-        return redirect()->back()->with('info','Your Message has been sent successfully');
+        return redirect()->back()->with('info', 'Your Message has been sent successfully');
     }
 }