Runalyze\BSON\LazyFilesystemObject\LazyFilesystemObjectStorage

Complexity

Total Complexity

5

Size/Duplication

Total Lines	59
Duplicated Lines	0 %

Test Coverage

Coverage

80.95%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	storeObject()	0	5	1
A	getObject()	0	20	3

<?php

/*
 * This file is part of the Runalyze BSON.
 * (c) RUNALYZE <mail@runalyze.com>
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Runalyze\BSON\LazyFilesystemObject;

use League\Flysystem\Filesystem;
use ProxyManager\Configuration;
use ProxyManager\Factory\LazyLoadingValueHolderFactory;
use Runalyze\BSON\BsonSerializable\BsonSerializableInterface;

class LazyFilesystemObjectStorage
{
    /** @var Filesystem */
    protected $Filesystem;

    /** @var HashToPathMapperInterface */
    protected $PathMapper;

    /** @var LazyLoadingValueHolderFactory */
    protected $ProxyFactory;

    public function __construct(
        Filesystem $filesystem,
        HashToPathMapperInterface $pathMapper,
        Configuration $proxyConfiguration = null
    ) {
        $this->Filesystem = $filesystem;
        $this->PathMapper = $pathMapper;
        $this->ProxyFactory = new LazyLoadingValueHolderFactory($proxyConfiguration);
    }

    /**
     * @param  string                    $hash
     * @param  string                    $class
     * @return BsonSerializableInterface
     */
    public function getObject($hash, $class)
    {
        $path = $this->PathMapper->hashToPath($hash);

        return $this->ProxyFactory->createProxy(

The expression return $this->ProxyFactory->createProxy($class, function(...) { /* ... */ }) returns the type ProxyManager\Proxy\VirtualProxyInterface which is incompatible with the documented return type Runalyze\BSON\BsonSerializable\BsonSerializableInterface.

            $class,
            function (&$wrappedObject, $proxy, $method, $parameters, &$initializer) use ($path, $class) {
                try {
                    $contents = $this->Filesystem->read($path);
                } catch (\Exception $e) {
                    throw $e; // TODO
                }

                try {
                    $wrappedObject = \MongoDB\BSON\toPHP($contents, ['root' => $class]);

It seems like $contents can also be of type false; however, parameter $bson of MongoDB\BSON\toPHP() does only seem to accept string, maybe add an additional type check?

                } catch (\Exception $e) {
                    throw $e; // TODO
                }

                $initializer = null;
            }
        );
    }

    /**
     * @param string                    $hash
     * @param BsonSerializableInterface $object
     */
    public function storeObject($hash, BsonSerializableInterface $object)
    {
        $path = $this->PathMapper->hashToPath($hash);

        $this->Filesystem->put($path, $object->toBinary());
    }
}