Fix all coding issues

Thavarshan · Jun 22, 2021 · 5ba255a · 5ba255a
1 parent 13a26ff
commit 5ba255a
Show file tree

Hide file tree

Showing 34 changed files with 3,169 additions and 1,202 deletions.
diff --git a/app/Actions/API/CreateNewApiToken.php b/app/Actions/API/CreateNewApiToken.php
@@ -2,9 +2,9 @@
 
 namespace App\Actions\API;
 
-use InvalidArgumentException;
 use Emberfuse\Blaze\API\Permission;
 use Emberfuse\Blaze\Contracts\Actions\CreatesNewApiTokens;
+use InvalidArgumentException;
 
 class CreateNewApiToken implements CreatesNewApiTokens
 {
@@ -18,7 +18,8 @@ class CreateNewApiToken implements CreatesNewApiTokens
     public function create(array $data, ?array $options = null)
     {
         $token = optional($options['user'])->createToken(
-            $data['name'], Permission::validPermissions($data['permissions'] ?? [])
+            $data['name'],
+            Permission::validPermissions($data['permissions'] ?? [])
         );
 
         if (! is_null($token)) {

diff --git a/app/Actions/Auth/AuthenticateUser.php b/app/Actions/Auth/AuthenticateUser.php
@@ -2,9 +2,9 @@
 
 namespace App\Actions\Auth;
 
-use Illuminate\Http\Request;
-use Emberfuse\Scorch\Scorch\Config;
 use Emberfuse\Scorch\Contracts\Actions\AuthenticatesUsers;
+use Emberfuse\Scorch\Scorch\Config;
+use Illuminate\Http\Request;
 
 class AuthenticateUser extends AuthAction implements AuthenticatesUsers
 {

diff --git a/app/Actions/Auth/CreateNewUser.php b/app/Actions/Auth/CreateNewUser.php
@@ -3,11 +3,11 @@
 namespace App\Actions\Auth;
 
 use App\Models\User;
+use Emberfuse\Scorch\Contracts\Actions\CreatesNewUsers;
+use Emberfuse\Scorch\Support\Traits\Fillable;
+use Emberfuse\Scorch\Support\Util;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Hash;
-use Emberfuse\Scorch\Support\Util;
-use Emberfuse\Scorch\Support\Traits\Fillable;
-use Emberfuse\Scorch\Contracts\Actions\CreatesNewUsers;
 
 class CreateNewUser implements CreatesNewUsers
 {

diff --git a/app/Actions/Auth/DeleteUser.php b/app/Actions/Auth/DeleteUser.php
@@ -3,8 +3,8 @@
 namespace App\Actions\Auth;
 
 use App\Models\User;
-use Illuminate\Support\Facades\DB;
 use Emberfuse\Scorch\Contracts\Actions\DeletesUsers;
+use Illuminate\Support\Facades\DB;
 
 class DeleteUser implements DeletesUsers
 {

diff --git a/app/Actions/Auth/LogoutUser.php b/app/Actions/Auth/LogoutUser.php
@@ -2,9 +2,9 @@
 
 namespace App\Actions\Auth;
 
-use Illuminate\Http\Request;
-use Illuminate\Contracts\Session\Session;
 use Emberfuse\Scorch\Contracts\Actions\LogsoutUsers;
+use Illuminate\Contracts\Session\Session;
+use Illuminate\Http\Request;
 
 class LogoutUser extends AuthAction implements LogsoutUsers
 {

diff --git a/app/Actions/Auth/ResetUserPassword.php b/app/Actions/Auth/ResetUserPassword.php
@@ -2,12 +2,12 @@
 
 namespace App\Actions\Auth;
 
+use App\Actions\Auth\Traits\PasswordUpdater;
 use Closure;
+use Emberfuse\Scorch\Contracts\Actions\ResetsUserPasswords;
 use Illuminate\Auth\Events\PasswordReset;
-use App\Actions\Auth\Traits\PasswordUpdater;
-use Illuminate\Contracts\Auth\StatefulGuard;
 use Illuminate\Contracts\Auth\PasswordBroker;
-use Emberfuse\Scorch\Contracts\Actions\ResetsUserPasswords;
+use Illuminate\Contracts\Auth\StatefulGuard;
 
 class ResetUserPassword implements ResetsUserPasswords
 {
@@ -51,7 +51,8 @@ public function __construct(PasswordBroker $broker, StatefulGuard $guard)
     public function reset(array $data)
     {
         return $this->broker->reset(
-            $data, $this->resetPasswordCallback($data['password'])
+            $data,
+            $this->resetPasswordCallback($data['password'])
         );
     }
 

diff --git a/app/Actions/Auth/Traits/PasswordUpdater.php b/app/Actions/Auth/Traits/PasswordUpdater.php
@@ -2,10 +2,10 @@
 
 namespace App\Actions\Auth\Traits;
 
-use Illuminate\Support\Str;
+use Illuminate\Contracts\Auth\Authenticatable;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Hash;
-use Illuminate\Contracts\Auth\Authenticatable;
+use Illuminate\Support\Str;
 
 trait PasswordUpdater
 {

diff --git a/app/Actions/Auth/UpdateUserPassword.php b/app/Actions/Auth/UpdateUserPassword.php
@@ -2,8 +2,8 @@
 
 namespace App\Actions\Auth;
 
-use App\Models\User;
 use App\Actions\Auth\Traits\PasswordUpdater;
+use App\Models\User;
 use Emberfuse\Scorch\Contracts\Actions\UpdatesUserPasswords;
 
 class UpdateUserPassword implements UpdatesUserPasswords

diff --git a/app/Actions/Auth/UpdateUserProfile.php b/app/Actions/Auth/UpdateUserProfile.php
@@ -3,8 +3,8 @@
 namespace App\Actions\Auth;
 
 use App\Models\User;
-use Illuminate\Contracts\Auth\MustVerifyEmail;
 use Emberfuse\Scorch\Contracts\Actions\UpdatesUserProfiles;
+use Illuminate\Contracts\Auth\MustVerifyEmail;
 
 class UpdateUserProfile implements UpdatesUserProfiles
 {

diff --git a/app/Actions/Conversions/ConvertAudio.php b/app/Actions/Conversions/ConvertAudio.php
@@ -3,10 +3,10 @@
 namespace App\Actions\Conversions;
 
 use App\Models\Conversion;
-use Illuminate\Support\Str;
+use CloudConvert\CloudConvert;
 use CloudConvert\Models\Job;
 use CloudConvert\Models\Task;
-use CloudConvert\CloudConvert;
+use Illuminate\Support\Str;
 
 class ConvertAudio
 {
@@ -75,18 +75,6 @@ public function process(Conversion $conversion): void
         );
     }
 
-    /**
-     * Create new job instance.
-     *
-     * @return \App\Actions\Conversions\ConvertAudio
-     */
-    protected function createJob(): ConvertAudio
-    {
-        $this->job = (new Job())->setTag(Str::random());
-
-        return $this;
-    }
-
     /**
      * Configure task for job.
      *
@@ -117,4 +105,16 @@ public function createTask(string $process): Task
     {
         return new Task($process, $this->processes[$process]);
     }
+
+    /**
+     * Create new job instance.
+     *
+     * @return \App\Actions\Conversions\ConvertAudio
+     */
+    protected function createJob(): ConvertAudio
+    {
+        $this->job = (new Job())->setTag(Str::random());
+
+        return $this;
+    }
 }
diff --git a/app/Actions/Conversions/DownloadConverted.php b/app/Actions/Conversions/DownloadConverted.php
@@ -2,8 +2,8 @@
 
 namespace App\Actions\Conversions;
 
-use CloudConvert\Models\Job;
 use CloudConvert\CloudConvert;
+use CloudConvert\Models\Job;
 use Illuminate\Support\Facades\Storage;
 
 class DownloadConverted

diff --git a/app/Actions/Conversions/ProcessConversion.php b/app/Actions/Conversions/ProcessConversion.php
@@ -2,9 +2,9 @@
 
 namespace App\Actions\Conversions;
 
-use Illuminate\Support\Facades\DB;
 use App\Jobs\ProcessAudioConversion;
 use Emberfuse\Scorch\Contracts\Actions\CreatesNewResources;
+use Illuminate\Support\Facades\DB;
 
 class ProcessConversion implements CreatesNewResources
 {

diff --git a/app/Console/Kernel.php b/app/Console/Kernel.php
@@ -2,8 +2,8 @@
 
 namespace App\Console;
 
-use Illuminate\Console\Scheduling\Schedule;
 use Emberfuse\Scorch\Console\RequestMakeCommand;
+use Illuminate\Console\Scheduling\Schedule;
 use Illuminate\Foundation\Console\Kernel as ConsoleKernel;
 
 class Kernel extends ConsoleKernel

diff --git a/app/Events/ConversionStatusUpdated.php b/app/Events/ConversionStatusUpdated.php
@@ -2,11 +2,11 @@
 
 namespace App\Events;
 
-use App\Models\User;
 use App\Models\Conversion;
-use Illuminate\Queue\SerializesModels;
-use Illuminate\Foundation\Events\Dispatchable;
+use App\Models\User;
 use Illuminate\Broadcasting\InteractsWithSockets;
+use Illuminate\Foundation\Events\Dispatchable;
+use Illuminate\Queue\SerializesModels;
 
 class ConversionStatusUpdated
 {

diff --git a/app/Exceptions/Handler.php b/app/Exceptions/Handler.php
@@ -13,7 +13,7 @@ class Handler extends ExceptionHandler
      * @var array
      */
     protected $dontReport = [
-        //
+
     ];
 
     /**
@@ -35,7 +35,6 @@ class Handler extends ExceptionHandler
     public function register()
     {
         $this->reportable(function (Throwable $e) {
-            //
         });
     }
 }
diff --git a/app/Http/Controllers/Controller.php b/app/Http/Controllers/Controller.php
@@ -2,10 +2,10 @@
 
 namespace App\Http\Controllers;
 
+use Emberfuse\Blaze\Http\Controllers\Controller as BaseController;
+use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
 use Illuminate\Foundation\Bus\DispatchesJobs;
 use Illuminate\Foundation\Validation\ValidatesRequests;
-use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
-use Emberfuse\Blaze\Http\Controllers\Controller as BaseController;
 
 class Controller extends BaseController
 {

diff --git a/app/Http/Controllers/ConversionController.php b/app/Http/Controllers/ConversionController.php
@@ -2,12 +2,12 @@
 
 namespace App\Http\Controllers;
 
-use Inertia\Inertia;
-use App\Models\Conversion;
-use Illuminate\Http\Response;
+use App\Actions\Conversions\ProcessConversion;
 use App\Http\Requests\ConversionRequest;
 use App\Http\Responses\ConversionResponse;
-use App\Actions\Conversions\ProcessConversion;
+use App\Models\Conversion;
+use Illuminate\Http\Response;
+use Inertia\Inertia;
 
 class ConversionController extends Controller
 {
@@ -52,6 +52,7 @@ public function show(Conversion $conversion)
     {
         return Response::download(
             $conversion->download,
-            $conversion->audio_file_name);
+            $conversion->audio_file_name
+        );
     }
 }
diff --git a/app/Http/Middleware/Authenticate.php b/app/Http/Middleware/Authenticate.php
@@ -10,6 +10,7 @@ class Authenticate extends Middleware
      * Get the path the user should be redirected to when they are not authenticated.
      *
      * @param  \Illuminate\Http\Request  $request
+     *
      * @return string|null
      */
     protected function redirectTo($request)

diff --git a/app/Http/Middleware/EncryptCookies.php b/app/Http/Middleware/EncryptCookies.php
@@ -12,6 +12,6 @@ class EncryptCookies extends Middleware
      * @var array
      */
     protected $except = [
-        //
+
     ];
 }
diff --git a/app/Http/Middleware/HandleInertiaRequests.php b/app/Http/Middleware/HandleInertiaRequests.php
@@ -11,6 +11,7 @@ class HandleInertiaRequests extends Middleware
      * The root template that's loaded on the first page visit.
      *
      * @see https://inertiajs.com/server-side-setup#root-template
+     *
      * @var string
      */
     protected $rootView = 'app';
@@ -19,7 +20,9 @@ class HandleInertiaRequests extends Middleware
      * Determines the current asset version.
      *
      * @see https://inertiajs.com/asset-versioning
+     *
      * @param  \Illuminate\Http\Request  $request
+     *
      * @return string|null
      */
     public function version(Request $request)
@@ -31,13 +34,15 @@ public function version(Request $request)
      * Defines the props that are shared by default.
      *
      * @see https://inertiajs.com/shared-data
+     *
      * @param  \Illuminate\Http\Request  $request
+     *
      * @return array
      */
     public function share(Request $request)
     {
         return array_merge(parent::share($request), [
-            //
+
         ]);
     }
 }
diff --git a/app/Http/Middleware/PreventRequestsDuringMaintenance.php b/app/Http/Middleware/PreventRequestsDuringMaintenance.php
@@ -12,6 +12,6 @@ class PreventRequestsDuringMaintenance extends Middleware
      * @var array
      */
     protected $except = [
-        //
+
     ];
 }
diff --git a/app/Http/Middleware/RedirectIfAuthenticated.php b/app/Http/Middleware/RedirectIfAuthenticated.php
@@ -15,6 +15,7 @@ class RedirectIfAuthenticated
      * @param  \Illuminate\Http\Request  $request
      * @param  \Closure  $next
      * @param  string|null  ...$guards
+     *
      * @return mixed
      */
     public function handle(Request $request, Closure $next, ...$guards)

diff --git a/app/Http/Middleware/VerifyCsrfToken.php b/app/Http/Middleware/VerifyCsrfToken.php
@@ -12,6 +12,6 @@ class VerifyCsrfToken extends Middleware
      * @var array
      */
     protected $except = [
-        'webhook/cloudconvert'
+        'webhook/cloudconvert',
     ];
 }
diff --git a/app/Jobs/ProcessAudioConversion.php b/app/Jobs/ProcessAudioConversion.php
@@ -2,14 +2,14 @@
 
 namespace App\Jobs;
 
-use Throwable;
+use App\Actions\Conversions\ConvertAudio;
 use App\Models\Conversion;
 use Illuminate\Bus\Queueable;
-use Illuminate\Queue\SerializesModels;
-use Illuminate\Queue\InteractsWithQueue;
-use App\Actions\Conversions\ConvertAudio;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Foundation\Bus\Dispatchable;
+use Illuminate\Queue\InteractsWithQueue;
+use Illuminate\Queue\SerializesModels;
+use Throwable;
 
 class ProcessAudioConversion implements ShouldQueue
 {

diff --git a/app/Models/Conversion.php b/app/Models/Conversion.php
@@ -3,12 +3,12 @@
 namespace App\Models;
 
 use App\Events\ConversionStatusUpdated;
+use App\Models\Concerns\InteractsWithFiles;
+use Illuminate\Database\Eloquent\Factories\HasFactory;
 use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Support\Facades\Storage;
 use Illuminate\Support\Traits\Tappable;
-use App\Models\Concerns\InteractsWithFiles;
-use Illuminate\Database\Eloquent\Relations\BelongsTo;
-use Illuminate\Database\Eloquent\Factories\HasFactory;
 
 class Conversion extends Model
 {