Nip\Http\Request\Traits\HasJsonParameterBag

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	json()	0	11	3

<?php

namespace Nip\Http\Request\Traits;

use Symfony\Component\HttpFoundation\ParameterBag;

/**
 * Trait HasJsonParameterBag
 * @package Nip\Http\Request\Traits
 */
trait HasJsonParameterBag
{

    /**
     * The decoded JSON content for the request.
     *
     * @var \Symfony\Component\HttpFoundation\ParameterBag|null
     */
    protected $json;

    /**
     * Get the JSON payload for the request.
     *
     * @param string|null $key
     * @param mixed $default
     * @return \Symfony\Component\HttpFoundation\ParameterBag|mixed
     */
    public function json($key = null, $default = null)
    {
        if (!isset($this->json)) {
            $this->json = new ParameterBag((array)json_decode($this->getContent(), true));

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

        }

        if (is_null($key)) {
            return $this->json;
        }

        return data_get($this->json->all(), $key, $default);

The method all() does not exist on null.

    }
}