LaravelFreelancerNL / arangodb-php-client

src/Transactions/SupportsTransactions.php

<?php

declare(strict_types=1);

namespace ArangoClient\Transactions;

use ArangoClient\Exceptions\ArangoException;
use ArangoClient\Http\HttpRequestOptions;
use stdClass;

trait SupportsTransactions
{
    protected ?TransactionManager $transactionManager = null;

    public function transactions(): TransactionManager
    {
        if (!(property_exists($this, 'transactionManager') && $this->transactionManager !== null)) {
            $this->transactionManager = new TransactionManager($this);
        }

        return $this->transactionManager;
    }

    /**
     * Shortcut to begin() on the transactionManager
     *
     * @param  array<string, array<string>>  $collections
     * @param  array<mixed>  $options
     *
     * @throws ArangoException
     */
    public function begin(array $collections = [], array $options = []): string
    {
        return $this->transactions()->begin($collections, $options);
    }

    /**
     * Shortcut to begin() on the transactionManager
     *
     * @param  array<string, array<string>>  $collections
     * @param  array<mixed>  $options
     *
     * @throws ArangoException
     */
    public function beginTransaction(array $collections = [], array $options = []): string
    {
        return $this->transactions()->begin($collections, $options);
    }

    /**
     * Shortcut to abort() on the transactionManager
     *
     *
     * @throws ArangoException
     */
    public function abort(?string $id = null): bool
    {
        return $this->transactions()->abort($id);
    }

    /**
     * Shortcut to abort() on the transactionManager
     *
     *
     * @throws ArangoException
     */
    public function rollBack(?string $id = null): bool
    {
        return $this->transactions()->abort($id);
    }

    /**
     * Shortcut to commit() on the transactionManager
     *
     *
     * @throws ArangoException
     */
    public function commit(?string $id = null): bool
    {
        return $this->transactions()->commit($id);
    }

    /**
     * @param  array<mixed>|HttpRequestOptions  $options
     *
     * @throws ArangoException
     */
    public function transactionAwareRequest(
        string $method,
        string $uri,
        array|HttpRequestOptions $options = [],
        ?string $database = null,
        ?int $transactionId = null,
    ): stdClass {
        if (is_array($options)) {

The condition is_array($options) is always true.

            $options = $this->prepareRequestOptions($options);

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

        }
        try {
            if (!isset($transactionId)) {
                $transactionId = $this->transactions()->getTransaction();
            }
            $options->addHeader('x-arango-trx-id', $transactionId);
        } finally {
            return $this->request($method, $uri, $options, $database);

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

        }
    }

    public function setTransactionManager(TransactionManager $transactionManager): void
    {
        $this->transactionManager = $transactionManager;
    }

    public function getTransactionManager(): ?TransactionManager
    {
        return $this->transactionManager;
    }
}