Skip to content

Commit

Permalink
minor #4043 Use ::class everywhere (fabpot)
Browse files Browse the repository at this point in the history
This PR was merged into the 3.x branch.

Discussion
----------

Use ::class everywhere

Commits
-------

a71ae19 Use ::class everywhere
  • Loading branch information
fabpot committed Apr 19, 2024
2 parents 12625c0 + a71ae19 commit 861c76c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/Extension/CoreExtension.php
Expand Up @@ -1837,7 +1837,7 @@ public static function arrayEvery(Environment $env, $array, $arrow)
*/
public static function checkArrowInSandbox(Environment $env, $arrow, $thing, $type)
{
if (!$arrow instanceof \Closure && $env->hasExtension('\Twig\Extension\SandboxExtension') && $env->getExtension('\Twig\Extension\SandboxExtension')->isSandboxed()) {
if (!$arrow instanceof \Closure && $env->hasExtension(SandboxExtension::class) && $env->getExtension(SandboxExtension::class)->isSandboxed()) {
throw new RuntimeError(sprintf('The callable passed to the "%s" %s must be a Closure in sandbox mode.', $thing, $type));
}
}
Expand Down
4 changes: 2 additions & 2 deletions tests/EnvironmentTest.php
Expand Up @@ -271,8 +271,8 @@ public function testHasGetExtensionByClassName()
{
$twig = new Environment($this->createMock(LoaderInterface::class));
$twig->addExtension($ext = new EnvironmentTest_Extension());
$this->assertSame($ext, $twig->getExtension('Twig\Tests\EnvironmentTest_Extension'));
$this->assertSame($ext, $twig->getExtension('\Twig\Tests\EnvironmentTest_Extension'));
$this->assertSame($ext, $twig->getExtension(EnvironmentTest_Extension::class));
$this->assertSame($ext, $twig->getExtension(EnvironmentTest_Extension::class));
}

public function testAddExtension()
Expand Down
2 changes: 1 addition & 1 deletion tests/Extension/EscaperTest.php
Expand Up @@ -400,7 +400,7 @@ public function testObjectEscaping(string $escapedHtml, string $escapedJs, array
{
$obj = new Extension_TestClass();
$twig = new Environment($this->createMock(LoaderInterface::class));
$twig->getExtension('\Twig\Extension\EscaperExtension')->setSafeClasses($safeClasses);
$twig->getExtension(EscaperExtension::class)->setSafeClasses($safeClasses);
$this->assertSame($escapedHtml, EscaperExtension::escape($twig, $obj, 'html', null, true));
$this->assertSame($escapedJs, EscaperExtension::escape($twig, $obj, 'js', null, true));
}
Expand Down

0 comments on commit 861c76c

Please sign in to comment.