VGirol\JsonApiStructure\Concern\ValidateMetaObject

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	validateMetaObject()	0	6	2

<?php

declare(strict_types=1);

namespace VGirol\JsonApiStructure\Concern;

use VGirol\JsonApiStructure\Messages;

/**
 * Validations relating to the meta object
 */
trait ValidateMetaObject
{
    /**
     * Assert that a json fragment is a valid meta object.
     *
     * It will do the following checks :
     * 1) validates that the meta object is not an array of objects (@see mustNotBeArrayOfObjects).
     * 2) validates that each member of the meta object is valid (@see validateMemberName).
     *
     * @param array   $json
     * @param boolean $strict If true, unsafe characters are not allowed when checking members name.
     *
     * @return void
     * @throws \VGirol\JsonApiStructure\Exception\ValidationException
     */
    public function validateMetaObject($json, bool $strict): void
    {
        $this->mustNotBeArrayOfObjects($json, Messages::META_OBJECT_MUST_BE_ARRAY, 403);

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

        foreach (array_keys($json) as $key) {
            $this->validateMemberName($key, $strict);

The method validateMemberName() does not exist on VGirol\JsonApiStructure\Concern\ValidateMetaObject. Did you maybe mean validateMetaObject()?

        }
    }
}