Guillermoandrae\DynamoDb\Operation\GetItemOperation::execute()

Complexity

Conditions	3
Paths	5

Size

Total Lines	11
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	3

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function execute(): array
    {
        try {
            $item = [];
            $results = $this->client->getItem($this->toArray());
            if (is_array($results['Item'])) {
                $item = $this->getMarshaler()->unmarshalItem($results['Item'], false);

It seems like $results['Item'] can also be of type null; however, parameter $data of Aws\DynamoDb\Marshaler::unmarshalItem() does only seem to accept array, maybe add an additional type check?

            }
            return $item;

The expression return $item could return the type stdClass which is incompatible with the type-hinted return array. Consider adding an additional type-check to rule them out.

        } catch (DynamoDbException $ex) {
            throw ExceptionFactory::factory($ex);
        }