Helix\Shopify\Base\AbstractEntity\MetafieldTrait::getMetafields()

Complexity

Conditions	1
Paths	1

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getMetafields()
    {
        assert($this->hasId());

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

        // the endpoints are all over the place.
        // querying the main directory works for all entities.
        $remote = $this->api->get('metafields', [
            'metafield[owner_id]' => $this->getId(),

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

            'metafield[owner_resource]' => $this->_metafieldType()
        ]);
        /** @var AbstractEntity $that */
        $that = $this;
        return $this->api->factoryAll($that, Metafield::class, $remote['metafields']);
    }
}