diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[PropertyInfo] Fix an error in PropertyInfoCacheExtractor

Browse Source
This commit is contained in:
Guilhem N 2016-07-26 16:23:09 +02:00 committed by Fabien Potencier
parent b1fb82d6db
commit d19b1515a3
3 changed files with 50 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Symfony/Component/PropertyInfo/PropertyInfoCacheExtractor.php
View File

@ -86,7 +86,7 @@ class PropertyInfoCacheExtractor implements PropertyInfoExtractorInterface
*/
public function getTypes($class, $property, array $context = array())
{
return $this->extract('getTypes', array($class, $context));
return $this->extract('getTypes', array($class, $property, $context));
}
/**
@ -108,7 +108,7 @@ class PropertyInfoCacheExtractor implements PropertyInfoExtractorInterface
$key = $this->escape($method.'.'.$serializedArguments);
if (isset($this->arrayCache[$key])) {
if (array_key_exists($key, $this->arrayCache)) {
return $this->arrayCache[$key];
}

18
src/Symfony/Component/PropertyInfo/Tests/Fixtures/NullExtractor.php
View File

@ -28,6 +28,8 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function getShortDescription($class, $property, array $context = array())
{
$this->assertIsString($class);
$this->assertIsString($property);
}
/**
@ -35,6 +37,8 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function getLongDescription($class, $property, array $context = array())
{
$this->assertIsString($class);
$this->assertIsString($property);
}
/**
@ -42,6 +46,8 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function getTypes($class, $property, array $context = array())
{
$this->assertIsString($class);
$this->assertIsString($property);
}
/**
@ -49,6 +55,8 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function isReadable($class, $property, array $context = array())
{
$this->assertIsString($class);
$this->assertIsString($property);
}
/**
@ -56,6 +64,8 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function isWritable($class, $property, array $context = array())
{
$this->assertIsString($class);
$this->assertIsString($property);
}
/**
@ -63,5 +73,13 @@ class NullExtractor implements PropertyListExtractorInterface, PropertyDescripti
*/
public function getProperties($class, array $context = array())
{
$this->assertIsString($class);
}
private function assertIsString($string)
{
if (!is_string($string)) {
throw new \InvalidArgumentException(sprintf('"%s" expects strings, given "%s".', __CLASS__, gettype($string)));
}
}
}

35
src/Symfony/Component/PropertyInfo/Tests/PropertyInfoCacheExtractorTest.php
View File

@ -26,15 +26,40 @@ class PropertyInfoCacheExtractorTest extends AbstractPropertyInfoExtractorTest
$this->propertyInfo = new PropertyInfoCacheExtractor($this->propertyInfo, new ArrayAdapter());
}
public function testCache()
public function testGetShortDescription()
{
$this->assertSame('short', $this->propertyInfo->getShortDescription('Foo', 'bar', array()));
$this->assertSame('short', $this->propertyInfo->getShortDescription('Foo', 'bar', array()));
parent::testGetShortDescription();
parent::testGetShortDescription();
}
public function testNotSerializableContext()
public function testGetLongDescription()
{
$this->assertSame('short', $this->propertyInfo->getShortDescription('Foo', 'bar', array('foo' => function () {})));
parent::testGetLongDescription();
parent::testGetLongDescription();
}
public function testGetTypes()
{
parent::testGetTypes();
parent::testGetTypes();
}
public function testIsReadable()
{
parent::testIsReadable();
parent::testIsReadable();
}
public function testIsWritable()
{
parent::testIsWritable();
parent::testIsWritable();
}
public function testGetProperties()
{
parent::testGetProperties();
parent::testGetProperties();
}
/**