PhpWinTools\WmiScripting\Support\ApiObjects\SWbemObject

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

94.12%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	1
A	instantiateWin32Model()	0	9	3
A	buildDerivations()	0	3	1
A	getModel()	0	3	1

<?php

namespace PhpWinTools\WmiScripting\Support\ApiObjects;

use PhpWinTools\Support\COM\VariantWrapper;
use PhpWinTools\WmiScripting\Models\Classes;
use PhpWinTools\WmiScripting\Models\Win32Model;
use function PhpWinTools\WmiScripting\Support\resolve;
use PhpWinTools\WmiScripting\Support\ApiObjects\Contracts\ObjectItem;
use PhpWinTools\WmiScripting\Support\ApiObjects\VariantInterfaces\ObjectVariant;

/**
 * @link https://docs.microsoft.com/en-us/windows/win32/wmisdk/swbemobject
 */
class SWbemObject extends AbstractWbemObject implements ObjectItem
{
    protected $win32Model;

    protected $derivations = [];

    protected $propertySet;

    protected $qualifierSet;

    protected $path;

    /** @var VariantWrapper|ObjectVariant */
    protected $object;

    public function __construct(VariantWrapper $variant, array $resolve_property_sets = [])
    {
        parent::__construct($variant);

        $this->buildDerivations();

        $this->propertySet = resolve()->propertySet($this->object->Properties_, $resolve_property_sets);

The property Properties_ does not exist on PhpWinTools\Support\COM\VariantWrapper. Since you implemented __get, consider adding a @property annotation.

It seems like $this->object->Properties_ can also be of type PhpWinTools\Support\COM\ComWrapper; however, parameter $variant of PhpWinTools\WmiScripting\Configuration\Resolver::propertySet() does only seem to accept PhpWinTools\Support\COM\VariantWrapper, maybe add an additional type check?

        $this->qualifierSet = resolve()->qualifierSet($this->object->Qualifiers_);

It seems like $this->object->Qualifiers_ can also be of type PhpWinTools\Support\COM\ComWrapper; however, parameter $variant of PhpWinTools\WmiScripting\Configuration\Resolver::qualifierSet() does only seem to accept PhpWinTools\Support\COM\VariantWrapper, maybe add an additional type check?

The property Qualifiers_ does not exist on PhpWinTools\Support\COM\VariantWrapper. Since you implemented __get, consider adding a @property annotation.

        $this->path = resolve()->objectPath($this->object->Path_);

The property Path_ does not exist on PhpWinTools\Support\COM\VariantWrapper. Since you implemented __get, consider adding a @property annotation.

It seems like $this->object->Path_ can also be of type PhpWinTools\Support\COM\ComWrapper; however, parameter $variant of PhpWinTools\WmiScripting\Configuration\Resolver::objectPath() does only seem to accept PhpWinTools\Support\COM\VariantWrapper, maybe add an additional type check?

        $this->instantiateWin32Model();
    }

    /**
     * @return Win32Model
     */
    public function getModel(): Win32Model
    {
        return $this->getAttribute('win32Model');
    }

    public function instantiateWin32Model(Win32Model $model = null): Win32Model
    {
        $model = $model ?? Classes::getClass($this->path->getAttribute('class'));

        if (!is_null($this->win32Model) && get_class($this->win32Model) === $model) {

It seems like $this->win32Model can also be of type mixed; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

            return $this->win32Model;

The expression return $this->win32Model could return the type mixed which is incompatible with the type-hinted return PhpWinTools\WmiScripting\Models\Win32Model. Consider adding an additional type-check to rule them out.

        }

        return $this->win32Model = new $model($this->propertySet->toArray(), $this->path);
    }

    protected function buildDerivations()
    {
        $this->derivations = $this->object->propertyToArray('Derivation_');

The method propertyToArray() does not exist on PhpWinTools\WmiScripting\Support\ApiObjects\VariantInterfaces\ObjectVariant. Since you implemented __call, consider adding a @method annotation.

    }
}