feat: register all app services as singleton

CHANGELOG

@@ -13,6 +13,7 @@ New features:
 
 Enhancements:
 
+* Register all app services as singleton
 * Docker image: add sentry-cli and run sentry:release command if sentry is enabled
 * Refactor reminders by removing Notifications table and creating two new tables: reminder outbox and reminder sent
 * Display the name of the contact field instead of the URL (Facebook instead of https:/facebook.com/johndoe)

app/Console/Commands/ImportCSV.php

@@ -178,7 +178,7 @@ private function csvToContact($data, $account_id, $gender_id)
                 'postal_code' => $postalCode,
             ];
 
-            (new CreateAddress)->execute($request);
+            app(CreateAddress::class)->execute($request);
         }
 
         if (! empty($data[42])) {
@@ -196,7 +196,7 @@ private function csvToContact($data, $account_id, $gender_id)
 
             $specialDate = $contact->setSpecialDate('birthdate', $birthdate->format('Y'), $birthdate->format('m'), $birthdate->format('d'));
 
-            (new CreateReminder)->execute([
+            app(CreateReminder::class)->execute([
                 'account_id' => $contact->account_id,
                 'contact_id' => $contact->id,
                 'initial_date' => $specialDate->date->toDateString(),

app/Helpers/WeatherHelper.php

@@ -45,7 +45,7 @@ public static function getWeatherForAddress($address)
     private static function callWeatherAPI(Address $address)
     {
         try {
-            $weather = (new GetWeatherInformation)->execute([
+            $weather = app(GetWeatherInformation::class)->execute([
                 'place_id' => $address->place->id,
             ]);
         } catch (\Exception $e) {

app/Http/Controllers/Api/Account/ApiCompanyController.php

@@ -61,7 +61,7 @@ public function show(Request $request, $companyId)
     public function store(Request $request)
     {
         try {
-            $company = (new CreateCompany)->execute(
+            $company = app(CreateCompany::class)->execute(
                 $request->all()
                     +
                     [
@@ -89,7 +89,7 @@ public function store(Request $request)
     public function update(Request $request, $companyId)
     {
         try {
-            $company = (new UpdateCompany)->execute(
+            $company = app(UpdateCompany::class)->execute(
                 $request->all()
                     +
                     [
@@ -117,7 +117,7 @@ public function update(Request $request, $companyId)
     public function destroy(Request $request, $companyId)
     {
         try {
-            (new DestroyCompany)->execute([
+            app(DestroyCompany::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'company_id' => $companyId,
             ]);

app/Http/Controllers/Api/Account/ApiGenderController.php

@@ -61,7 +61,7 @@ public function show(Request $request, $genderId)
     public function store(Request $request)
     {
         try {
-            $gender = (new CreateGender)->execute(
+            $gender = app(CreateGender::class)->execute(
                 $request->all()
                     +
                     [
@@ -89,7 +89,7 @@ public function store(Request $request)
     public function update(Request $request, $genderId)
     {
         try {
-            $gender = (new UpdateGender)->execute(
+            $gender = app(UpdateGender::class)->execute(
                 $request->all()
                     +
                     [
@@ -117,7 +117,7 @@ public function update(Request $request, $genderId)
     public function destroy(Request $request, $genderId)
     {
         try {
-            (new DestroyGender)->execute([
+            app(DestroyGender::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'gender_id' => $genderId,
             ]);

app/Http/Controllers/Api/Account/ApiPlaceController.php

@@ -61,7 +61,7 @@ public function show(Request $request, $placeId)
     public function store(Request $request)
     {
         try {
-            $place = (new CreatePlace)->execute(
+            $place = app(CreatePlace::class)->execute(
                 $request->all()
                     +
                     [
@@ -89,7 +89,7 @@ public function store(Request $request)
     public function update(Request $request, $placeId)
     {
         try {
-            $place = (new UpdatePlace)->execute(
+            $place = app(UpdatePlace::class)->execute(
                 $request->all()
                     +
                     [
@@ -117,7 +117,7 @@ public function update(Request $request, $placeId)
     public function destroy(Request $request, $placeId)
     {
         try {
-            (new DestroyPlace)->execute([
+            app(DestroyPlace::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'place_id' => $placeId,
             ]);

app/Http/Controllers/Api/ApiContactController.php

@@ -92,7 +92,7 @@ public function show(Request $request, $id)
     public function store(Request $request)
     {
         try {
-            $contact = (new CreateContact)->execute(
+            $contact = app(CreateContact::class)->execute(
                 $request->all()
                     +
                     [
@@ -118,7 +118,7 @@ public function store(Request $request)
     public function update(Request $request, $contactId)
     {
         try {
-            $contact = (new UpdateContact)->execute(
+            $contact = app(UpdateContact::class)->execute(
                 $request->all()
                     +
                     [
@@ -148,7 +148,7 @@ public function destroy(Request $request, $contactId)
             'contact_id' => $contactId,
             'account_id' => auth()->user()->account->id,
         ];
-        (new DestroyContact)->execute($data);
+        app(DestroyContact::class)->execute($data);
 
         return $this->respondObjectDeleted($contactId);
     }

app/Http/Controllers/Api/ApiContactTagController.php

@@ -31,7 +31,7 @@ public function setTags(Request $request, $contactId)
             });
 
         foreach ($tags as $tag) {
-            (new AssociateTag)->execute([
+            app(AssociateTag::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'contact_id' => $contact->id,
                 'name' => $tag,
@@ -60,7 +60,7 @@ public function unsetTags(Request $request, $contactId)
         $contactTags = $contact->tags()->get();
 
         foreach ($contactTags as $tag) {
-            (new DetachTag)->execute([
+            app(DetachTag::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'contact_id' => $contact->id,
                 'tag_id' => $tag->id,
@@ -89,7 +89,7 @@ public function unsetTag(Request $request, $contactId)
             });
 
         foreach ($tags as $tag) {
-            (new DetachTag)->execute([
+            app(DetachTag::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'contact_id' => $contact->id,
                 'tag_id' => $tag,

app/Http/Controllers/Api/ApiReminderController.php

@@ -59,7 +59,7 @@ public function show(Request $request, $reminderId)
     public function store(Request $request)
     {
         try {
-            $reminder = (new CreateReminder)->execute(
+            $reminder = app(CreateReminder::class)->execute(
                 $request->all()
                     +
                     [
@@ -86,7 +86,7 @@ public function store(Request $request)
     public function update(Request $request, $reminderId)
     {
         try {
-            $reminder = (new UpdateReminder)->execute(
+            $reminder = app(UpdateReminder::class)->execute(
                 $request->all()
                     +
                     [
@@ -113,7 +113,7 @@ public function update(Request $request, $reminderId)
     public function destroy(Request $request, $reminderId)
     {
         try {
-            (new DestroyReminder)->execute([
+            app(DestroyReminder::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'reminder_id' => $reminderId,
             ]);

app/Http/Controllers/Api/ApiTagController.php

@@ -60,7 +60,7 @@ public function show(Request $request, $id)
     public function store(Request $request)
     {
         try {
-            $tag = (new CreateTag)->execute(
+            $tag = app(CreateTag::class)->execute(
                 $request->all()
                     +
                     [
@@ -84,7 +84,7 @@ public function store(Request $request)
     public function update(Request $request, $id)
     {
         try {
-            $tag = (new UpdateTag)->execute(
+            $tag = app(UpdateTag::class)->execute(
                 $request->all()
                     +
                     [
@@ -109,7 +109,7 @@ public function update(Request $request, $id)
     public function destroy(Request $request, $id)
     {
         try {
-            (new DestroyTag)->execute([
+            app(DestroyTag::class)->execute([
                 'tag_id' => $id,
                 'account_id' => auth()->user()->account->id,
             ]);

app/Http/Controllers/Api/ApiTaskController.php

@@ -59,7 +59,7 @@ public function show(Request $request, $taskId)
     public function store(Request $request)
     {
         try {
-            $task = (new CreateTask)->execute([
+            $task = app(CreateTask::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'contact_id' => ($request->get('contact_id') == '' ? null : $request->get('contact_id')),
                 'title' => $request->get('title'),
@@ -83,7 +83,7 @@ public function store(Request $request)
     public function update(Request $request, $taskId)
     {
         try {
-            $task = (new UpdateTask)->execute(
+            $task = app(UpdateTask::class)->execute(
                 $request->all()
                     +
                     [
@@ -108,7 +108,7 @@ public function update(Request $request, $taskId)
     public function destroy(Request $request, $taskId)
     {
         try {
-            (new DestroyTask)->execute([
+            app(DestroyTask::class)->execute([
                 'task_id' => $taskId,
                 'account_id' => auth()->user()->account->id,
             ]);

app/Http/Controllers/Api/Contact/ApiAddressController.php

@@ -61,7 +61,7 @@ public function show(Request $request, $id)
     public function store(Request $request)
     {
         try {
-            $address = (new CreateAddress)->execute(
+            $address = app(CreateAddress::class)->execute(
                 $request->all()
                     +
                     [
@@ -88,7 +88,7 @@ public function store(Request $request)
     public function update(Request $request, $addressId)
     {
         try {
-            $address = (new UpdateAddress)->execute(
+            $address = app(UpdateAddress::class)->execute(
                 $request->all()
                     +
                     [
@@ -115,7 +115,7 @@ public function update(Request $request, $addressId)
     public function destroy(Request $request, $addressId)
     {
         try {
-            (new DestroyAddress)->execute([
+            app(DestroyAddress::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'address_id' => $addressId,
             ]);

app/Http/Controllers/Api/Contact/ApiCallController.php

@@ -62,7 +62,7 @@ public function show(Request $request, $callId)
     public function store(Request $request)
     {
         try {
-            $call = (new CreateCall)->execute(
+            $call = app(CreateCall::class)->execute(
                 $request->all()
                     +
                     [
@@ -90,7 +90,7 @@ public function store(Request $request)
     public function update(Request $request, $callId)
     {
         try {
-            $call = (new UpdateCall)->execute(
+            $call = app(UpdateCall::class)->execute(
                 $request->all()
                     +
                     [
@@ -118,7 +118,7 @@ public function update(Request $request, $callId)
     public function destroy(Request $request, $callId)
     {
         try {
-            (new DestroyCall)->execute([
+            app(DestroyCall::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'call_id' => $callId,
             ]);

app/Http/Controllers/Api/Contact/ApiConversationController.php

@@ -88,7 +88,7 @@ public function show(Request $request, $conversationId)
     public function store(Request $request)
     {
         try {
-            $conversation = (new CreateConversation)->execute(
+            $conversation = app(CreateConversation::class)->execute(
                 $request->all()
                 +
                 [
@@ -116,7 +116,7 @@ public function store(Request $request)
     public function update(Request $request, $conversationId)
     {
         try {
-            $conversation = (new UpdateConversation)->execute(
+            $conversation = app(UpdateConversation::class)->execute(
                 $request->all()
                 +
                 [
@@ -145,7 +145,7 @@ public function update(Request $request, $conversationId)
     public function destroy(Request $request, $conversationId)
     {
         try {
-            (new DestroyConversation)->execute([
+            app(DestroyConversation::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'conversation_id' => $conversationId,
             ]);

app/Http/Controllers/Api/Contact/ApiLifeEventController.php

@@ -55,7 +55,7 @@ public function show(Request $request, $lifeEventId)
     public function store(Request $request)
     {
         try {
-            $lifeEvent = (new CreateLifeEvent)->execute(
+            $lifeEvent = app(CreateLifeEvent::class)->execute(
                 $request->all()
                 +
                 [
@@ -81,7 +81,7 @@ public function store(Request $request)
     public function update(Request $request, $lifeEventId)
     {
         try {
-            $lifeEvent = (new UpdateLifeEvent)->execute(
+            $lifeEvent = app(UpdateLifeEvent::class)->execute(
                 $request->all()
                     +
                     [
@@ -108,7 +108,7 @@ public function update(Request $request, $lifeEventId)
     public function destroy(Request $request, $lifeEventId)
     {
         try {
-            (new DestroyLifeEvent)->execute([
+            app(DestroyLifeEvent::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'life_event_id' => $lifeEventId,
             ]);

app/Http/Controllers/Api/Contact/ApiMessageController.php

@@ -31,7 +31,7 @@ public function store(Request $request, int $conversationId)
         }
 
         try {
-            (new AddMessageToConversation)->execute(
+            app(AddMessageToConversation::class)->execute(
                 $request->all()
                 +
                 [
@@ -69,7 +69,7 @@ public function update(Request $request, int $conversationId, int $messageId)
         }
 
         try {
-            (new UpdateMessage)->execute(
+            app(UpdateMessage::class)->execute(
                 $request->all()
                 +
                 [
@@ -108,7 +108,7 @@ public function destroy(Request $request, int $conversationId, int $messageId)
         }
 
         try {
-            (new DestroyMessage)->execute([
+            app(DestroyMessage::class)->execute([
                 'account_id' => auth()->user()->account->id,
                 'conversation_id' => $conversationId,
                 'message_id' => $messageId,