neayi / insights

app/Src/UseCases/Domain/Context/Model/CharacteristicMemento.php

Spacing +2 added lines, -2 removed lines

@@ -45,12 +45,12 @@
         return $this->visible;
     }
 
-    public function icon():? string
+    public function icon(): ? string
     {
         return $this->icon;
     }
 
-    public function pageId():? int
+    public function pageId(): ? int
     {
         return $this->pageId;
     }

app/Http/Controllers/Profile/ProfileController.php

Braces +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
         $allCharacteristics = app(GetAllCharacteristics::class)->get();
         try {
             $context = $contextQueryByUser->execute(Auth::user()->uuid)->toArray();
-        }catch (\Throwable $e){
+        } catch (\Throwable $e){
             Log::emergency($e->getMessage().' '.$e->getLine().' '.$e->getFile().' '.$e->getTraceAsString());
             return redirect()->route('wizard.profile');
         }
@@ -66,7 +66,7 @@ 
     {
         try {
             $user = app(GetUser::class)->get($userId)->toArray();
-        }catch (UserNotFound $e){
+        } catch (UserNotFound $e){
             throw new NotFoundHttpException();
         }
 

Spacing +6 added lines, -6 removed lines

@@ -35,7 +35,7 @@ 
         $allCharacteristics = app(GetAllCharacteristics::class)->get();
         try {
             $context = $contextQueryByUser->execute(Auth::user()->uuid)->toArray();
-        }catch (\Throwable $e){
+        } catch (\Throwable $e) {
             Log::emergency($e->getMessage().' '.$e->getLine().' '.$e->getFile().' '.$e->getTraceAsString());
             return redirect()->route('wizard.profile');
         }
@@ -43,7 +43,7 @@ 
         $roles = app(GetUserRole::class)->get()->toArray();
         $practises = app(GetUserPractises::class)->get(Auth::user()->uuid);
         $interactions = app(InteractionsQueryByUser::class)->get(Auth::user()->uuid);
-        $usersCharacteristics =  array_merge($context['productions'], $context['characteristics'], $context['characteristicsDepartement']);
+        $usersCharacteristics = array_merge($context['productions'], $context['characteristics'], $context['characteristicsDepartement']);
         $uuidsUserCharacteristics = array_column($usersCharacteristics, 'uuid');
         $role = last($user['roles']);
         $routeComment = route('profile.comments.show');
@@ -68,7 +68,7 @@ 
     {
         try {
             $user = app(GetUser::class)->get($userId)->toArray();
-        }catch (UserNotFound $e){
+        } catch (UserNotFound $e) {
             throw new NotFoundHttpException();
         }
 
@@ -77,7 +77,7 @@ 
         $roles = app(GetUserRole::class)->get()->toArray();
         $practises = app(GetUserPractises::class)->get($userId);
         $interactions = app(InteractionsQueryByUser::class)->get($userId);
-        $usersCharacteristics =  array_merge($context['productions'], $context['characteristics']);
+        $usersCharacteristics = array_merge($context['productions'], $context['characteristics']);
         $uuidsUserCharacteristics = array_column($usersCharacteristics, 'uuid');
         $role = last($user['roles']);
 
@@ -143,9 +143,9 @@ 
         $uri = config('wiki.api_uri').'?action=query&list=search&srwhat=text&srsearch='.$qry.'&srqiprofile=classic_noboostlinks&srnamespace=3000&format=json';
         $response = $client->get($uri);
         $content = json_decode($response->getBody()->getContents(), true);
-        if(isset($content['query']['search'])){
+        if (isset($content['query']['search'])) {
             $results = array_column($content['query']['search'], 'title');
-            return array_map(function ($item){
+            return array_map(function($item) {
                 return str_replace('Structure:', '', $item);
             }, $results);
         }

app/Src/UseCases/Domain/Context/UseCases/AddCharacteristicsToContext.php

Spacing +2 added lines, -2 removed lines

@@ -38,9 +38,9 @@
         $pages = $this->pageRepository->getByIds($pagesIds);
 
         $characteristics = [];
-        foreach ($pages as $page){
+        foreach ($pages as $page) {
             $characteristic = $this->characteristicsRepository->getByPageId($page->pageId());
-            if(!isset($characteristic)){
+            if (!isset($characteristic)) {
                 $type = $page->type() === 'Culture' ? Characteristic::FARMING_TYPE : Characteristic::CROPPING_SYSTEM;
                 $characteristic = new Characteristic(
                     Uuid::uuid4(),

app/Src/UseCases/Domain/Context/Queries/GetUserPractises.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
     {
         $practisesToReturn = [];
         $practises = $this->interactionsRepository->getDoneByUser($userId);
-        foreach ($practises as $practise){
+        foreach ($practises as $practise) {
             $year = $practise->doneAt() !== null ? $practise->doneAt()->format('Y') : 'Non datée';
             $practisesToReturn[$year][] = $practise->toArray();
         }

app/Console/Commands/SyncDryPagesFromWiki.php

Spacing +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
     {
         $httpClient = new Client();
 
-        PageModel::query()->where('dry', true)->chunkById(50, function ($items, $count) use($httpClient){
+        PageModel::query()->where('dry', true)->chunkById(50, function($items, $count) use($httpClient){
             $this->info(($count*50).' Pages');
             $pages = $items->pluck('page_id')->toArray();
             $pagesApiUri = config('wiki.api_uri').$this->queryPages.implode('|', $pages);
@@ -32,7 +32,7 @@ 
             $content = json_decode($response->getBody()->getContents(), true);
             $wikiPages = $content['query']['pages'];
 
-            foreach($wikiPages as $page){
+            foreach ($wikiPages as $page) {
                 $pageModel = PageModel::query()->where('page_id', $page['pageid'])->first();
 
                 if (!isset($page['title']))

app/Src/UseCases/Infra/Sql/ReportingCharacteristicSql.php

Spacing +8 added lines, -8 removed lines

@@ -18,12 +18,12 @@ 
             ')
             ->join('users', 'users.id', 'interactions.user_id')
             ->join('contexts', 'users.context_id', 'contexts.id')
-            ->where(function ($query) use ($type){
-                $query->when($type === 'follow', function ($query) {
+            ->where(function($query) use ($type){
+                $query->when($type === 'follow', function($query) {
                     $query->where('follow', true);
                     $query->orWhere('done', true);
                 })
-                        ->when($type === 'do', function ($query) {
+                        ->when($type === 'do', function($query) {
                             $query->where('done', true);
                         });
             })
@@ -33,7 +33,7 @@ 
             ->get();
 
         $interactionsToReturn = [];
-        foreach($interactions as $interaction){
+        foreach ($interactions as $interaction) {
             $characteristicsModel = CharacteristicsModel::query()->where('code', $interaction->department)->first();
 
             // FIXME : This test is only necessary because corsica breaks the line above. When corsica is correctly handled,
@@ -58,12 +58,12 @@ 
             ->join('contexts', 'users.context_id', 'contexts.id')
             ->join('user_characteristics', 'user_characteristics.user_id', 'users.id')
             ->join('characteristics', 'characteristics.id', 'user_characteristics.characteristic_id')
-            ->where(function ($query) use ($type){
-                $query->when($type === 'follow', function ($query) {
+            ->where(function($query) use ($type){
+                $query->when($type === 'follow', function($query) {
                     $query->where('follow', true);
                     $query->orWhere('done', true);
                 })
-                        ->when($type === 'do', function ($query) {
+                        ->when($type === 'do', function($query) {
                             $query->where('done', true);
                         });
             })
@@ -76,7 +76,7 @@ 
             ->get();
 
         $characteristicsToReturn = [];
-        foreach($characteristicsCount as $characteristicCount){
+        foreach ($characteristicsCount as $characteristicCount) {
             $characteristic = CharacteristicsModel::query()->find($characteristicCount->characteristic_id);
             $characteristic->count = $characteristicCount->count;
             $c = $characteristic->toArray();

Braces +3 added lines, -2 removed lines

@@ -38,8 +38,9 @@
 
             // FIXME : This test is only necessary because corsica breaks the line above. When corsica is correctly handled,
             // we should remove the test for emptyness.
-            if (!empty($characteristicsModel))
-                $characteristicsModel->icon = route('api.icon.serve', ['id' => $characteristicsModel->uuid]);
+            if (!empty($characteristicsModel)) {
+                            $characteristicsModel->icon = route('api.icon.serve', ['id' => $characteristicsModel->uuid]);
+            }
 
             $interaction->departmentData = $characteristicsModel;
             $interactionsToReturn[] = $interaction->toArray();

app/User.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
     public function adminlte_image()
     {
         $urlPicture = $this->path_picture != "" ? asset('storage/'.str_replace('app/public/', '', $this->path_picture)) : null;
-        if(!isset($urlPicture) || $urlPicture === ""){
+        if (!isset($urlPicture) || $urlPicture === "") {
             $urlPicture = url('').'/'.config('adminlte.logo_img');
         }
         return $urlPicture;
@@ -45,7 +45,7 @@ 
     public function adminlte_desc()
     {
         $desc = $this->firstname.' '.$this->lastname;
-        if($this->organization_id !== null){
+        if ($this->organization_id !== null) {
             $organization = app(OrganizationRepository::class)->get($this->organization_id);
             $desc .= ' - organisme : '.$organization->name();
         }

app/Src/UseCases/Domain/User.php

Spacing +10 added lines, -10 removed lines

@@ -61,14 +61,14 @@ 
         return $this->firstname.' '.$this->lastname;
     }
 
-    public function organizationId():?string
+    public function organizationId(): ?string
     {
         return $this->organizationId;
     }
 
     public function provider(string $provider, string $providerId):bool
     {
-        if(isset($this->providers[$provider]) && $this->providers[$provider] == $providerId){
+        if (isset($this->providers[$provider]) && $this->providers[$provider] == $providerId) {
             return true;
         }
         return false;
@@ -87,8 +87,8 @@ 
 
     public function create(string $passwordHashed = null, Picture $picture = null)
     {
-        if(isset($picture)) {
-            $picture->resize('app/public/users/' . $this->id);
+        if (isset($picture)) {
+            $picture->resize('app/public/users/'.$this->id);
             $this->pathPicture = $picture->relativePath();
         }
         app(UserRepository::class)->add($this, $passwordHashed);
@@ -130,20 +130,20 @@ 
         $this->email = $email;
         $this->firstname = $firstname;
         $this->lastname = $lastname;
-        if($pathPicture !== "") {
+        if ($pathPicture !== "") {
             $picture = new Picture($pathPicture);
-            $picture->resize('app/public/users/' . $this->id . '.' . $ext);
-            $this->pathPicture = 'app/public/users/' . $this->id . '.' . $ext;
+            $picture->resize('app/public/users/'.$this->id.'.'.$ext);
+            $this->pathPicture = 'app/public/users/'.$this->id.'.'.$ext;
         }
         app(UserRepository::class)->update($this);
     }
 
     public function updateAvatar(string $pathPicture, string $ext = 'jpg')
     {
-        if($pathPicture !== "") {
+        if ($pathPicture !== "") {
             $picture = new Picture($pathPicture);
-            $picture->resize('app/public/users/' . $this->id . '.' . $ext);
-            $this->pathPicture = 'app/public/users/' . $this->id . '.' . $ext;
+            $picture->resize('app/public/users/'.$this->id.'.'.$ext);
+            $this->pathPicture = 'app/public/users/'.$this->id.'.'.$ext;
         }
         app(UserRepository::class)->update($this);
         return $this->pathPicture;

app/Src/UseCases/Domain/Ports/ContextRepository.php

Spacing +2 added lines, -2 removed lines

@@ -9,8 +9,8 @@
 
 interface ContextRepository
 {
-    public function getByUser(string $userId):?Context;
+    public function getByUser(string $userId): ?Context;
     public function add(Context $context, string $userId);
     public function update(Context $context, string $userId);
-    public function getByUserDto(string $userId):?ContextDto;
+    public function getByUserDto(string $userId): ?ContextDto;
 }