Allow ints in source/dist reference
parent
832af78e28
commit
832f4007d6
|
@ -323,8 +323,8 @@ class ValidatingArrayLoader implements LoaderInterface
|
||||||
if (!is_string($this->config[$srcType]['url'])) {
|
if (!is_string($this->config[$srcType]['url'])) {
|
||||||
$this->errors[] = $srcType . '.url : should be a string, '.gettype($this->config[$srcType]['url']).' given';
|
$this->errors[] = $srcType . '.url : should be a string, '.gettype($this->config[$srcType]['url']).' given';
|
||||||
}
|
}
|
||||||
if (isset($this->config[$srcType]['reference']) && !is_string($this->config[$srcType]['reference'])) {
|
if (isset($this->config[$srcType]['reference']) && !is_string($this->config[$srcType]['reference']) && !is_int($this->config[$srcType]['reference'])) {
|
||||||
$this->errors[] = $srcType . '.reference : should be a string, '.gettype($this->config[$srcType]['reference']).' given';
|
$this->errors[] = $srcType . '.reference : should be a string or int, '.gettype($this->config[$srcType]['reference']).' given';
|
||||||
}
|
}
|
||||||
if (isset($this->config[$srcType]['reference']) && preg_match('{^\s*-}', $this->config[$srcType]['reference'])) {
|
if (isset($this->config[$srcType]['reference']) && preg_match('{^\s*-}', $this->config[$srcType]['reference'])) {
|
||||||
$this->errors[] = $srcType . '.reference : must not start with a "-", "'.$this->config[$srcType]['reference'].'" given';
|
$this->errors[] = $srcType . '.reference : must not start with a "-", "'.$this->config[$srcType]['reference'].'" given';
|
||||||
|
|
|
@ -194,6 +194,13 @@ class ValidatingArrayLoaderTest extends TestCase
|
||||||
'name' => 'npm-asset/angular--core',
|
'name' => 'npm-asset/angular--core',
|
||||||
),
|
),
|
||||||
),
|
),
|
||||||
|
array( // refs as int or string
|
||||||
|
array(
|
||||||
|
'name' => 'foo/bar',
|
||||||
|
'source' => array('url' => 'https://example.org', 'reference' => 1234, 'type' => 'baz'),
|
||||||
|
'dist' => array('url' => 'https://example.org', 'reference' => 'foobar', 'type' => 'baz'),
|
||||||
|
),
|
||||||
|
),
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue