bavix / laravel-entry

src/Jobs/BulkWriter.php

<?php

namespace Bavix\Entry\Jobs;

use Bavix\LaravelClickHouse\Database\Eloquent\Model as Entry;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Illuminate\Bus\Queueable;

class BulkWriter implements ShouldQueue
{

    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by Bavix\Entry\Jobs\BulkWriter: $id, $relations, $class, $keyBy

    /**
     * @var Entry
     */
    protected $entry;

    /**
     * @var array[]
     */
    protected $data;

    /**
     * Create a new job instance.
     *
     * @param Entry $entry
     * @param array[] $data
     * @return void
     */
    public function __construct(Entry $entry, ?array $data = null)
    {
        if ($data === null) {
            $data = $entry->toArray();
        }

        $this->entry = $entry;
        $this->data = $data;
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle(): void
    {
        if ($this->data) {

The expression $this->data of type array<mixed,array> is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            \array_walk_recursive($this->data, static function (&$value) {
                $value = $value ?? raw('NULL');
            });

            $this->entry::insert($this->data);
        }
    }

}