Skip to content

Commit e9482f0

Browse files
fabpotfabpot
committed
Merge branch '6.4' into 7.0
* 6.4: don't use deprecated and internal Twig functions [FrameworkBundle] Add missing webhook parsers [FrameworkBundle] Fix webhook parser service removal and add notifier parser service removal [Notifier][Clickatell] Fixed minor typo add missing translation [Messenger] Add missing Redis cleanup in tests Make sure Serializer::denormalize have show what exception it throws [WebProfilerBundle] Fix "Copy as cURL" dark style
2 parents 7d89a65 + 114710d commit e9482f0

File tree

12 files changed

+98
-38
lines changed

12 files changed

+98
-38
lines changed

‎src/Symfony/Bridge/Twig/Node/SearchAndRenderBlockNode.php‎

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
namespace Symfony\Bridge\Twig\Node;
1313

1414
use Twig\Compiler;
15+
use Twig\Extension\CoreExtension;
1516
use Twig\Node\Expression\ArrayExpression;
1617
use Twig\Node\Expression\ConstantExpression;
1718
use Twig\Node\Expression\FunctionExpression;
@@ -50,7 +51,7 @@ public function compile(Compiler $compiler): void
5051
$labelIsExpression = false;
5152

5253
// Only insert the label into the array if it is not empty
53-
if (!twig_test_empty($label->getAttribute('value'))) {
54+
if (null !== $label->getAttribute('value') && false !== $label->getAttribute('value') && '' !== (string) $label->getAttribute('value')) {
5455
$originalVariables = $variables;
5556
$variables = new ArrayExpression([], $lineno);
5657
$labelKey = new ConstantExpression('label', $lineno);
@@ -97,7 +98,12 @@ public function compile(Compiler $compiler): void
9798

9899
// Check at runtime whether the label is empty.
99100
// If not, add it to the array at runtime.
100-
$compiler->raw('(twig_test_empty($_label_ = ');
101+
if (method_exists(CoreExtension::class, 'testEmpty')) {
102+
$compiler->raw('(CoreExtension::testEmpty($_label_ = ');
103+
} else {
104+
$compiler->raw('(twig_test_empty($_label_ = ');
105+
}
106+
101107
$compiler->subcompile($label);
102108
$compiler->raw(') ? [] : ["label" => $_label_])');
103109
}

‎src/Symfony/Bridge/Twig/Tests/Node/SearchAndRenderBlockNodeTest.php‎

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
use Symfony\Bridge\Twig\Node\SearchAndRenderBlockNode;
1616
use Twig\Compiler;
1717
use Twig\Environment;
18+
use Twig\Extension\CoreExtension;
1819
use Twig\Loader\LoaderInterface;
1920
use Twig\Node\Expression\ArrayExpression;
2021
use Twig\Node\Expression\ConditionalExpression;
@@ -226,8 +227,9 @@ public function testCompileLabelWithLabelThatEvaluatesToNull()
226227
// https://github.com/symfony/symfony/issues/5029
227228
$this->assertEquals(
228229
sprintf(
229-
'$this->env->getRuntime(\'Symfony\Component\Form\FormRenderer\')->searchAndRenderBlock(%s, \'label\', (twig_test_empty($_label_ = ((true) ? (null) : (null))) ? [] : ["label" => $_label_]))',
230-
$this->getVariableGetter('form')
230+
'$this->env->getRuntime(\'Symfony\Component\Form\FormRenderer\')->searchAndRenderBlock(%s, \'label\', (%s($_label_ = ((true) ? (null) : (null))) ? [] : ["label" => $_label_]))',
231+
$this->getVariableGetter('form'),
232+
method_exists(CoreExtension::class, 'testEmpty') ? 'CoreExtension::testEmpty' : 'twig_test_empty'
231233
),
232234
trim($compiler->compile($node)->getSource())
233235
);
@@ -263,8 +265,9 @@ public function testCompileLabelWithLabelThatEvaluatesToNullAndAttributes()
263265
// https://github.com/symfony/symfony/issues/5029
264266
$this->assertEquals(
265267
sprintf(
266-
'$this->env->getRuntime(\'Symfony\Component\Form\FormRenderer\')->searchAndRenderBlock(%s, \'label\', ["foo" => "bar", "label" => "value in attributes"] + (twig_test_empty($_label_ = ((true) ? (null) : (null))) ? [] : ["label" => $_label_]))',
267-
$this->getVariableGetter('form')
268+
'$this->env->getRuntime(\'Symfony\Component\Form\FormRenderer\')->searchAndRenderBlock(%s, \'label\', ["foo" => "bar", "label" => "value in attributes"] + (%s($_label_ = ((true) ? (null) : (null))) ? [] : ["label" => $_label_]))',
269+
$this->getVariableGetter('form'),
270+
method_exists(CoreExtension::class, 'testEmpty') ? 'CoreExtension::testEmpty' : 'twig_test_empty'
268271
),
269272
trim($compiler->compile($node)->getSource())
270273
);

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

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2585,13 +2585,15 @@ private function registerMailerConfiguration(array $config, ContainerBuilder $co
25852585

25862586
if ($webhookEnabled) {
25872587
$webhookRequestParsers = [
2588+
MailerBridge\Brevo\Webhook\BrevoRequestParser::class => 'mailer.webhook.request_parser.brevo',
25882589
MailerBridge\Mailgun\Webhook\MailgunRequestParser::class => 'mailer.webhook.request_parser.mailgun',
2590+
MailerBridge\Mailjet\Webhook\MailjetRequestParser::class => 'mailer.webhook.request_parser.mailjet',
25892591
MailerBridge\Postmark\Webhook\PostmarkRequestParser::class => 'mailer.webhook.request_parser.postmark',
25902592
MailerBridge\Sendgrid\Webhook\SendgridRequestParser::class => 'mailer.webhook.request_parser.sendgrid',
25912593
];
25922594

25932595
foreach ($webhookRequestParsers as $class => $service) {
2594-
$package = substr($service, \strlen('mailer.transport_factory.'));
2596+
$package = substr($service, \strlen('mailer.webhook.request_parser.'));
25952597

25962598
if (!ContainerBuilder::willBeAvailable(sprintf('symfony/%s-mailer', 'gmail' === $package ? 'google' : $package), $class, ['symfony/framework-bundle', 'symfony/mailer'])) {
25972599
$container->removeDefinition($service);
@@ -2802,6 +2804,19 @@ private function registerNotifierConfiguration(array $config, ContainerBuilder $
28022804

28032805
if ($webhookEnabled) {
28042806
$loader->load('notifier_webhook.php');
2807+
2808+
$webhookRequestParsers = [
2809+
NotifierBridge\Twilio\Webhook\TwilioRequestParser::class => 'notifier.webhook.request_parser.twilio',
2810+
NotifierBridge\Vonage\Webhook\VonageRequestParser::class => 'notifier.webhook.request_parser.vonage',
2811+
];
2812+
2813+
foreach ($webhookRequestParsers as $class => $service) {
2814+
$package = substr($service, \strlen('notifier.webhook.request_parser.'));
2815+
2816+
if (!ContainerBuilder::willBeAvailable(sprintf('symfony/%s-notifier', $package), $class, ['symfony/framework-bundle', 'symfony/notifier'])) {
2817+
$container->removeDefinition($service);
2818+
}
2819+
}
28052820
}
28062821
}
28072822

‎src/Symfony/Bundle/FrameworkBundle/Resources/config/notifier_webhook.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,14 @@
1212
namespace Symfony\Component\DependencyInjection\Loader\Configurator;
1313

1414
use Symfony\Component\Notifier\Bridge\Twilio\Webhook\TwilioRequestParser;
15+
use Symfony\Component\Notifier\Bridge\Vonage\Webhook\VonageRequestParser;
1516

1617
return static function (ContainerConfigurator $container) {
1718
$container->services()
1819
->set('notifier.webhook.request_parser.twilio', TwilioRequestParser::class)
1920
->alias(TwilioRequestParser::class, 'notifier.webhook.request_parser.twilio')
21+
22+
->set('notifier.webhook.request_parser.vonage', VonageRequestParser::class)
23+
->alias(VonageRequestParser::class, 'notifier.webhook.request_parser.vonage')
2024
;
2125
};

‎src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/base_js.html.twig‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,12 +186,10 @@
186186
187187
copyToClipboardElement.textContent = `✅ Copied!`;
188188
copyToClipboardElement.disabled = true;
189-
copyToClipboardElement.classList.add('status-success');
190189
191190
setTimeout(() => {
192191
copyToClipboardElement.textContent = oldContent;
193192
copyToClipboardElement.disabled = false;
194-
copyToClipboardElement.classList.remove('status-success');
195193
}, 7000);
196194
});
197195
});

‎src/Symfony/Bundle/WebProfilerBundle/Tests/Twig/WebProfilerExtensionTest.php‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,6 @@
1515
use Symfony\Bundle\WebProfilerBundle\Twig\WebProfilerExtension;
1616
use Symfony\Component\VarDumper\Cloner\VarCloner;
1717
use Twig\Environment;
18-
use Twig\Extension\CoreExtension;
19-
use Twig\Extension\EscaperExtension;
2018

2119
class WebProfilerExtensionTest extends TestCase
2220
{
@@ -25,9 +23,6 @@ class WebProfilerExtensionTest extends TestCase
2523
*/
2624
public function testDumpHeaderIsDisplayed(string $message, array $context, bool $dump1HasHeader, bool $dump2HasHeader)
2725
{
28-
class_exists(CoreExtension::class); // Load twig_convert_encoding()
29-
class_exists(EscaperExtension::class); // Load twig_escape_filter()
30-
3126
$twigEnvironment = $this->mockTwigEnvironment();
3227
$varCloner = new VarCloner();
3328

‎src/Symfony/Bundle/WebProfilerBundle/Twig/WebProfilerExtension.php‎

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
use Symfony\Component\VarDumper\Cloner\Data;
1515
use Symfony\Component\VarDumper\Dumper\HtmlDumper;
1616
use Twig\Environment;
17+
use Twig\Extension\EscaperExtension;
1718
use Twig\Extension\ProfilerExtension;
1819
use Twig\Profiler\Profile;
1920
use Twig\TwigFunction;
@@ -78,12 +79,12 @@ public function dumpData(Environment $env, Data $data, int $maxDepth = 0): strin
7879

7980
public function dumpLog(Environment $env, string $message, Data $context = null): string
8081
{
81-
$message = twig_escape_filter($env, $message);
82+
$message = self::escape($env, $message);
8283
$message = preg_replace('/&quot;(.*?)&quot;/', '&quot;<b>$1</b>&quot;', $message);
8384

8485
$replacements = [];
8586
foreach ($context ?? [] as $k => $v) {
86-
$k = '{'.twig_escape_filter($env, $k).'}';
87+
$k = '{'.self::escape($env, $k).'}';
8788
if (str_contains($message, $k)) {
8889
$replacements[$k] = $v;
8990
}
@@ -104,4 +105,14 @@ public function getName(): string
104105
{
105106
return 'profiler';
106107
}
108+
109+
private static function escape(Environment $env, string $s): string
110+
{
111+
if (method_exists(EscaperExtension::class, 'escape')) {
112+
return EscaperExtension::escape($env, $s);
113+
}
114+
115+
// to be removed when support for Twig 3 is dropped
116+
return twig_escape_filter($env, $s);
117+
}
107118
}

‎src/Symfony/Component/Messenger/Bridge/Redis/Tests/Transport/RedisExtIntegrationTest.php‎

Lines changed: 40 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -293,15 +293,19 @@ public function testLazy()
293293
$connection = Connection::fromDsn('redis://localhost/messenger-lazy?lazy=1', [], $redis);
294294

295295
$connection->add('1', []);
296-
$this->assertNotEmpty($message = $connection->get());
297-
$this->assertSame([
298-
'message' => json_encode([
299-
'body' => '1',
300-
'headers' => [],
301-
]),
302-
], $message['data']);
303-
$connection->reject($message['id']);
304-
$redis->del('messenger-lazy');
296+
297+
try {
298+
$this->assertNotEmpty($message = $connection->get());
299+
$this->assertSame([
300+
'message' => json_encode([
301+
'body' => '1',
302+
'headers' => [],
303+
]),
304+
], $message['data']);
305+
$connection->reject($message['id']);
306+
} finally {
307+
$redis->del('messenger-lazy');
308+
}
305309
}
306310

307311
public function testDbIndex()
@@ -328,13 +332,16 @@ public function testFromDsnWithMultipleHosts()
328332
public function testJsonError()
329333
{
330334
$redis = $this->createRedisClient();
331-
$connection = Connection::fromDsn('redis://localhost/json-error', [], $redis);
335+
$connection = Connection::fromDsn('redis://localhost/messenger-json-error', [], $redis);
332336
try {
333337
$connection->add("\xB1\x31", []);
338+
339+
$this->fail('Expected exception to be thrown.');
334340
} catch (TransportException $e) {
341+
$this->assertSame('Malformed UTF-8 characters, possibly incorrectly encoded', $e->getMessage());
342+
} finally {
343+
$redis->del('messenger-json-error');
335344
}
336-
337-
$this->assertSame('Malformed UTF-8 characters, possibly incorrectly encoded', $e->getMessage());
338345
}
339346

340347
public function testGetNonBlocking()
@@ -343,29 +350,41 @@ public function testGetNonBlocking()
343350

344351
$connection = Connection::fromDsn('redis://localhost/messenger-getnonblocking', ['sentinel_master' => null], $redis);
345352

346-
$this->assertNull($connection->get()); // no message, should return null immediately
347-
$connection->add('1', []);
348-
$this->assertNotEmpty($message = $connection->get());
349-
$connection->reject($message['id']);
350-
$redis->del('messenger-getnonblocking');
353+
try {
354+
$this->assertNull($connection->get()); // no message, should return null immediately
355+
$connection->add('1', []);
356+
$this->assertNotEmpty($message = $connection->get());
357+
$connection->reject($message['id']);
358+
} finally {
359+
$redis->del('messenger-getnonblocking');
360+
}
351361
}
352362

353363
public function testGetAfterReject()
354364
{
355365
$redis = $this->createRedisClient();
356366
$connection = Connection::fromDsn('redis://localhost/messenger-rejectthenget', ['sentinel_master' => null], $redis);
357367

358-
$connection->add('1', []);
359-
$connection->add('2', []);
368+
try {
369+
$connection->add('1', []);
370+
$connection->add('2', []);
360371

361-
$failing = $connection->get();
362-
$connection->reject($failing['id']);
372+
$failing = $connection->get();
373+
$connection->reject($failing['id']);
363374

375+
<<<<<<< HEAD
364376
$connection = Connection::fromDsn('redis://localhost/messenger-rejectthenget', ['sentinel_master' => null]);
365377

366378
$this->assertNotNull($connection->get());
367379

368380
$redis->del('messenger-rejectthenget');
381+
=======
382+
$connection = Connection::fromDsn('redis://localhost/messenger-rejectthenget', ['delete_after_ack' => true]);
383+
$this->assertNotNull($connection->get());
384+
} finally {
385+
$redis->del('messenger-rejectthenget');
386+
}
387+
>>>>>>> 5.4
369388
}
370389

371390
public function testItProperlyHandlesEmptyMessages()

‎src/Symfony/Component/Notifier/Bridge/Clickatell/ClickatellTransport.php‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ protected function doSend(MessageInterface $message): SentMessage
7979
try {
8080
$statusCode = $response->getStatusCode();
8181
} catch (TransportExceptionInterface $e) {
82-
throw new TransportException('Could not reach the remote Clicktell server.', $response, 0, $e);
82+
throw new TransportException('Could not reach the remote Clickatell server.', $response, 0, $e);
8383
}
8484

8585
if (202 === $statusCode) {

‎src/Symfony/Component/Serializer/Serializer.php‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -184,6 +184,7 @@ public function normalize(mixed $data, string $format = null, array $context = [
184184

185185
/**
186186
* @throws NotNormalizableValueException
187+
* @throws PartialDenormalizationException Occurs when one or more properties of $type fails to denormalize
187188
*/
188189
public function denormalize(mixed $data, string $type, string $format = null, array $context = []): mixed
189190
{

0 commit comments

Comments
 (0)