Skip to content

Commit

Permalink
Use AbstractLogger instead of Closure in LoadDataFixturesDoctrineODMC…
Browse files Browse the repository at this point in the history
…ommand.
  • Loading branch information
mickverm committed Nov 29, 2024
1 parent 33a9d71 commit a7d204d
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
4 changes: 2 additions & 2 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@
},
"require-dev": {
"doctrine/coding-standard": "^11.0",
"doctrine/data-fixtures": "^1.7",
"doctrine/data-fixtures": "^1.8 || ^2.0",
"phpunit/phpunit": "^9.5.5",
"psalm/plugin-symfony": "^5.0",
"symfony/browser-kit": "^6.4 || ^7.0",
Expand All @@ -53,7 +53,7 @@
"vimeo/psalm": "^5.25"
},
"conflict": {
"doctrine/data-fixtures": "<1.3"
"doctrine/data-fixtures": "<1.8"
},
"suggest": {
"doctrine/data-fixtures": "Load data fixtures"
Expand Down
10 changes: 8 additions & 2 deletions src/Command/LoadDataFixturesDoctrineODMCommand.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
use Doctrine\Bundle\MongoDBBundle\ManagerRegistry;
use Doctrine\Common\DataFixtures\Executor\MongoDBExecutor;
use Doctrine\Common\DataFixtures\Purger\MongoDBPurger;
use Psr\Log\AbstractLogger;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;
Expand Down Expand Up @@ -83,8 +84,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int

$purger = new MongoDBPurger($dm);
$executor = new MongoDBExecutor($dm, $purger);
$executor->setLogger(static function ($message) use ($output): void {
$output->writeln(sprintf(' <comment>></comment> <info>%s</info>', $message));
$executor->setLogger(new class($output) extends AbstractLogger {

Check failure on line 87 in src/Command/LoadDataFixturesDoctrineODMCommand.php

View workflow job for this annotation

GitHub Actions / Coding Standards / Coding Standards (PHP: 8.3)

Expected 1 space after class keyword; 0 found
public function __construct(private readonly OutputInterface $output) {}

Check failure on line 88 in src/Command/LoadDataFixturesDoctrineODMCommand.php

View workflow job for this annotation

GitHub Actions / Coding Standards / Coding Standards (PHP: 8.3)

Opening brace should be on a new line

Check failure on line 88 in src/Command/LoadDataFixturesDoctrineODMCommand.php

View workflow job for this annotation

GitHub Actions / Coding Standards / Coding Standards (PHP: 8.3)

Closing brace must be on a line by itself

public function log(mixed $level, string|\Stringable $message, array $context = []): void

Check failure on line 90 in src/Command/LoadDataFixturesDoctrineODMCommand.php

View workflow job for this annotation

GitHub Actions / Coding Standards / Coding Standards (PHP: 8.3)

Class \Stringable should not be referenced via a fully qualified name, but via a use statement.
{
$this->output->writeln(sprintf(' <comment>></comment> <info>%s</info>', $message));
}
});
$executor->execute($fixtures, $input->getOption('append'));

Expand Down

0 comments on commit a7d204d

Please sign in to comment.