8000 Merge branch '3.1' into 3.2 · symfony/symfony@02b1e75 · GitHub
[go: up one dir, main page]
Skip to content
8000

Commit 02b1e75

Browse files
fabpotfabpot
committed
Merge branch '3.1' into 3.2
* 3.1: [Filesystem] Remove extra argv in dumpFile() tests [DI] minor FileLoaders tests update [HttpKernel] Revert BC breaking change of Request::isMethodSafe() [DOMCrawler] Bug fixed [FrameworkBundle] Mark cache.default_*_provider services private [Process] Do feat test before enabling TTY mode bumped Symfony version to 3.1.8 updated VERSION for 3.1.7 updated CHANGELOG for 3.1.7 bumped Symfony version to 2.8.15 updated VERSION for 2.8.14 updated CHANGELOG for 2.8.14 bumped Symfony version to 2.7.22 updated VERSION for 2.7.21 update CONTRIBUTORS for 2.7.21 updated CHANGELOG for 2.7.21 Fix annotation type for $context [Doctrine][Form] support large integers
2 parents cabc225 + 08c869e commit 02b1e75

File tree

17 files changed

+87
-32
lines changed

17 files changed

+87
-32
lines changed

‎CHANGELOG-3.1.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,33 @@ in 3.1 minor versions.
77
To get the diff for a specific change, go to https://github.com/symfony/symfony/commit/XXX where XXX is the change hash
88
To get the diff between two versions, go to https://github.com/symfony/symfony/compare/v3.1.0...v3.1.1
99

10+
* 3.1.7 (2016-11-21)
11+
12+
* bug #20550 [YAML] Fix processing timestamp strings with timezone (myesain)
13+
* bug #20543 [DI] Fix error when trying to resolve a DefinitionDecorator (nicolas-grekas)
14+
* bug #20544 [PhpUnitBridge] Fix time-sensitive tests that use data providers (julienfalque)
15+
* bug #20484 bumped min version of Twig to 1.28 (fabpot)
16+
* bug #20519 [Debug] Remove GLOBALS from exception context to avoid endless recursion (Seldaek)
17+
* bug #20455 [ClassLoader] Fix ClassCollectionLoader inlining with __halt_compiler (giosh94mhz)
18+
* bug #20307 [Form] Fix Date\TimeType marked as invalid on request with single_text and zero seconds (LuisDeimos)
19+
* bug #20480 [FrameworkBundle] Register the ArrayDenormalizer (dunglas)
20+
* bug #20286 [Serializer] Fix DataUriNormalizer's regex (dunglas)
21+
* bug #20466 [Translation] fixed nested fallback catalogue using multiple locales. (aitboudad)
22+
* bug #20465 [#18637][TranslationDebug] workaround for getFallbackLocales. (aitboudad)
23+
* bug #20453 [Cache] Make directory hashing case insensitive (nicolas-grekas)
24+
* bug #20440 [TwigBridge][TwigBundle][HttpKernel] prefer getSourceContext() over getSource() (xabbuh)
25+
* bug #20287 Properly format value in UniqueEntityValidator (alcaeus)
26+
* bug #20422 [Translation][fallback] add missing resources in parent catalogues. (aitboudad)
27+
* bug #20378 [Form] Fixed show float values as choice value in ChoiceType (yceruto)
28+
* bug #20294 Improved the design of the metrics in the profiler (javiereguiluz)
29+
* bug #20375 [HttpFoundation][Session] Fix memcache session handler (klandaika)
30+
* bug #20377 [Console] Fix infinite loop on missing input (chalasr)
31+
* bug #20372 [Console] simplified code (fabpot)
32+
* bug #20342 [Form] Fix UrlType transforms valid protocols (ogizanagi)
33+
* bug #20292 Enhance GAE compat by removing some realpath() (nicolas-grekas)
34+
* bug #20326 [VarDumper] Fix dumping Twig source in stack traces (nicolas-grekas)
35+
* bug #20321 Compatibility with Twig 1.27 (xkobal)
36+
1037
* 3.1.6 (2016-10-27)
1138

1239
* bug #20291 [Yaml] Fix 7.1 compat (nicolas-grekas)

‎CONTRIBUTORS.md

Lines changed: 21 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -22,16 +22,16 @@ Symfony is the result of the work of many people who made the code better
2222
- Abdellatif Ait boudad (aitboudad)
2323
- Pascal Borreli (pborreli)
2424
- Wouter De Jong (wouterj)
25-
- Joseph Bielawski (stloyd)
2625
- Romain Neutron (romain)
26+
- Joseph Bielawski (stloyd)
2727
- Karma Dordrak (drak)
2828
- Lukas Kahwe Smith (lsmith)
2929
- Martin Hasoň (hason)
3030
- Jeremy Mikola (jmikola)
3131
- Jean-François Simon (jfsimon)
32+
- Grégoire Pineau (lyrixx)
3233
- Benjamin Eberlei (beberlei)
3334
- Igor Wiedler (igorw)
34-
- Grégoire Pineau (lyrixx)
3535
- Eriksen Costa (eriksencosta)
3636
- Jules Pietri (heah)
3737
- Sarah Khalil (saro0h)
@@ -45,11 +45,11 @@ Symfony is the result of the work of many people who made the code better
4545
- stealth35 ‏ (stealth35)
4646
- Alexander Mols (asm89)
4747
- Bulat Shakirzyanov (avalanche123)
48+
- Robin Chalas (chalas_r)
4849
- Saša Stamenković (umpirsky)
4950
- Henrik Bjørnskov (henrikbjorn)
5051
- Miha Vrhovnik
5152
- Diego Saint Esteben (dii3g0)
52-
- Robin Chalas (chalas_r)
5353
- Ener-Getick (energetick)
5454
- Konstantin Kudryashov (everzet)
5555
- Bilal Amarni (bamarni)
@@ -84,19 +84,19 @@ Symfony is the result of the work of many people who made the code better
8484
- Fran Moreno (franmomu)
8585
- Antoine Hérault (herzult)
8686
- Paráda József (paradajozsef)
87+
- Roland Franssen (ro0)
88+
- Dariusz Ruminski
8789
- Jáchym Toušek (enumag)
8890
- Arnaud Le Blanc (arnaud-lb)
8991
- Jérôme Tamarelle (gromnan)
9092
- Michal Piotrowski (eventhorizon)
9193
- Tim Nagel (merk)
92-
- Dariusz Ruminski
9394
- Brice BERNARD (brikou)
9495
- Alexander M. Turek (derrabus)
9596
- marc.weistroff
9697
- Issei Murasawa (issei_m)
9798
- lenar
9899
- Włodzimierz Gajda (gajdaw)
99-
- Roland Franssen (ro0)
100100
- Baptiste Clavié (talus)
101101
- Alexander Schwenn (xelaris)
102102
- Florian Voutzinos (florianv)
@@ -132,6 +132,7 @@ Symfony is the result of the work of many people who made the code better
132132
- Rafael Dohms (rdohms)
133133
- Arnaud Kleinpeter (nanocom)
134134
- jwdeitch
135+
- Tobias Nyholm (tobias)
135136
- Joel Wurtz (brouznouf)
136137
- Philipp Wahala (hifi)
137138
- Vyacheslav Pavlov
@@ -145,12 +146,14 @@ Symfony is the result of the work of many people who made the code better
145146
- Clemens Tolboom
146147
- Helmer Aaviksoo
147148
- Hiromi Hishida (77web)
149+
- Yonel Ceruto González (yonelceruto)
148150
- Richard van Laak (rvanlaak)
149151
- Matthieu Ouellette-Vachon (maoueh)
150152
- Michał Pipa (michal.pipa)
151153
- Amal Raghav (kertz)
152154
- Jonathan Ingram (jonathaningram)
153155
- Artur Kotyrba
156+
- jeremyFreeAgent (Jérémy Romey) (jeremyfreeagent)
154157
- Warnar Boekkooi (boekkooi)
155158
- Dmitrii Chekaliuk (lazyhammer)
156159
- Clément JOBEILI (dator)
@@ -160,13 +163,12 @@ Symfony is the result of the work of many people who made the code better
160163
- Richard Miller (mr_r_miller)
161164
- Mario A. Alvarez Garcia (nomack84)
162165
- Dennis Benkert (denderello)
163-
- jeremyFreeAgent (Jérémy Romey) (jeremyfreeagent)
164166
- Benjamin Dulau (dbenjamin)
165167
- Mathieu Lemoine (lemoinem)
166168
- Andreas Hucks (meandmymonkey)
167169
- Noel Guilbert (noel)
168170
- Lars Strojny (lstrojny)
169-
- Yonel Ceruto González (yonelceruto)
171+
- Maxime STEINHAUSSER
170172
- Stepan Anchugov (kix)
171173
- bronze1man
172174
- sun (sun)
@@ -236,7 +238,6 @@ Symfony is the result of the work of many people who made the code better
236238
- Grégoire Paris (greg0ire)
237239
- Leo Feyer
238240
- Chekote
239-
- Tobias Nyholm (tobias)
240241
- Thomas Adam
241242
- Albert Casademont (acasademont)
242243
- Jhonny Lidfors (jhonne)
@@ -245,6 +246,7 @@ Symfony is the result of the work of many people who made the code better
245246
- Nikita Konstantinov
246247
- Wodor Wodorski
247248
- Thomas Lallement (raziel057)
249+
- Giorgio Premi
248250
- Matthieu Napoli (mnapoli)
249251
- Beau Simensen (simensen)
250252
- Michael Hirschler (mvhirsch)
@@ -295,7 +297,6 @@ Symfony is the result of the work of many people who made the code better
295297
- Mark Challoner (markchalloner)
296298
- Gregor Harlan (gharlan)
297299
- Gennady Telegin (gtelegin)
298-
- Giorgio Premi
299300
- Ben Davies (bendavies)
300301
- Erin Millard
301302
- Artur Melo (restless)
@@ -393,7 +394,6 @@ Symfony is the result of the work of many people who made the code better
393394
- Christian Schmidt
394395
- Marcin Sikoń (marphi)
395396
- Dominik Zogg (dominik.zogg)
396-
- Maxime STEINHAUSSER
397397
- Marek Pietrzak
398398
- Chad Sikorra (chadsikorra)
399399
- franek (franek)
@@ -439,6 +439,7 @@ Symfony is the result of the work of many people who made the code better
439439
- Vicent Soria Durá (vicentgodella)
440440
- Nicolas Dewez (nicolas_dewez)
441441
- Anthony Ferrara
442+
- Victor Bocharsky (bocharsky_bw)
442443
- Ioan Negulescu
443444
- Jakub Škvára (jskvara)
444445
- Andrew Udvare (audvare)
@@ -611,7 +612,6 @@ Symfony is the result of the work of many people who made the code better
611612
- Andrew Tchircoff (andrewtch)
612613
- michaelwilliams
613614
- 1emming
614-
- Victor Bocharsky (bocharsky_bw)
615615
- Leevi Graham (leevigraham)
616616
- Jordan Deitch
617617
- Casper Valdemar Poulsen
@@ -769,6 +769,7 @@ Symfony is the result of the work of many people who made the code better
769769
- fabios
770770
- Sander Coolen (scoolen)
771771
- Nicolas Le Goff (nlegoff)
772+
- Andreas Braun
772773
- Ben Oman
773774
- Manuele Menozzi
774775
- Anton Babenko (antonbabenko)
@@ -858,6 +859,7 @@ Symfony is the result of the work of many people who made the code better
858859
- rpg600
859860
- Péter Buri (burci)
860861
- Davide Borsatto (davide.borsatto)
862+
- Indra Gunawan (guind)
861863
- kaiwa
862864
- Charles Sanquer (csanquer)
863865
- Albert Ganiev (helios-ag)
@@ -969,6 +971,7 @@ Symfony is the result of the work of many people who made the code better
969971
- Sandro Hopf
970972
- Łukasz Makuch
971973
- George Giannoulopoulos
974+
- Luis Ramirez (luisdeimos)
972975
- Daniel Richter (richtermeister)
973976
- ChrisC
974977
- Ilya Biryukov
@@ -985,6 +988,7 @@ Symfony is the result of the work of many people who made the code better
985988
- Pete Mitchell (peterjmit)
986989
- Tom Corrigan (tomcorrigan)
987990
- Martin Pärtel
991+
- Noah Heck (myesain)
988992
- Patrick Daley (padrig)
989993
- Xavier Briand (xavierbriand)
990994
- Max Summe
@@ -1041,6 +1045,7 @@ Symfony is the result of the work of many people who made the code better
10411045
- Sebastian Ionescu
10421046
- Thomas Ploch
10431047
- Simon Neidhold
1048+
- Xavier HAUSHERR
10441049
- Valentin VALCIU
10451050
- Kevin Dew
10461051
- James Cowgill
@@ -1163,6 +1168,7 @@ Symfony is the result of the work of many people who made the code better
11631168
- Koalabaerchen
11641169
- michalmarcinkowski
11651170
- Warwick
1171+
- VJ
11661172
- Chris
11671173
- JakeFr
11681174
- Simon Sargeant
@@ -1236,6 +1242,7 @@ Symfony is the result of the work of many people who made the code better
12361242
- Joel Marcey
12371243
- David Christmann
12381244
- root
1245+
- Wouter J
12391246
- James Hudson
12401247
- Tom Maguire
12411248
- David Zuelke
@@ -1312,6 +1319,7 @@ Symfony is the result of the work of many people who made the code better
13121319
- Jelle Bekker (jbekker)
13131320
- Ian Jenkins (jenkoian)
13141321
- Jorge Martin (jorgemartind)
1322+
- Julien Falque (julienfalque)
13151323
- Kevin Herrera (kherge)
13161324
- Luis Ramón López López (lrlopez)
13171325
- Muriel (metalmumu)
@@ -1379,6 +1387,7 @@ Symfony is the result of the work of many people who made the code better
13791387
- Jörg Rühl
13801388
- wesleyh
13811389
- sergey
1390+
- Karim Miladi
13821391
- Michael Genereux
13831392
- patrick-mcdougle
13841393
- Dariusz Czech
@@ -1559,6 +1568,7 @@ Symfony is the result of the work of many people who made the code better
15591568
- Sergey Fedotov
15601569
- Michael
15611570
- fh-github@fholzhauer.de
1571+
- Jan Emrich
15621572
- Mark Topper
15631573
- Xavier REN
15641574
- Zander Baldwin

‎src/Symfony/Bridge/Doctrine/Form/ChoiceList/ORMQueryBuilderLoader.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ public function getEntitiesByIds($identifier, array $values)
7070
// Filter out non-integer values (e.g. ""). If we don't, some
7171
// databases such as PostgreSQL fail.
7272
$values = array_values(array_filter($values, function ($v) {
73-
return (string) $v === (string) (int) $v;
73+
return (string) $v === (string) (int) $v || ctype_digit($v);
7474
}));
7575
} elseif ('guid' === $metadata->getTypeOfField($identifier)) {
7676
$parameterType = Connection::PARAM_STR_ARRAY;

‎src/Symfony/Bridge/Doctrine/Tests/Form/ChoiceList/ORMQueryBuilderLoaderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ public function testFilterNonIntegerValues()
6767

6868
$query->expects($this->once())
6969
->method('setParameter')
70-
->with('ORMQueryBuilderLoader_getEntitiesByIds_id', array(1, 2, 3), Connection::PARAM_INT_ARRAY)
70+
->with('ORMQueryBuilderLoader_getEntitiesByIds_id', array(1, 2, 3, '9223372036854775808'), Connection::PARAM_INT_ARRAY)
7171
->willReturn($query);
7272

7373
$qb = $this->getMockBuilder('Doctrine\ORM\QueryBuilder')
@@ -83,7 +83,7 @@ public function testFilterNonIntegerValues()
8383
->from('Symfony\Bridge\Doctrine\Tests\Fixtures\SingleIntIdEntity', 'e');
8484

8585
$loader = new ORMQueryBuilderLoader($qb);
86-
$loader->getEntitiesByIds('id', array(1, '', 2, 3, 'foo'));
86+
$loader->getEntitiesByIds('id', array(1, '', 2, 3, 'foo', '9223372036854775808'));
8787
}
8888

8989
public function testEmbeddedIdentifierName()

‎src/Symfony/Bundle/FrameworkBundle/DependencyInjection/FrameworkExtension.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1248,6 +1248,7 @@ private function registerCacheConfiguration(array $config, ContainerBuilder $con
12481248
foreach (array('doctrine', 'psr6', 'redis') as $name) {
12491249
if (isset($config[$name = 'default_'.$name.'_provider'])) {
12501250
$container->setAlias('cache.'.$name, Compiler\CachePoolPass::getServiceProvider($container, $config[$name]));
1251+
$container->getAlias('cache.'.$name)->setPublic(false);
12511252
}
12521253
}
12531254
foreach (array('app', 'system') as $name) {

‎src/Symfony/Component/DependencyInjection/Tests/Loader/XmlFileLoaderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,8 +131,8 @@ public function testLoadImports()
131131
{
132132
$container = new ContainerBuilder();
133133
$resolver = new LoaderResolver(array(
134-
new IniFileLoader($container, new FileLocator(self::$fixturesPath.'/xml')),
135-
new YamlFileLoader($container, new FileLocator(self::$fixturesPath.'/xml')),
134+
new IniFileLoader($container, new FileLocator(self::$fixturesPath.'/ini')),
135+
new YamlFileLoader($container, new FileLocator(self::$fixturesPath.'/yml')),
136136
$loader = new XmlFileLoader($container, new FileLocator(self::$fixturesPath.'/xml')),
137137
));
138138
$loader->setResolver($resolver);

‎src/Symfony/Component/DependencyInjection/Tests/Loader/YamlFileLoaderTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,8 +104,8 @@ public function testLoadImports()
104104
{
105105
$container = new ContainerBuilder();
106106
$resolver = new LoaderResolver(array(
107-
new IniFileLoader($container, new FileLocator(self::$fixturesPath.'/yaml')),
108-
new XmlFileLoader($container, new FileLocator(self::$fixturesPath.'/yaml')),
107+
new IniFileLoader($container, new FileLocator(self::$fixturesPath.'/ini')),
108+
new XmlFileLoader($container, new FileLocator(self::$fixturesPath.'/xml')),
109109
new PhpFileLoader($container, new FileLocator(self::$fixturesPath.'/php')),
110110
$loader = new YamlFileLoader($container, new FileLocator(self::$fixturesPath.'/yaml')),
111111
));

‎src/Symfony/Component/DomCrawler/Crawler.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -930,7 +930,7 @@ public static function xpathLiteral($s)
930930
}
931931
}
932932

933-
return sprintf('concat(%s)', implode($parts, ', '));
933+
return sprintf('concat(%s)', implode(', ', $parts));
934934
}
935935

936936
/**

‎src/Symfony/Component/Filesystem/Tests/FilesystemTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1388,7 +1388,7 @@ public function testDumpFileWithFileScheme()
13881388
$scheme = 'file://';
13891389
$filename = $scheme.$this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'baz.txt';
13901390

1391-
$this->filesystem->dumpFile($filename, 'bar', null);
1391+
$this->filesystem->dumpFile($filename, 'bar');
13921392

13931393
$this->assertFileExists($filename);
13941394
$this->assertSame('bar', file_get_contents($filename));
@@ -1399,7 +1399,7 @@ public function testDumpFileWithZlibScheme()
13991399
$scheme = 'compress.zlib://';
14001400
$filename = $this->workspace.DIRECTORY_SEPARATOR.'foo'.DIRECTORY_SEPARATOR.'baz.txt';
14011401

1402-
$this->filesystem->dumpFile($filename, 'bar', null);
1402+
$this->filesystem->dumpFile($filename, 'bar');
14031403

14041404
// Zlib stat uses file:// wrapper so remove scheme
14051405
$this->assertFileExists(str_replace($scheme, '', $filename));

‎src/Symfony/Component/HttpFoundation/BinaryFileResponse.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ public function prepare(Request $request)
190190

191191
if (!$this->headers->has('Accept-Ranges')) {
192192
// Only accept ranges on safe HTTP methods
193-
$this->headers->set('Accept-Ranges', $request->isMethodSafe() ? 'bytes' : 'none');
193+
$this->headers->set('Accept-Ranges', $request->isMethodSafe(false) ? 'bytes' : 'none');
194194
}
195195

196196
if (!$this->headers->has('Content-Type')) {

‎src/Symfony/Component/HttpFoundation/Request.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1475,11 +1475,13 @@ public function isMethod($method)
14751475
/**
14761476
* Checks whether or not the method is safe.
14771477
*
1478+
* @param bool $andCacheable Adds the additional condition that the method should be cacheable. True by default.
1479+
*
14781480
* @return bool
14791481
*/
1480-
public function isMethodSafe()
1482+
public function isMethodSafe(/* $andCacheable = true */)
14811483
{
1482-
return in_array($this->getMethod(), array('GET', 'HEAD', 'OPTIONS', 'TRACE'));
1484+
return in_array($this->getMethod(), 0 < func_num_args() && !func_get_arg(0) ? array('GET', 'HEAD', 'OPTIONS', 'TRACE') : array('GET', 'HEAD'));
14831485
}
14841486

14851487
/**

‎src/Symfony/Component/HttpFoundation/Tests/RequestTest.php

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1994,7 +1994,7 @@ public function testMethodSafe($method, $safe)
19941994
{
19951995
$request = new Request();
19961996
$request->setMethod($method);
1997-
$this->assertEquals($safe, $request->isMethodSafe());
1997+
$this->assertEquals($safe, $request->isMethodSafe(false));
19981998
}
19991999

20002000
public function methodSafeProvider()
@@ -2013,6 +2013,13 @@ public function methodSafeProvider()
20132013
);
20142014
}
20152015

2016+
public function testMethodSafeChecksCacheable()
2017+
{
2018+
$request = new Request();
2019+
$request->setMethod('OPTION');
2020+
$this->assertFalse($request->isMethodSafe());
2021+
}
2022+
20162023
/**
20172024
* @dataProvider methodCacheableProvider
20182025
*/

< 8753 a class="Link--primary prc-Link-Link-85e08" href="#diff-ed2e471dbc8a34b7ce3547d04d3610b88e1eec1f60e61356659c4f39a35220e9" data-analytics-opt-out="true">‎src/Symfony/Component/HttpKernel/EventListener/FragmentListener.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ public function onKernelRequest(GetResponseEvent $event)
8181
protected function validateRequest(Request $request)
8282
{
8383
// is the Request safe?
84-
if (!$request->isMethodSafe()) {
84+
if (!$request->isMethodSafe(false)) {
8585
throw new AccessDeniedHttpException();
8686
}
8787

0 commit comments

Comments
 (0)
0