Skip to content

Commit 372c817

Browse files
committed
Merge branch '7.3' into 7.4
* 7.3: [WebProfilerBundle] Upgrade mermaid to v11.12.0 Simplify ConsoleFormatterTest [Security] Fix preload warning in `AuthorizationChecker`
2 parents ce54488 + de83371 commit 372c817

File tree

4 files changed

+308
-369
lines changed

4 files changed

+308
-369
lines changed

src/Symfony/Bridge/Monolog/Tests/Formatter/ConsoleFormatterTest.php

Lines changed: 3 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -11,51 +11,17 @@
1111

1212
namespace Symfony\Bridge\Monolog\Tests\Formatter;
1313

14-
use Monolog\Logger;
15-
use Monolog\LogRecord;
16-
use PHPUnit\Framework\Attributes\DataProvider;
1714
use PHPUnit\Framework\TestCase;
1815
use Symfony\Bridge\Monolog\Formatter\ConsoleFormatter;
1916
use Symfony\Bridge\Monolog\Tests\RecordFactory;
2017

2118
class ConsoleFormatterTest extends TestCase
2219
{
23-
#[DataProvider('providerFormatTests')]
24-
public function testFormat(array|LogRecord $record, $expectedMessage)
20+
public function testFormat()
2521
{
22+
$record = RecordFactory::create(datetime: new \DateTimeImmutable('2013-01-13 12:34:56 Europe/Berlin'));
2623
$formatter = new ConsoleFormatter();
27-
self::assertSame($expectedMessage, $formatter->format($record));
28-
}
29-
30-
public static function providerFormatTests(): array
31-
{
32-
$currentDateTime = new \DateTimeImmutable();
33-
34-
$tests = [
35-
'record with DateTime object in datetime field' => [
36-
'record' => RecordFactory::create(datetime: $currentDateTime),
37-
'expectedMessage' => \sprintf(
38-
"%s <fg=cyan>WARNING </> <comment>[test]</> test\n",
39-
$currentDateTime->format(ConsoleFormatter::SIMPLE_DATE)
40-
),
41-
],
42-
];
43-
44-
if (Logger::API < 3) {
45-
$tests['record with string in datetime field'] = [
46-
'record' => [
47-
'message' => 'test',
48-
'context' => [],
49-
'level' => Level::Warning,
50-
'level_name' => Logger::getLevelName(Level::Warning),
51-
'channel' => 'test',
52-
'datetime' => '2019-01-01T00:42:00+00:00',
53-
'extra' => [],
54-
],
55-
'expectedMessage' => "2019-01-01T00:42:00+00:00 <fg=cyan>WARNING </> <comment>[test]</> test\n",
56-
];
57-
}
5824

59-
return $tests;
25+
self::assertSame("12:34:56 <fg=cyan>WARNING </> <comment>[test]</> test\n", $formatter->format($record));
6026
}
6127
}

src/Symfony/Bundle/WebProfilerBundle/Resources/views/Script/Mermaid/Makefile

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,32 @@
11
define diagram-orchestration
2-
import { diagram as flowchartV2 } from '../diagrams/flowchart/flowDiagram-v2.js';
2+
import flowchartV2 from '../diagrams/flowchart/flowDetector-v2.js';
33
import { registerDiagram } from './diagramAPI.js';
44

55
let hasLoadedDiagrams = false;
6-
export const addDiagrams = () => {
6+
export const addDiagrams = async () => {
77
if (hasLoadedDiagrams) {
88
return;
99
}
1010
hasLoadedDiagrams = true;
11-
registerDiagram('flowchart-v2', flowchartV2, () => true);
11+
const {id, diagram} = await flowchartV2.loader();
12+
registerDiagram(id, diagram, () => true);
1213
};
1314
endef
1415

15-
override tag := v10.9.0
16+
override tag := 11.12.0
1617

1718
.PHONY: mermaid-flowchart-v2.min.js
1819
mermaid-flowchart-v2.min.js: | repo-$(tag)/node_modules
1920
$(file >repo-$(tag)/packages/mermaid/src/diagram-api/diagram-orchestration.ts,$(diagram-orchestration))
20-
pnpm -C repo-$(tag) run build
21-
cp repo-$(tag)/packages/mermaid/dist/mermaid.min.js $@
21+
pnpm -C repo-$(tag) run build:mermaid
22+
cp repo-$(tag)/packages/tiny/dist/mermaid.tiny.js $@
2223

2324
repo-$(tag)/node_modules: | repo-$(tag)
2425
pnpm -C $(@D) install --ignore-scripts
2526

2627
.SECONDARY: repo-$(tag)
2728
repo-$(tag):
28-
curl -fL https://github.com/mermaid-js/mermaid/archive/refs/tags/$(tag).tar.gz | tar -xz --strip-components=1 --one-top-level=$@
29+
curl -fL https://github.com/mermaid-js/mermaid/archive/refs/tags/mermaid@$(tag).tar.gz | tar -xz --strip-components=1 --one-top-level=$@
2930

3031
.PHONY: clean
3132
clean:

src/Symfony/Bundle/WebProfilerBundle/Resources/views/Script/Mermaid/mermaid-flowchart-v2.min.js

Lines changed: 293 additions & 325 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Symfony/Component/Security/Core/Authorization/AuthorizationChecker.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,10 @@
1717
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
1818
use Symfony\Component\Security\Core\User\UserInterface;
1919

20+
// Help opcache.preload discover always-needed symbols
21+
class_exists(AbstractToken::class);
22+
class_exists(OfflineTokenInterface::class);
23+
2024
/**
2125
* AuthorizationChecker is the main authorization point of the Security component.
2226
*

0 commit comments

Comments
 (0)