neayi / insights

app/Http/Middleware/CheckEmailVerified.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
     public function handle($request, Closure $next)
     {
         $user = Auth::user();
-        if(!$user->hasVerifiedEmail()){
+        if (!$user->hasVerifiedEmail()) {
             $request->session()->flash('from_forum', true);
             return redirect()->route('verification.notice');
         }

app/Observers/PageObserver.php

Spacing +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
 
     public function deleted(PageModel $pageModel)
     {
-        Log::info('Deleting all data from the page internal id: ' . $pageModel->id);
-        DB::transaction(function () use($pageModel){
+        Log::info('Deleting all data from the page internal id: '.$pageModel->id);
+        DB::transaction(function() use($pageModel){
             try {
                 $characteristic = CharacteristicsModel::query()->where('page_id', $pageModel->page_id)->first();
                 if (isset($characteristic)) {
@@ -36,7 +36,7 @@ 
                     UserCharacteristicsModel::query()->where('characteristic_id', $characteristic->id)->delete();
                 }
             } catch (\Throwable $e) {
-                Log::emergency('Error deleting all data from the page internal id: ' . $pageModel->id);
+                Log::emergency('Error deleting all data from the page internal id: '.$pageModel->id);
                 throw $e;
             }
         });

app/Src/UseCases/Infra/Gateway/SocialiteGatewayImpl.php

Spacing +8 added lines, -8 removed lines

@@ -12,31 +12,31 @@
 {
     public function user(string $provider): SocialiteUser
     {
-        if($provider === 'twitter'){
+        if ($provider === 'twitter') {
             $user = Socialite::driver($provider)->user();
-        }elseif($provider === 'facebook'){
+        }elseif ($provider === 'facebook') {
             $user = Socialite::driver($provider)->fields(['name', 'first_name', 'last_name', 'email'])->user();
-        }else {
+        } else {
             $user = Socialite::driver($provider)->stateless()->user();
         }
 
         $email = $user->getEmail();
 
         // For twitter, we prefer to use the name. If empty we'll use the twitter account (nickname):
-        $firstname = $user->getName() !== null ? $user->getName() :  $user->getNickname();
-        if(!empty($user->user['given_name'])){
+        $firstname = $user->getName() !== null ? $user->getName() : $user->getNickname();
+        if (!empty($user->user['given_name'])) {
             // Google
             $firstname = $user->user['given_name'];
-        }elseif (!empty($user->user['first_name'])){
+        }elseif (!empty($user->user['first_name'])) {
             // Facebook
             $firstname = $user->user['first_name'];
         }
 
         $lastname = $user->getName();
-        if(!empty($user->user['family_name'])){
+        if (!empty($user->user['family_name'])) {
             // Google
             $lastname = $user->user['family_name'];
-        }elseif (!empty($user->user['last_name'])){
+        }elseif (!empty($user->user['last_name'])) {
             // Facebook
             $lastname = $user->user['last_name'];
         }

Braces +4 added lines, -4 removed lines

@@ -14,9 +14,9 @@ 
     {
         if($provider === 'twitter'){
             $user = Socialite::driver($provider)->user();
-        }elseif($provider === 'facebook'){
+        } elseif($provider === 'facebook'){
             $user = Socialite::driver($provider)->fields(['name', 'first_name', 'last_name', 'email'])->user();
-        }else {
+        } else {
             $user = Socialite::driver($provider)->stateless()->user();
         }
 
@@ -27,7 +27,7 @@ 
         if(!empty($user->user['given_name'])){
             // Google
             $firstname = $user->user['given_name'];
-        }elseif (!empty($user->user['first_name'])){
+        } elseif (!empty($user->user['first_name'])){
             // Facebook
             $firstname = $user->user['first_name'];
         }
@@ -36,7 +36,7 @@ 
         if(!empty($user->user['family_name'])){
             // Google
             $lastname = $user->user['family_name'];
-        }elseif (!empty($user->user['last_name'])){
+        } elseif (!empty($user->user['last_name'])){
             // Facebook
             $lastname = $user->user['last_name'];
         }

app/Console/Commands/Debug/TestMailers.php

Indentation +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
     protected $description = 'Add email to newsletter';
 
     public function handle(SendinBlueService $sendinBlueService,
-                           MailerLiteService $mailerLiteService)
+                            MailerLiteService $mailerLiteService)
     {
         $email = $this->argument('email');
 

app/Console/Commands/ImportCharacteristicsFromWiki.php

Spacing +6 added lines, -6 removed lines

@@ -55,7 +55,7 @@ 
         $content = $wikiClient->searchCharacteristics($opt);
         $characteristics = $content['query']['results'];
 
-        foreach ($characteristics as  $characteristic){
+        foreach ($characteristics as  $characteristic) {
             $page = key($characteristic);
             $this->info(sprintf("Importing %s", $page));
 
@@ -63,7 +63,7 @@ 
 
             $icon = '';
             $uuid = Uuid::uuid4();
-            if(isset($characteristic['printouts']['A un glyph'][0])) {
+            if (isset($characteristic['printouts']['A un glyph'][0])) {
                 $icon = $characteristic['printouts']['A un glyph'][0];
             }
 
@@ -77,21 +77,21 @@ 
             $characteristicsToSave = [
                 'uuid' => $uuid,
                 'main' => $main,
-                'priority' => (int)last($characteristic['printouts']['A une priorité d\'affichage']),
+                'priority' => (int) last($characteristic['printouts']['A une priorité d\'affichage']),
                 'icon' => $icon,
                 'page_label' => $label,
                 'pretty_page_label' => $prettyPage,
-                'page_id' => (int)$pageInfo['pageid'],
+                'page_id' => (int) $pageInfo['pageid'],
                 'type' => $type,
                 'code' => $pageInfo['title'],
                 'wiki' => strtolower($wikiCode),
             ];
 
             $model = CharacteristicsModel::query()
-                ->where('page_id', (int)$pageInfo['pageid'])
+                ->where('page_id', (int) $pageInfo['pageid'])
                 ->where('wiki', $wikiCode)
                 ->first();
-            if(!isset($model)) {
+            if (!isset($model)) {
                 $model = new CharacteristicsModel();
             }
             $model->fill($characteristicsToSave);

app/Console/Commands/SyncUsersDiscourse.php

Spacing +11 added lines, -11 removed lines

@@ -38,8 +38,8 @@ 
             ->where('lastname', '<>', '')
             ->whereNotNull('email_verified_at')
             ->whereNull('sync_at_discourse')
-            ->chunkById(50, function ($items) use ($clients) {
-                foreach($items as $user) {
+            ->chunkById(50, function($items) use ($clients) {
+                foreach ($items as $user) {
                     $this->processOneUser($clients, $user);
                 }
             });
@@ -60,13 +60,13 @@ 
             $this->updateUserDetailsOnDiscourse($user);
             $user->sync_at_discourse = (new \DateTime())->format('Y-m-d H:i:s');
             $user->save();
-        } catch (\Throwable $e){
+        } catch (\Throwable $e) {
             if ($e->getCode() === 429) {
                 // Too many requests - just sleeping
                 $this->error('Too many requests - restarting in a minute....');
                 sleep(60);
             } else {
-                $message = 'Discourse sync failed for user : ' . $user->uuid . ' [' . $e->getCode() . '] ' . $e->getMessage();
+                $message = 'Discourse sync failed for user : '.$user->uuid.' ['.$e->getCode().'] '.$e->getMessage();
                 $this->error($message);
                 \Sentry\captureException($e);
             }
@@ -82,7 +82,7 @@ 
         $username = $this->formatUsername($user, $increment);
 
         $result = $this->forumApiClient->createUser($username, $user);
-        if($result['success'] === false){
+        if ($result['success'] === false) {
             if (!empty($result['errors']['email'])) {
                 return $this->updateUsernameFromDiscourse($user);
             }
@@ -121,7 +121,7 @@ 
             throw $th;
         }
 
-        if(empty($result['user'])){
+        if (empty($result['user'])) {
             throw new \Exception('Duplicate email not corresponding to existing user');
         }
         $user->discourse_id = $result['user']['id'];
@@ -145,7 +145,7 @@ 
     {
         $result = $this->forumApiClient->getUserByEmail($user->email);
 
-        if(empty($result[0]['email']) || strtolower($result[0]['email']) != strtolower($user->email)){
+        if (empty($result[0]['email']) || strtolower($result[0]['email']) != strtolower($user->email)) {
             throw new \Exception('Duplicate email not corresponding to existing user');
         }
 
@@ -162,7 +162,7 @@ 
     {
         try {
             $this->forumApiClient->updateEmail($user->discourse_username, $user->email);
-        }catch (\Throwable $e){
+        } catch (\Throwable $e) {
             $this->error('User email not updated on discourse with id : '.$user->discourse_username);
         }
 
@@ -180,7 +180,7 @@ 
 
         $result = $this->forumApiClient->updateUser($user, $newBio);
 
-        if($result['success'] === false){
+        if ($result['success'] === false) {
             $this->error('Not Updating bio : '.$result['message']);
             throw new \Exception($result['message']);
         }
@@ -198,11 +198,11 @@ 
      */
     public function formatUsername(User $user, int $increment): string
     {
-        $username = trim(substr((string)Str::of($user->fullname)->slug('.'), 0, 20), '.');
+        $username = trim(substr((string) Str::of($user->fullname)->slug('.'), 0, 20), '.');
         if (empty($username)) {
             throw new \Exception("Empty username", 55);
         }
 
-        return !empty($increment) ? $username . $increment : $username;
+        return !empty($increment) ? $username.$increment : $username;
     }
 }

Braces +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
     {
         try {
             $this->forumApiClient->updateEmail($user->discourse_username, $user->email);
-        }catch (\Throwable $e){
+        } catch (\Throwable $e){
             $this->error('User email not updated on discourse with id : '.$user->discourse_username);
         }
 

app/Console/Commands/SyncDryPagesFromWiki.php

Spacing +3 added lines, -3 removed lines

@@ -27,16 +27,16 @@
             PageModel::query()
                 ->where('dry', true)
                 ->where('wiki', $wikiCode)
-                ->chunkById(50, function ($items, $count) use($client){
+                ->chunkById(50, function($items, $count) use($client){
                     $this->info(($count*50).' Pages');
                     $pages = $items->pluck('page_id')->toArray();
                     $content = $client->searchPagesById($pages);
                     $wikiPages = $content['query']['pages'];
 
-                    foreach($wikiPages as $page){
+                    foreach ($wikiPages as $page) {
                         $pageModel = PageModel::query()->where('page_id', $page['pageid'])->first();
 
-                        if(!isset($pageModel)){
+                        if (!isset($pageModel)) {
                             continue;
                         }
 

app/Listeners/AddEmailToNewsletter.php

Spacing +2 added lines, -2 removed lines

@@ -28,12 +28,12 @@
         try {
             $this->sendingBlueService->addEmailToList($verified->user->email, $verified->user->lastname, $verified->user->firstname);
         } catch (\Throwable $e) {
-            Log::critical('Error when adding email to sending blue : ' . $verified->user->email);
+            Log::critical('Error when adding email to sending blue : '.$verified->user->email);
         }
         try {
             $this->mailerLiteService->addEmailToList($verified->user->email);
         } catch (\Exception $e) {
-            Log::critical('Error when adding email to mailerlite : ' . $verified->user->email);
+            Log::critical('Error when adding email to mailerlite : '.$verified->user->email);
         }
     }
 }

app/Listeners/SetUserToSyncOnDiscourse.php

Spacing +2 added lines, -2 removed lines

@@ -13,13 +13,13 @@
     {
         try {
             $user = $verified->user;
-            if(empty($user->firstname) || empty($user->lastname)){
+            if (empty($user->firstname) || empty($user->lastname)) {
                 return;
             }
 
             $user->sync_at_discourse = null;
             $user->save();
-        }catch (\Throwable $e){
+        } catch (\Throwable $e) {
             Log::emergency('Error when sync asking for user : '.$user->id);
             \Sentry\captureException($e);
         }

Braces +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
             $user->sync_at_discourse = null;
             $user->save();
-        }catch (\Throwable $e){
+        } catch (\Throwable $e){
             Log::emergency('Error when sync asking for user : '.$user->id);
             \Sentry\captureException($e);
         }