danog/psalm
1
0
Fork 0
mirror of https://github.com/danog/psalm.git synced 2024-11-27 04:45:20 +01:00
Code Issues Projects Releases Wiki Activity

Fix #1518 - add escaping of object-like keys

Browse Source
This commit is contained in:
Brown 2020-04-26 17:36:02 -04:00
parent ebcb0b8cc4
commit 5e2af1d7f6
4 changed files with 40 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Psalm/Type.php
View File

@ -648,6 +648,10 @@ abstract class Type
);
}
if ($property_key[0] === '\'' || $property_key[0] === '"') {
$property_key = \stripslashes(substr($property_key, 1, -1));
}
if (!$property_type instanceof Union) {
$property_type = new Union([$property_type]);
}

30
src/Psalm/Type/Atomic/ObjectLike.php
View File

@ -75,13 +75,11 @@ class ObjectLike extends \Psalm\Type\Atomic
public function __toString()
{
$union_type_parts = array_map(
/**
* @param string|int $name
* @param Union $type
*
* @return string
*/
function ($name, Union $type) {
if (\is_string($name) && \preg_match('/[ "\'\\\\.\n]/', $name)) {
$name = '\'' . \str_replace("\n", '\n', \addslashes($name)) . '\'';
}
return $name . ($type->possibly_undefined ? '?' : '') . ': ' . $type;
},
array_keys($this->properties),
@ -95,13 +93,11 @@ class ObjectLike extends \Psalm\Type\Atomic
public function getId(bool $nested = false)
{
$union_type_parts = array_map(
/**
* @param string|int $name
* @param Union $type
*
* @return string
*/
function ($name, Union $type) {
if (\is_string($name) && \preg_match('/[ "\'\\\\.\n]/', $name)) {
$name = '\'' . \str_replace("\n", '\n', \addslashes($name)) . '\'';
}
return $name . ($type->possibly_undefined ? '?' : '') . ': ' . $type->getId();
},
array_keys($this->properties),
@ -143,12 +139,6 @@ class ObjectLike extends \Psalm\Type\Atomic
implode(
', ',
array_map(
/**
* @param string|int $name
* @param Union $type
*
* @return string
*/
function (
$name,
Union $type
@ -158,6 +148,10 @@ class ObjectLike extends \Psalm\Type\Atomic
$this_class,
$use_phpdoc_format
) {
if (\is_string($name) && \preg_match('/[ "\'\\\\.\n]/', $name)) {
$name = '\'' . \str_replace("\n", '\n', \addslashes($name)) . '\'';
}
return $name . ($type->possibly_undefined ? '?' : '') . ': ' . $type->toNamespacedString(
$namespace,
$aliased_classes,

7
tests/AnnotationTest.php
View File

@ -1129,6 +1129,13 @@ class AnnotationTest extends TestCase
[],
'7.4'
],
'arrayWithKeySlashesAndNewline' => [
'<?php
$arr = ["foo\\bar\nbaz" => "literal"];',
[
'$arr' => 'array{\'foo\\\\bar\nbaz\': string}'
]
],
];
}

17
tests/TypeParseTest.php
View File

@ -398,6 +398,23 @@ class TypeParseTest extends TestCase
$this->assertSame('array{a: int, b: string}', (string) Type:: parseString('array{a: int, b: string}'));
}
/**
* @return void
*/
public function testObjectLikeWithSpace()
{
$this->assertSame('array{\'a \': int, \'b \': string}', (string) Type:: parseString('array{\'a \': int, \'b \': string}'));
}
/**
* @return void
*/
public function testObjectLikeWithQuotedKeys()
{
$this->assertSame('array{\'\\"\': int, \'\\\'\': string}', (string) Type:: parseString('array{\'"\': int, \'\\\'\': string}'));
$this->assertSame('array{\'\\"\': int, \'\\\'\': string}', (string) Type:: parseString('array{"\\"": int, "\\\'": string}'));
}
/**
* @return void
*/