MailMotor\Bundle\MailMotorBundle\MailMotor

Complexity

Total Complexity

3

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getListId()	0	3	2
A	__construct()	0	3	1

<?php

namespace MailMotor\Bundle\MailMotorBundle;

/**
 * MailMotor
 *
 * @author Jeroen Desloovere <info@jeroendesloovere.be>
 */
class MailMotor
{
    /** @var string|null - The default list id */
    protected $listId;

    public function __construct(?string $listId)
    {
        $this->listId = $listId;
    }

    /**
     * Get list id
     *
     * @param string|null $listId - If you want to use a custom list id
     * @return string|null
     */
    public function getListId(string $listId = null): ?string
    {
        return ($listId == null) ? $this->listId : $listId;

It seems like you are loosely comparing $listId of type null|string against null; this is ambiguous if the string can be empty. Consider using a strict comparison === instead.

    }
}