Skip to content

Commit

Permalink
Merge branch '3.4' into 4.4
Browse files Browse the repository at this point in the history
* 3.4:
  Make PHP 8 green on Travis
  • Loading branch information
nicolas-grekas committed May 23, 2020
2 parents 471c0b3 + 4b330f9 commit 843940a
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Normalizer/AbstractObjectNormalizer.php
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ protected function instantiateObject(array &$data, $class, array &$context, \Ref
*
* @return string[]
*/
protected function getAttributes($object, $format = null, array $context)
protected function getAttributes($object, $format, array $context)
{
$class = $this->objectClassResolver ? ($this->objectClassResolver)($object) : \get_class($object);
$key = $class.'-'.$context['cache_key'];
Expand Down
2 changes: 1 addition & 1 deletion Tests/Normalizer/GetSetMethodNormalizerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -663,7 +663,7 @@ class GetConstructorArgsWithDefaultValueDummy
protected $foo;
protected $bar;

public function __construct($foo = [], $bar)
public function __construct($foo = [], $bar = null)
{
$this->foo = $foo;
$this->bar = $bar;
Expand Down
4 changes: 2 additions & 2 deletions Tests/Normalizer/ObjectNormalizerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -941,7 +941,7 @@ class ObjectConstructorArgsWithDefaultValueDummy
protected $foo;
protected $bar;

public function __construct($foo = [], $bar)
public function __construct($foo = [], $bar = null)
{
$this->foo = $foo;
$this->bar = $bar;
Expand Down Expand Up @@ -1091,7 +1091,7 @@ class DummyWithConstructorObjectAndDefaultValue
private $foo;
private $inner;

public function __construct($foo = 'a', ObjectInner $inner)
public function __construct($foo = 'a', ObjectInner $inner = null)
{
$this->foo = $foo;
$this->inner = $inner;
Expand Down

0 comments on commit 843940a

Please sign in to comment.