bug #22172 Fix port usage in server:status command (alcaeus)

fabpot · fabpot · commit 7c5e3c09e91d · 2017-03-27T07:09:20.000-07:00
This PR was merged into the 2.7 branch. Discussion ---------- Fix port usage in server:status command | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | License | MIT This fixes a bug where running `console server:status -p 8001` won't work because the port is already contained in the default value for `address`. Commits ------- dbcfa5c Remove port from default host in server:status command
diff --git a/src/Symfony/Bundle/FrameworkBundle/Command/ServerStatusCommand.php b/src/Symfony/Bundle/FrameworkBundle/Command/ServerStatusCommand.php
@@ -31,7 +31,7 @@ protected function configure()
     {
         $this
             ->setDefinition(array(
-                new InputArgument('address', InputArgument::OPTIONAL, 'Address:port', '127.0.0.1:8000'),
+                new InputArgument('address', InputArgument::OPTIONAL, 'Address:port', '127.0.0.1'),
                 new InputOption('port', 'p', InputOption::VALUE_REQUIRED, 'Address port number', '8000'),
             ))
             ->setName('server:status')

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ protected function configure()`
`31`	`31`	`{`
`32`	`32`	`$this`
`33`	`33`	`->setDefinition(array(`
`34`		`- new InputArgument('address', InputArgument::OPTIONAL, 'Address:port', '127.0.0.1:8000'),`
	`34`	`+ new InputArgument('address', InputArgument::OPTIONAL, 'Address:port', '127.0.0.1'),`
`35`	`35`	`new InputOption('port', 'p', InputOption::VALUE_REQUIRED, 'Address port number', '8000'),`
`36`	`36`	`))`
`37`	`37`	`->setName('server:status')`