LaravelFreelancerNL / laravel-arangodb

src/Query/Concerns/BuildsInserts.php

<?php

declare(strict_types=1);

namespace LaravelFreelancerNL\Aranguent\Query\Concerns;

use LaravelFreelancerNL\Aranguent\Query\Grammar;
use LaravelFreelancerNL\FluentAQL\Exceptions\BindException;

trait BuildsInserts
{
    /**
     * Insert a new record into the database.
     *
     * @param array<mixed> $values
     * @throws BindException
     */
    public function insert(array $values): bool
    {
        assert($this->grammar instanceof Grammar);

        if (!array_is_list($values)) {
            $values = [$values];
        }

        // Convert id to _key
        foreach ($values as $key => $value) {
            $values[$key] = $this->convertIdToKey($value);

It seems like convertIdToKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        $bindVar = $this->bindValue($values, 'insert');

It seems like bindValue() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $aql = $this->grammar->compileInsert($this, $values, $bindVar);
        return $this->getConnection()->insert($aql, $this->getBindings());

It seems like getBindings() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like getConnection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Insert a new record and get the value of the primary key.
     *
     * @param  array<mixed>  $values
     * @param  string|null  $sequence
     * @return int|string
     */
    public function insertGetId(array $values, $sequence = null)
    {
        assert($this->grammar instanceof Grammar);

        if (!array_is_list($values)) {
            $values = [$values];
        }

        // Convert id to _key
        foreach ($values as $key => $value) {
            $values[$key] = $this->convertIdToKey($value);
        }

        $bindVar = $this->bindValue($values, 'insert');

        $aql = $this->grammar->compileInsertGetId($this, $values, $sequence, $bindVar);

$this of type LaravelFreelancerNL\Aranguent\Query\Concerns\BuildsInserts is incompatible with the type Illuminate\Database\Query\Builder expected by parameter $query of LaravelFreelancerNL\Aranguent\Query\Grammar::compileInsertGetId().

        $response = $this->getConnection()->execute($aql, $this->getBindings());

        return (is_array($response)) ? end($response) : $response;
    }

    /**
     * Insert a new record into the database.
     *
     * @param array<mixed> $values
     *
     * @throws BindException
     */
    /** @phpstan-ignore-next-line */
    public function insertOrIgnore(array $values): bool
    {
        assert($this->grammar instanceof Grammar);

        if (!array_is_list($values)) {
            $values = [$values];
        }

        // Convert id to _key
        foreach ($values as $key => $value) {
            $values[$key] = $this->convertIdToKey($value);
        }

        $bindVar = $this->bindValue($values, 'insert');


        $aql = $this->grammar->compileInsertOrIgnore($this, $values, $bindVar);

$this of type LaravelFreelancerNL\Aranguent\Query\Concerns\BuildsInserts is incompatible with the type Illuminate\Database\Query\Builder expected by parameter $query of LaravelFreelancerNL\Aranguent\Query\Grammar::compileInsertOrIgnore().

        return $this->getConnection()->insert($aql, $this->getBindings());
    }
}