Merge branch '4.4'
* 4.4: cs fix Partially Revert "Remove trailing space in all markdown files" [Intl] Explicit check Fix getMaxFilesize() returning zero Remove unused import [DI] fix docblocks in Container* Fixed markdown file [HttpFoundation] Fix deprecation message in ::isMethodSafe() [TwigBridge] Replaced plain doc block copies with inheritdoc. [DomCrawler] Fixed CHANGELOG markup [Intl] Cleanup unused language aliases entry
This commit is contained in:
commit
203060a9a3
@ -760,7 +760,7 @@ Symfony is the result of the work of many people who made the code better
|
||||
- Thomas Ploch
|
||||
- Benjamin Grandfond (benjamin)
|
||||
- Tiago Brito (blackmx)
|
||||
-
|
||||
-
|
||||
- Richard van den Brand (ricbra)
|
||||
- develop
|
||||
- flip111
|
||||
|
@ -33,9 +33,7 @@ class RoutingExtension extends AbstractExtension
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns a list of functions to add to the existing list.
|
||||
*
|
||||
* @return array An array of functions
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function getFunctions()
|
||||
{
|
||||
|
@ -24,9 +24,7 @@ use Twig\TokenParser\AbstractTokenParser;
|
||||
class TransDefaultDomainTokenParser extends AbstractTokenParser
|
||||
{
|
||||
/**
|
||||
* Parses a token and returns a node.
|
||||
*
|
||||
* @return Node
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function parse(Token $token)
|
||||
{
|
||||
@ -38,9 +36,7 @@ class TransDefaultDomainTokenParser extends AbstractTokenParser
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets the tag name associated with this token parser.
|
||||
*
|
||||
* @return string The tag name
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function getTag()
|
||||
{
|
||||
|
@ -28,11 +28,7 @@ use Twig\TokenParser\AbstractTokenParser;
|
||||
class TransTokenParser extends AbstractTokenParser
|
||||
{
|
||||
/**
|
||||
* Parses a token and returns a node.
|
||||
*
|
||||
* @return Node
|
||||
*
|
||||
* @throws SyntaxError
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function parse(Token $token)
|
||||
{
|
||||
@ -90,9 +86,7 @@ class TransTokenParser extends AbstractTokenParser
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets the tag name associated with this token parser.
|
||||
*
|
||||
* @return string The tag name
|
||||
* {@inheritdoc}
|
||||
*/
|
||||
public function getTag()
|
||||
{
|
||||
|
@ -1016,7 +1016,7 @@ class ContainerBuilder extends Container implements TaggedContainerInterface
|
||||
/**
|
||||
* Creates a service for a service definition.
|
||||
*
|
||||
* @return object The service described by the service definition
|
||||
* @return mixed The service described by the service definition
|
||||
*
|
||||
* @throws RuntimeException When the factory definition is incomplete
|
||||
* @throws RuntimeException When the service is a synthetic service
|
||||
@ -1574,7 +1574,7 @@ class ContainerBuilder extends Container implements TaggedContainerInterface
|
||||
/**
|
||||
* Shares a given service in the container.
|
||||
*
|
||||
* @param object $service
|
||||
* @param mixed $service
|
||||
*/
|
||||
private function shareService(Definition $definition, $service, ?string $id, array &$inlineServices)
|
||||
{
|
||||
|
@ -41,7 +41,7 @@ interface ContainerInterface extends PsrContainerInterface
|
||||
* @param string $id The service identifier
|
||||
* @param int $invalidBehavior The behavior when the service does not exist
|
||||
*
|
||||
* @return object The associated service
|
||||
* @return object|null The associated service
|
||||
*
|
||||
* @throws ServiceCircularReferenceException When a circular reference is detected
|
||||
* @throws ServiceNotFoundException When the service is not defined
|
||||
|
@ -15,7 +15,7 @@ CHANGELOG
|
||||
4.3.0
|
||||
-----
|
||||
|
||||
* Added PHPUnit constraints: `CrawlerSelectorAttributeValueSame`, `CrawlerSelectorExists`, `CrawlerSelectorTextContains``
|
||||
* Added PHPUnit constraints: `CrawlerSelectorAttributeValueSame`, `CrawlerSelectorExists`, `CrawlerSelectorTextContains`
|
||||
and `CrawlerSelectorTextSame`
|
||||
* Added return of element name (`_name`) in `extract()` method.
|
||||
* Added ability to return a default value in `text()` and `html()` instead of throwing an exception when node is empty.
|
||||
@ -27,7 +27,7 @@ CHANGELOG
|
||||
|
||||
* The `$currentUri` constructor argument of the `AbstractUriElement`, `Link` and
|
||||
`Image` classes is now optional.
|
||||
* The `Crawler::children()` method will have a new `$selector` argument in version 5.0,
|
||||
* The `Crawler::children()` method will have a new `$selector` argument in version 5.0,
|
||||
not defining it is deprecated.
|
||||
|
||||
3.1.0
|
||||
|
@ -44,9 +44,8 @@ abstract class IteratorTestCase extends TestCase
|
||||
* $a and $b such that $a goes before $b in $expected, the method
|
||||
* asserts that any element of $a goes before any element of $b
|
||||
* in the sequence generated by $iterator
|
||||
* @param \Traversable $iterator
|
||||
*/
|
||||
protected function assertOrderedIteratorForGroups($expected, \Traversable $iterator)
|
||||
protected function assertOrderedIteratorForGroups(array $expected, \Traversable $iterator)
|
||||
{
|
||||
$values = array_values(array_map(function (\SplFileInfo $fileinfo) { return $fileinfo->getPathname(); }, iterator_to_array($iterator)));
|
||||
|
||||
@ -63,11 +62,8 @@ abstract class IteratorTestCase extends TestCase
|
||||
|
||||
/**
|
||||
* Same as IteratorTestCase::assertIterator with foreach usage.
|
||||
*
|
||||
* @param array $expected
|
||||
* @param \Traversable $iterator
|
||||
*/
|
||||
protected function assertIteratorInForeach($expected, \Traversable $iterator)
|
||||
protected function assertIteratorInForeach(array $expected, \Traversable $iterator)
|
||||
{
|
||||
$values = [];
|
||||
foreach ($iterator as $file) {
|
||||
@ -83,11 +79,8 @@ abstract class IteratorTestCase extends TestCase
|
||||
|
||||
/**
|
||||
* Same as IteratorTestCase::assertOrderedIterator with foreach usage.
|
||||
*
|
||||
* @param array $expected
|
||||
* @param \Traversable $iterator
|
||||
*/
|
||||
protected function assertOrderedIteratorInForeach($expected, \Traversable $iterator)
|
||||
protected function assertOrderedIteratorInForeach(array $expected, \Traversable $iterator)
|
||||
{
|
||||
$values = [];
|
||||
foreach ($iterator as $file) {
|
||||
|
@ -219,7 +219,7 @@ class UploadedFile extends File
|
||||
$sizePostMax = self::parseFilesize(ini_get('post_max_size'));
|
||||
$sizeUploadMax = self::parseFilesize(ini_get('upload_max_filesize'));
|
||||
|
||||
return min([$sizePostMax, $sizeUploadMax]);
|
||||
return min($sizePostMax ?: PHP_INT_MAX, $sizeUploadMax ?: PHP_INT_MAX);
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -319,4 +319,18 @@ class UploadedFileTest extends TestCase
|
||||
|
||||
$this->assertFalse($file->isValid());
|
||||
}
|
||||
|
||||
public function testGetMaxFilesize()
|
||||
{
|
||||
$size = UploadedFile::getMaxFilesize();
|
||||
|
||||
$this->assertIsInt($size);
|
||||
$this->assertGreaterThan(0, $size);
|
||||
|
||||
if (0 === (int) ini_get('post_max_size') && 0 === (int) ini_get('upload_max_filesize')) {
|
||||
$this->assertSame(PHP_INT_MAX, $size);
|
||||
} else {
|
||||
$this->assertLessThan(PHP_INT_MAX, $size);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -169,7 +169,6 @@ class LanguageDataGenerator extends AbstractDataGenerator
|
||||
return [
|
||||
'Version' => $rootBundle['Version'],
|
||||
'Languages' => $this->languageCodes,
|
||||
'Aliases' => array_column(iterator_to_array($metadataBundle['alias']['language']), 'replacement'),
|
||||
'Alpha2ToAlpha3' => $alpha2ToAlpha3,
|
||||
'Alpha3ToAlpha2' => $alpha3ToAlpha2,
|
||||
];
|
||||
@ -185,9 +184,9 @@ class LanguageDataGenerator extends AbstractDataGenerator
|
||||
$aliases = iterator_to_array($metadataBundle['alias']['language']);
|
||||
$alpha2ToAlpha3 = [];
|
||||
|
||||
foreach ($aliases as $alias => $language) {
|
||||
$language = $language['replacement'];
|
||||
if (2 === \strlen($language) && 3 === \strlen($alias)) {
|
||||
foreach ($aliases as $alias => $data) {
|
||||
$language = $data['replacement'];
|
||||
if (2 === \strlen($language) && 3 === \strlen($alias) && 'overlong' === $data['reason']) {
|
||||
if (isset(self::$preferredAlpha2ToAlpha3Mapping[$language])) {
|
||||
// Validate to prevent typos
|
||||
if (!isset($aliases[self::$preferredAlpha2ToAlpha3Mapping[$language]])) {
|
||||
|
@ -618,7 +618,6 @@
|
||||
"zun",
|
||||
"zza"
|
||||
],
|
||||
"Aliases": [],
|
||||
"Alpha2ToAlpha3": {
|
||||
"aa": "aar",
|
||||
"ab": "abk",
|
||||
|
@ -6,7 +6,7 @@ A Ldap client for PHP on top of PHP's ldap extension.
|
||||
Disclaimer
|
||||
----------
|
||||
|
||||
This component is only stable since Symfony 3.1. Earlier versions
|
||||
This component is only stable since Symfony 3.1. Earlier versions
|
||||
have been marked as internal as they still needed some work.
|
||||
Breaking changes were introduced in Symfony 3.1, so code relying on
|
||||
previous version of the component will break with this version.
|
||||
|
@ -155,7 +155,7 @@ CHANGELOG
|
||||
|
||||
* added `$context` support for XMLEncoder.
|
||||
* [DEPRECATION] JsonEncode and JsonDecode where modified to throw
|
||||
an exception if error found. No need for get*Error() functions
|
||||
an exception if error found. No need for `get*Error()` functions
|
||||
|
||||
2.3.0
|
||||
-----
|
||||
|
Reference in New Issue
Block a user