formatting

Author: limenet

src/Commands/IndexCommand.php

@@ -14,36 +14,18 @@
 
 class IndexCommand extends Command
 {
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
     protected $signature = 'elastica-bridge:index {index?*} {--delete}';
 
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
     protected $description = 'Re-create the ES index and populate with data';
 
-    /**
-     * Create a new command instance.
-     *
-     * @return void
-     */
-    public function __construct(protected ElasticaClient $elastica, protected IndexRepository $indexRepository)
-    {
+    public function __construct(
+        protected ElasticaClient $elastica,
+        protected IndexRepository $indexRepository
+    ) {
         parent::__construct();
     }
 
-    /**
-     * Execute the console command.
-     *
-     * @return int
-     */
-    public function handle()
+    public function handle(): int
     {
         foreach ($this->indexRepository->all() as $indexConfig) {
             if (
@@ -68,6 +50,6 @@ public function handle()
                 ->dispatch();
         }
 
-        return 0;
+        return self::SUCCESS;
     }
 }

src/Commands/StatusCommand.php

@@ -12,36 +12,18 @@
 
 class StatusCommand extends Command
 {
-    /**
-     * The name and signature of the console command.
-     *
-     * @var string
-     */
     protected $signature = 'elastica-bridge:status';
 
-    /**
-     * The console command description.
-     *
-     * @var string
-     */
     protected $description = 'Displays the status of the configured Elasticsearch indices';
 
-    /**
-     * Create a new command instance.
-     *
-     * @return void
-     */
-    public function __construct(protected ElasticaClient $elastica, protected IndexRepository $indexRepository)
-    {
+    public function __construct(
+        protected ElasticaClient $elastica,
+        protected IndexRepository $indexRepository
+    ) {
         parent::__construct();
     }
 
-    /**
-     * Execute the console command.
-     *
-     * @return int
-     */
-    public function handle()
+    public function handle(): int
     {
         $table = new Table($this->output);
         $table
@@ -100,7 +82,7 @@ public function handle()
             ->setHeaderTitle('Indices');
         $table->render();
 
-        return 0;
+        return self::SUCCESS;
     }
 
     protected function formatBoolean(bool $val): string

src/Jobs/ActivateIndex.php

@@ -8,19 +8,10 @@
 
 class ActivateIndex extends AbstractIndexJob
 {
-    /**
-     * Create a new job instance.
-     *
-     * @return void
-     */
     public function __construct(protected IndexInterface $indexConfig)
     {
-        //
     }
 
-    /**
-     * Execute the job.
-     */
     public function handle(): void
     {
         $oldIndex = $this->indexConfig->getBlueGreenActiveElasticaIndex();

src/Jobs/PopulateBatchIndex.php

@@ -22,19 +22,15 @@ class PopulateBatchIndex implements ShouldQueue
     use Queueable;
     use SerializesModels;
 
-    /**
-     * Create a new job instance.
-     *
-     * @return void
-     */
-    public function __construct(protected Index $index, protected IndexInterface $indexConfig, protected string $indexDocument, protected int $limit, protected int $offset)
-    {
-        //
+    public function __construct(
+        protected Index $index,
+        protected IndexInterface $indexConfig,
+        protected string $indexDocument,
+        protected int $limit,
+        protected int $offset
+    ) {
     }
 
-    /**
-     * Execute the job.
-     */
     public function handle(): void
     {
         if ($this->batch()?->cancelled()) {

src/Jobs/PopulateIndex.php

@@ -11,19 +11,10 @@ class PopulateIndex extends AbstractIndexJob
 {
     use Batchable;
 
-    /**
-     * Create a new job instance.
-     *
-     * @return void
-     */
     public function __construct(protected IndexInterface $indexConfig)
     {
-        //
     }
 
-    /**
-     * Execute the job.
-     */
     public function handle(): void
     {
         if ($this->batch()?->cancelled()) {
@@ -41,7 +32,13 @@ public function handle(): void
             $modelCount = $indexDocument::count();
 
             for ($batchNumber = 0; $batchNumber < ceil($modelCount / $this->indexConfig->getBatchSize()); $batchNumber++) {
-                $jobs[] = new PopulateBatchIndex($index, $this->indexConfig, $indexDocument, $this->indexConfig->getBatchSize(), $batchNumber * $this->indexConfig->getBatchSize());
+                $jobs[] = new PopulateBatchIndex(
+                    $index,
+                    $this->indexConfig,
+                    $indexDocument,
+                    $this->indexConfig->getBatchSize(),
+                    $batchNumber * $this->indexConfig->getBatchSize()
+                );
             }
         }
 

src/Jobs/SetupIndex.php

@@ -13,19 +13,12 @@ class SetupIndex extends AbstractIndexJob
 {
     use Batchable;
 
-    /**
-     * Create a new job instance.
-     *
-     * @return void
-     */
-    public function __construct(protected IndexInterface $indexConfig, protected bool $deleteExisting)
-    {
-        //
+    public function __construct(
+        protected IndexInterface $indexConfig,
+        protected bool $deleteExisting
+    ) {
     }
 
-    /**
-     * Execute the job.
-     */
     public function handle(ElasticaClient $elastica): void
     {
         if ($this->batch()?->cancelled()) {

src/Services/ModelEvent.php

@@ -17,7 +17,8 @@ class ModelEvent
     public const EVENT_SAVED = 'saved';
     public const EVENT_RESTORED = 'restored';
     public const EVENT_DELETED = 'deleted';
-    public const EVENTS = [self::EVENT_CREATED,  self::EVENT_UPDATED,  self::EVENT_SAVED,  self::EVENT_RESTORED, self::EVENT_DELETED];
+    public const EVENTS = [
+        self::EVENT_CREATED,  self::EVENT_UPDATED,  self::EVENT_SAVED,  self::EVENT_RESTORED, self::EVENT_DELETED, ];
 
     public function __construct(protected IndexRepository $indexRepository)
     {
@@ -37,7 +38,9 @@ public function handle(string $event, Model $model): void
                 $shouldBePresent = false;
             }
 
-            $shouldBePresent ? $this->ensureModelPresentInIndex($index, $model) : $this->ensureModelMissingFromIndex($index, $model);
+            $shouldBePresent
+                ? $this->ensureModelPresentInIndex($index, $model)
+                : $this->ensureModelMissingFromIndex($index, $model);
         }
     }
 
@@ -53,7 +56,6 @@ protected function ensureModelMissingFromIndex(IndexInterface $index, Model $mod
         try {
             $index->getElasticaIndex()->deleteById($model->getElasticsearchId());
         } catch (NotFoundException) {
-            //
         }
     }