Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: symfony/symfony
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 7.4
Choose a base ref
...
head repository: symfony/symfony
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 7.3
Choose a head ref
  • 4 commits
  • 5 files changed
  • 3 contributors

Commits on Aug 7, 2025

  1. Configuration menu
    Copy the full SHA
    697fca1 View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2025

  1. bug #61033 [Console] Restore SHELL_VERBOSITY after a command is ran…

    … (lyrixx)
    
    This PR was merged into the 7.3 branch.
    
    Discussion
    ----------
    
    [Console] Restore `SHELL_VERBOSITY` after a command is ran
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.2
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Issues        |
    | License       | MIT
    
    ---
    
    Considereing the following code:
    
    ```php
    require __DIR__.'/vendor/autoload.php';
    
    use Symfony\Component\Console\Application;
    use Symfony\Component\Console\Attribute\AsCommand;
    use Symfony\Component\Console\Input\ArgvInput;
    use Symfony\Component\Console\Output\OutputInterface;
    
    #[AsCommand('a')]
    class A
    {
        public function __invoke(OutputInterface $output)
        {
            $output->writeln('Command A executed');
    
            return 0;
        }
    }
    
    #[AsCommand('b')]
    class B
    {
        public function __invoke(OutputInterface $output)
        {
            $output->writeln('Command B executed');
    
            return 0;
        }
    }
    
    #[AsCommand('main')]
    class Main
    {
        public function __construct(
            private Application $app
        ) {}
    
        public function __invoke(OutputInterface $output)
        {
            $this->app->setAutoExit(false);
    
            $this->app->run(new ArgvInput([__FILE__, 'a', '--quiet']));
    
            $this->app->run(new ArgvInput([__FILE__, 'b']));
    
            $output->writeln('Main command executed');
    
            return 0;
        }
    }
    
    $app = new Application();
    $app->addCommand(new A());
    $app->addCommand(new B());
    $app->addCommand(new Main($app));
    
    $app->run();
    ```
    
    Without this patch,
    * the output of B command is not displayed. It should be!
    * But the output `Main command executed` is. It should be, it's correct. But it's hard to understand. the current $output has already been configured, and it's not silenced by `--quiet` yet
    
    Commits
    -------
    
    697fca1 [Console] Restore SHELL_VERBOSITY after a command is ran
    fabpot committed Aug 23, 2025
    Configuration menu
    Copy the full SHA
    7c54399 View commit details
    Browse the repository at this point in the history
  2. [Messenger] Fix Oracle errors 'ORA-00955: Name is already used by an …

    …existing object' with Doctrine transport
    atgitwk authored and fabpot committed Aug 23, 2025
    Configuration menu
    Copy the full SHA
    45c3203 View commit details
    Browse the repository at this point in the history
  3. bug #60211 [Messenger] Fix Oracle errors 'ORA-00955: Name is already …

    …used by an existing object' with Doctrine transport (atgitwk)
    
    This PR was submitted for the 7.2 branch but it was squashed and merged into the 7.3 branch instead.
    
    Discussion
    ----------
    
    [Messenger] Fix Oracle errors 'ORA-00955: Name is already used by an existing object' with Doctrine transport
    
    | Q             | A
    | ------------- | ---
    | Branch?       | 7.2
    | Bug fix?      | yes
    | New feature?  | no
    | Deprecations? | no
    | Issues        | Fix #59903
    | License       | MIT
    
    Initialization of MESSENGER objects causes an Oracle error when using `messenger:setup-transports`.
    For more explanation, please see issue #59903
    
    Commits
    -------
    
    45c3203 [Messenger] Fix Oracle errors 'ORA-00955: Name is already used by an existing object' with Doctrine transport
    fabpot committed Aug 23, 2025
    Configuration menu
    Copy the full SHA
    211aaa4 View commit details
    Browse the repository at this point in the history
Loading