From ab16dcac6106124b017355b1addd4934b97afe4b Mon Sep 17 00:00:00 2001
From: merdan <merdan.m@gmail.com>
Date: Fri, 29 Jul 2022 12:47:47 +0500
Subject: [PATCH] Account profile with strategy pattern

---
 app/AccountTypes/AccountTypesService.php      |  6 ++++
 app/AccountTypes/BusinessAccount.php          | 17 ++++++++++
 app/AccountTypes/CompanyAccount.php           | 18 ++++++++++
 .../Controllers/API/AccountController.php     | 34 +++++++++++++++++++
 .../Controllers/{ => API}/AuthController.php  | 12 ++++---
 app/Http/Requests/API/RegisterRequest.php     |  2 +-
 app/Http/Requests/BankAccountRequest.php      | 30 ++++++++++++++++
 app/Http/Requests/ContactsRequest.php         | 30 ++++++++++++++++
 app/Http/Resources/AccountResource.php        | 32 +++++++++++++++++
 .../Resources/BusinessProfileResource.php     | 19 +++++++++++
 app/Http/Resources/CompanyProfileResource.php | 20 +++++++++++
 app/Models/Account.php                        |  2 +-
 app/Services/AccountTypesService.php          |  6 ----
 app/Services/Business.php                     | 12 -------
 app/Services/Company.php                      | 13 -------
 config/services.php                           | 10 ++++--
 routes/api.php                                | 19 ++++++++---
 17 files changed, 238 insertions(+), 44 deletions(-)
 create mode 100644 app/AccountTypes/AccountTypesService.php
 create mode 100644 app/AccountTypes/BusinessAccount.php
 create mode 100644 app/AccountTypes/CompanyAccount.php
 create mode 100644 app/Http/Controllers/API/AccountController.php
 rename app/Http/Controllers/{ => API}/AuthController.php (98%)
 create mode 100644 app/Http/Requests/BankAccountRequest.php
 create mode 100644 app/Http/Requests/ContactsRequest.php
 create mode 100644 app/Http/Resources/AccountResource.php
 create mode 100644 app/Http/Resources/BusinessProfileResource.php
 create mode 100644 app/Http/Resources/CompanyProfileResource.php
 delete mode 100644 app/Services/AccountTypesService.php
 delete mode 100644 app/Services/Business.php
 delete mode 100644 app/Services/Company.php

diff --git a/app/AccountTypes/AccountTypesService.php b/app/AccountTypes/AccountTypesService.php
new file mode 100644
index 00000000..711994ee
--- /dev/null
+++ b/app/AccountTypes/AccountTypesService.php
@@ -0,0 +1,6 @@
+<?php
+namespace App\AccountTypes;
+interface AccountTypesService
+{
+    public function create(array $data) : AccountTypesService;
+}
diff --git a/app/AccountTypes/BusinessAccount.php b/app/AccountTypes/BusinessAccount.php
new file mode 100644
index 00000000..2bc614db
--- /dev/null
+++ b/app/AccountTypes/BusinessAccount.php
@@ -0,0 +1,17 @@
+<?php
+
+namespace App\AccountTypes;
+
+use App\Http\Resources\BusinessProfileResource;
+
+class BusinessAccount implements AccountTypesService
+{
+
+    public function create(array $data) : AccountTypesService
+    {
+        $account = new BusinessAccount();
+
+        return $account->fill($data);
+    }
+
+}
diff --git a/app/AccountTypes/CompanyAccount.php b/app/AccountTypes/CompanyAccount.php
new file mode 100644
index 00000000..319a1881
--- /dev/null
+++ b/app/AccountTypes/CompanyAccount.php
@@ -0,0 +1,18 @@
+<?php
+
+namespace App\AccountTypes;
+
+use App\Http\Resources\CompanyProfileResource;
+
+class CompanyAccount implements AccountTypesService
+{
+
+    public function create(array $data) : AccountTypesService
+    {
+        $account = new CompanyAccount();
+
+        return $account->fill($data);
+    }
+
+}
+
diff --git a/app/Http/Controllers/API/AccountController.php b/app/Http/Controllers/API/AccountController.php
new file mode 100644
index 00000000..e479ae77
--- /dev/null
+++ b/app/Http/Controllers/API/AccountController.php
@@ -0,0 +1,34 @@
+<?php
+
+namespace App\Http\Controllers\API;
+
+use App\Http\Controllers\Controller;
+use App\Http\Requests\BankAccountRequest;
+use App\Http\Requests\ContactsRequest;
+use App\Http\Resources\AccountResource;
+use Illuminate\Http\Request;
+
+class AccountController extends Controller
+{
+    public function account(Request $request) : AccountResource
+    {
+        $account = $request->user()
+            ->account()
+            ->with('profile');
+
+        return AccountResource::make($account);
+
+    }
+
+    public function storeContacts(ContactsRequest $request){
+
+    }
+
+    public function storeBankAccount(BankAccountRequest $request){
+
+    }
+
+    public function storeProfile(){
+
+    }
+}
diff --git a/app/Http/Controllers/AuthController.php b/app/Http/Controllers/API/AuthController.php
similarity index 98%
rename from app/Http/Controllers/AuthController.php
rename to app/Http/Controllers/API/AuthController.php
index 2dc1cf50..1620cde0 100644
--- a/app/Http/Controllers/AuthController.php
+++ b/app/Http/Controllers/API/AuthController.php
@@ -1,6 +1,6 @@
 <?php
 
-namespace App\Http\Controllers;
+namespace App\Http\Controllers\API;
 
 use App\Http\Requests\API\LoginRequest;
 use App\Http\Requests\API\RegisterRequest;
@@ -153,7 +153,8 @@ public function login(LoginRequest $request){
         *     )
         * )
     */
-    public function register(RegisterRequest $request){
+    public function register(RegisterRequest $request) : ClientResource
+    {
 
         $data = $request->only(['email','firstname','lastname']);
 
@@ -165,10 +166,11 @@ public function register(RegisterRequest $request){
 
         $data['verification_token'] = rand(10000, 99999);
 
-        $client = Client::create($data);
+        $account = Account::create($request->only(['country_id','account_type']));
 
-        //create account
-//        $account = new Account()
+        $data['account_id'] = $account->id;
+
+        $client = Client::create($data);
 
         if($email_verification)
         {
diff --git a/app/Http/Requests/API/RegisterRequest.php b/app/Http/Requests/API/RegisterRequest.php
index 4fe52ecc..0cd23bc7 100644
--- a/app/Http/Requests/API/RegisterRequest.php
+++ b/app/Http/Requests/API/RegisterRequest.php
@@ -30,7 +30,7 @@ public function rules()
             'lastname' => 'required',
             'account_type' => 'required|in:company,business',
             'country' => 'required',
-            'legalization_number' => 'required_if:is_registered,on'
+//            'legalization_number' => 'required_if:is_registered,on'
         ];
     }
 }
diff --git a/app/Http/Requests/BankAccountRequest.php b/app/Http/Requests/BankAccountRequest.php
new file mode 100644
index 00000000..2152bed0
--- /dev/null
+++ b/app/Http/Requests/BankAccountRequest.php
@@ -0,0 +1,30 @@
+<?php
+
+namespace App\Http\Requests;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class BankAccountRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     *
+     * @return bool
+     */
+    public function authorize()
+    {
+        return false;
+    }
+
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array<string, mixed>
+     */
+    public function rules()
+    {
+        return [
+            //
+        ];
+    }
+}
diff --git a/app/Http/Requests/ContactsRequest.php b/app/Http/Requests/ContactsRequest.php
new file mode 100644
index 00000000..69672844
--- /dev/null
+++ b/app/Http/Requests/ContactsRequest.php
@@ -0,0 +1,30 @@
+<?php
+
+namespace App\Http\Requests;
+
+use Illuminate\Foundation\Http\FormRequest;
+
+class ContactsRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     *
+     * @return bool
+     */
+    public function authorize()
+    {
+        return false;
+    }
+
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array<string, mixed>
+     */
+    public function rules()
+    {
+        return [
+            //
+        ];
+    }
+}
diff --git a/app/Http/Resources/AccountResource.php b/app/Http/Resources/AccountResource.php
new file mode 100644
index 00000000..1b341c80
--- /dev/null
+++ b/app/Http/Resources/AccountResource.php
@@ -0,0 +1,32 @@
+<?php
+
+namespace App\Http\Resources;
+
+use Illuminate\Http\Resources\Json\JsonResource;
+
+class AccountResource extends JsonResource
+{
+    /**
+     * Transform the resource into an array.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @return array|\Illuminate\Contracts\Support\Arrayable|\JsonSerializable
+     */
+    public function toArray($request)
+    {
+        return [
+            'id'           => $this->id,
+            'bank_account' => $this->bank_account,
+            'contacts'     => $this->contacts,
+            'account_type' => $this->account_type,
+            'profile'      => $this->profile()
+        ];
+    }
+
+    private function profile() : JsonResource
+    {
+        $type = config('services.account.'.$this->account_type.'.resource');
+
+        return $type::make($this->profile);
+    }
+}
diff --git a/app/Http/Resources/BusinessProfileResource.php b/app/Http/Resources/BusinessProfileResource.php
new file mode 100644
index 00000000..117316bf
--- /dev/null
+++ b/app/Http/Resources/BusinessProfileResource.php
@@ -0,0 +1,19 @@
+<?php
+
+namespace App\Http\Resources;
+
+use Illuminate\Http\Resources\Json\JsonResource;
+
+class BusinessProfileResource extends JsonResource
+{
+    /**
+     * Transform the resource into an array.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @return array|\Illuminate\Contracts\Support\Arrayable|\JsonSerializable
+     */
+    public function toArray($request)
+    {
+        return parent::toArray($request);
+    }
+}
diff --git a/app/Http/Resources/CompanyProfileResource.php b/app/Http/Resources/CompanyProfileResource.php
new file mode 100644
index 00000000..178ce7c9
--- /dev/null
+++ b/app/Http/Resources/CompanyProfileResource.php
@@ -0,0 +1,20 @@
+<?php
+
+namespace App\Http\Resources;
+
+use Illuminate\Http\Resources\Json\JsonResource;
+
+class CompanyProfileResource extends JsonResource
+{
+    /**
+     * Transform the resource into an array.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @return array|\Illuminate\Contracts\Support\Arrayable|\JsonSerializable
+     */
+    public function toArray($request)
+    {
+        return parent::toArray($request);
+    }
+
+}
diff --git a/app/Models/Account.php b/app/Models/Account.php
index 6fc3455c..03f8e35c 100644
--- a/app/Models/Account.php
+++ b/app/Models/Account.php
@@ -20,7 +20,7 @@ class Account extends Model
     // public $timestamps = false;
     protected $guarded = ['id'];
     protected $fillable = [
-        'contacts', 'bank', 'vat', 'country_id', 'legalization_number'
+        'contacts', 'bank', 'vat', 'country_id', 'legalization_number','account_type'
     ];
     protected $casts = [
         'contacts' => 'array',
diff --git a/app/Services/AccountTypesService.php b/app/Services/AccountTypesService.php
deleted file mode 100644
index 121cf2ee..00000000
--- a/app/Services/AccountTypesService.php
+++ /dev/null
@@ -1,6 +0,0 @@
-<?php
-namespace App\Services;
-interface AccountTypesService
-{
-    public function create(array $data);
-}
diff --git a/app/Services/Business.php b/app/Services/Business.php
deleted file mode 100644
index 5de1eeae..00000000
--- a/app/Services/Business.php
+++ /dev/null
@@ -1,12 +0,0 @@
-<?php
-
-namespace App\Services;
-
-class Business implements AccountTypesService
-{
-
-    public function create(array $data)
-    {
-        return new Business();
-    }
-}
diff --git a/app/Services/Company.php b/app/Services/Company.php
deleted file mode 100644
index 6eea4934..00000000
--- a/app/Services/Company.php
+++ /dev/null
@@ -1,13 +0,0 @@
-<?php
-
-namespace App\Services;
-
-class Company implements AccountTypesService
-{
-
-    public function create(array $data)
-    {
-        return new Company();
-    }
-}
-
diff --git a/config/services.php b/config/services.php
index 0bf6127f..a741b739 100644
--- a/config/services.php
+++ b/config/services.php
@@ -32,8 +32,14 @@
     ],
 
     'account' => [
-        'business' => App\Services\Business::class,
-        'company' => App\Services\Company::class,
+        'business' => [
+            'class' => App\AccountTypes\BusinessAccount::class,
+            'resources' =>  App\Http\Resources\BusinessProfileResource::class
+        ],
+        'company' => [
+            'class' => App\AccountTypes\CompanyAccount::class,
+            'resources' => App\Http\Resources\CompanyProfileResource::class
+        ]
     ]
 
 ];
diff --git a/routes/api.php b/routes/api.php
index 894179af..793f0ac7 100644
--- a/routes/api.php
+++ b/routes/api.php
@@ -1,8 +1,7 @@
 <?php
 
-use App\Http\Controllers\AuthController;
-use App\Http\Controllers\TestController;
-use Illuminate\Http\Request;
+use App\Http\Controllers\API\AccountController;
+use App\Http\Controllers\API\AuthController;
 use Illuminate\Support\Facades\Route;
 
 /*
@@ -26,10 +25,22 @@
 Route::post('/reset-password', [AuthController::class, 'updatePassword']);
 Route::post('/forgot-password', [AuthController::class, 'sendPasswordResetLinkEmail']);
 Route::post('/verify-email', [AuthController::class, 'verifyEmail']);
-Route::get('/users', [TestController::class, 'users']);
+
 
 Route::middleware(['auth.client','auth:api'])->group(function () {
+    /**
+     * Client endpoints
+     */
     Route::get('/client', [AuthController::class, 'client']);
     Route::post('/logout', [AuthController::class, 'logout']);
     Route::post('/update-client', [AuthController::class, 'updateClient']);
+    /**
+     * Account endpoints
+     */
+    Route::group(['prefix' => 'account'],function () {
+        Route::get('/',[AccountController::class, 'account']);
+        Route::put('contacts',[AccountController::class,'storeContacts']);
+        Route::put('bank',[AccountController::class,'storeBankAccount']);
+    });
+
 });