for testing and deploying your application
for finding and fixing issues
for empowering human code reviews
<?php
declare(strict_types=1);
/*
* This file is part of the Sonata Project package.
*
* (c) Thomas Rabaix <[email protected]>
* For the full copyright and license information, please view the LICENSE
* file that was distributed with this source code.
*/
namespace Sonata\NotificationBundle\Tests\Entity;
use PHPUnit\Framework\TestCase;
use Sonata\NotificationBundle\Entity\BaseMessage;
use Sonata\NotificationBundle\Model\MessageInterface;
class Message extends BaseMessage
{
public function setId($id): void
$this->id = $id;
}
class BaseMessageTest extends TestCase
public function testClone(): void
$originalMessage = new Message();
$originalMessage->setId(42);
$originalMessage->setBody('body');
'body'
string
array
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example:
function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x);
$originalMessage->setState(MessageInterface::STATE_ERROR);
$clonedMessage = clone $originalMessage;
$this->assertSame('body', $clonedMessage->getBody());
$this->assertSame(MessageInterface::STATE_ERROR, $clonedMessage->getState());
$this->assertNull($clonedMessage->getId());
It seems like the type of the argument is not accepted by the function/method which you are calling.
In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.
We suggest to add an explicit type cast like in the following example: