chadicus / marvel-api-client

src/Cache/MongoCache.php

<?php

namespace Chadicus\Marvel\Api\Cache;

use Chadicus\Marvel\Api\Response;
use Chadicus\Marvel\Api\ResponseInterface;
use Chadicus\Marvel\Api\RequestInterface;
use MongoDB\BSON\UTCDateTime;
use MongoDB\Collection;
use MongoDB\Model\BSONArray;

/**
 * Concrete implementation of Cache using an array.
 */
final class MongoCache extends AbstractCache implements CacheInterface
{
    /**
     * MongoCollection containing the cached responses.
     *
     * @var \MongoCollection
     */
    private $collection;

    /**
     * Construct a new instance of MongoCache.
     *
     * @param Collection $collection        The collection containing the cached data.
     * @param integer    $defaultTimeToLive The default time to live in seconds.
     */
    public function __construct(Collection $collection, $defaultTimeToLive = CacheInterface::MAX_TTL)
    {
        $this->setDefaultTTL($defaultTimeToLive);
        $this->collection = $collection;

It seems like $collection of type object<MongoDB\Collection> is incompatible with the declared type object<MongoCollection> of property $collection.

        $this->collection->createIndex(['expires' => 1], ['expireAfterSeconds' => 0, 'background' => true]);
    }

    /**
     * Store the api $response as the cached result of the api $request.
     *
     * @param RequestInterface  $request    The request for which the response will be cached.
     * @param ResponseInterface $response   The reponse to cache.
     * @param integer  $timeToLive The time in seconds that the cache should live.
     *
     * @return void
     *
     * @throws \InvalidArgumentException Throw if $timeToLive is not an integer between 0 and 86400.
     */
    public function set(RequestInterface $request, ResponseInterface $response, $timeToLive = null)
    {
        $timeToLive = self::ensureTTL($timeToLive ?: $this->getDefaultTTL());

        $id = $request->getUrl();
        $cache = [
            '_id' => $id,
            'httpCode' => $response->getHttpCode(),
            'body' => $response->getBody(),
            'headers' => $response->getHeaders(),
            'expires' => new UTCDateTime(time() + $timeToLive),
        ];

        $this->collection->updateOne(['_id' => $id], ['$set' => $cache], ['upsert' => true]);

The method updateOne() does not exist on MongoCollection. Did you maybe mean update()?

    }

    /**
     * Retrieve the cached results of the api $request.
     *
     * @param RequestInterface $request A request for which the response may be cached.
     *
     * @return ResponseInterface|null
     */
    public function get(RequestInterface $request)
    {
        $cached = $this->collection->findOne(['_id' => $request->getUrl()]);
        if ($cached === null) {
            return null;
        }

        $headers = $cached['headers'];
        if ($headers instanceof BSONArray) {
            $headers = $headers->getArrayCopy();
        }

        $body = $cached['body'];
        if ($body instanceof BSONArray) {
            $body = $body->getArrayCopy();
        }

        return new Response($cached['httpCode'], $headers, $body);
    }
}