Graze\UnicontrollerClient\Serialiser\Serialiser\SerialiserVarUserId

Complexity

Total Complexity

1

Size/Duplication

Total Lines	12
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	serialise()	0	6	1

<?php
/**
 * This file is part of graze/unicontroller-client.
 *
 * Copyright (c) 2016 Nature Delivered Ltd. <https://www.graze.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @license https://github.com/graze/unicontroller-client/blob/master/LICENSE.md
 * @link https://github.com/graze/unicontroller-client
 */
namespace Graze\UnicontrollerClient\Serialiser\Serialiser;

use Graze\UnicontrollerClient\Serialiser\Serialiser\AbstractSerialiser;
use Graze\UnicontrollerClient\Serialiser\Serialiser\SerialiserInterface;
use Graze\UnicontrollerClient\Entity\Entity\EntityInterface;

class SerialiserVarUserId extends AbstractSerialiser implements SerialiserInterface
{
    /**
     * @param EntityInterface $entity
     * @return string
     */
    public function serialise(EntityInterface $entity)
    {
        $properties = [];
        $properties[] = $this->stringEscaper->escape($entity->getName());

The method getName() does not exist on Graze\UnicontrollerClient\Entity\Entity\EntityInterface. It seems like you code against a sub-type of said class. However, the method does not exist in Graze\UnicontrollerClient\Entity\Entity\EntityTtfItem or Graze\UnicontrollerClient\Entity\Entity\EntitySettingsById or Graze\UnicontrollerClient\Entity\Entity\EntityBarcodeItem or Graze\UnicontrollerClient\Entity\Entity\EntityBoxItem or Graze\UnicontrollerClient\Entity\Entity\EntityPictureItem or Graze\UnicontrollerClient\Entity\Entity\EntityLineItem or Graze\UnicontrollerClient\Entity\Entity\EntityShiftDefinition or Graze\UnicontrollerClient\Entity\Entity\EntityGeneric. Are you sure you never get one of those?

        return implode(',', $properties);
    }
}