Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 54d3d63

Browse filesBrowse files
chalasrchalasr
committed
Centralize input stream in base Input class
Add StreamableInputInterface Removed commented code Prevent BC by looking for QuestionHelper:: Logic fixes Check for that implements StreamableInputInterface Rollback E_USER_DEPRECATED notice in getInputStream Remove legacy tests to avoid KO because of deprecations Add missing use Keep old tests marked as legacy Undeprecate getInputStream, CS Fixes Move legacy tests in separated class Revert separated legacy test class Keep legacy createInputInterfaceMock() Depreciate QuestionHelper::getInputStream()
1 parent f400f01 commit 54d3d63
Copy full SHA for 54d3d63

File tree

Expand file treeCollapse file tree

7 files changed

+502
-52
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

7 files changed

+502
-52
lines changed
Open diff view settings
Collapse file

‎src/Symfony/Component/Console/Application.php‎

Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Application.php
+14-2Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
use Symfony\Component\Console\Helper\ProcessHelper;
1717
use Symfony\Component\Console\Helper\QuestionHelper;
1818
use Symfony\Component\Console\Input\InputInterface;
19+
use Symfony\Component\Console\Input\StreamableInputInterface;
1920
use Symfony\Component\Console\Input\ArgvInput;
2021
use Symfony\Component\Console\Input\ArrayInput;
2122
use Symfony\Component\Console\Input\InputDefinition;
@@ -769,8 +770,19 @@ protected function configureIO(InputInterface $input, OutputInterface $output)
769770

770771
if (true === $input->hasParameterOption(array('--no-interaction', '-n'), true)) {
771772
$input->setInteractive(false);
772-
} elseif (function_exists('posix_isatty') && $this->getHelperSet()->has('question')) {
773-
$inputStream = $this->getHelperSet()->get('question')->getInputStream();
773+
} elseif (function_exists('posix_isatty')) {
774+
$inputStream = null;
775+
776+
if ($input instanceof StreamableInputInterface) {
777+
$inputStream = $input->getStream();
778+
}
779+
780+
// This check ensures that calling QuestionHelper::setInputStream() works
781+
// To be removed in 4.0 (in the same time as QuestionHelper::setInputStream)
782+
if (!$inputStream && $this->getHelperSet()->has('question')) {
783+
$inputStream = $this->getHelperSet()->get('question')->getInputStream(false);
784+
}
785+
774786
if (!@posix_isatty($inputStream) && false === getenv('SHELL_INTERACTIVE')) {
775787
$input->setInteractive(false);
776788
}
Collapse file

‎src/Symfony/Component/Console/Helper/QuestionHelper.php‎

Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Helper/QuestionHelper.php
+18-1Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
use Symfony\Component\Console\Exception\InvalidArgumentException;
1515
use Symfony\Component\Console\Exception\RuntimeException;
1616
use Symfony\Component\Console\Input\InputInterface;
17+
use Symfony\Component\Console\Input\StreamableInputInterface;
1718
use Symfony\Component\Console\Output\ConsoleOutputInterface;
1819
use Symfony\Component\Console\Output\OutputInterface;
1920
use Symfony\Component\Console\Formatter\OutputFormatterStyle;
@@ -52,6 +53,10 @@ public function ask(InputInterface $input, OutputInterface $output, Question $qu
5253
return $question->getDefault();
5354
}
5455

56+
if ($input instanceof StreamableInputInterface && $stream = $input->getStream()) {
57+
$this->inputStream = $stream;
58+
}
59+
5560
if (!$question->getValidator()) {
5661
return $this->doAsk($output, $question);
5762
}
@@ -68,12 +73,17 @@ public function ask(InputInterface $input, OutputInterface $output, Question $qu
6873
*
6974
* This is mainly useful for testing purpose.
7075
*
76+
* @deprecated since version 3.2, to be removed in 4.0. Use
77+
* StreamableInputInterface::setStream() instead.
78+
*
7179
* @param resource $stream The input stream
7280
*
7381
* @throws InvalidArgumentException In case the stream is not a resource
7482
*/
7583
public function setInputStream($stream)
7684
{
85+
@trigger_error(sprintf('The %s() method is deprecated since version 3.2 and will be removed in 4.0. Use %s:setStream() instead.', __METHOD__, StreamableInputInterface::class), E_USER_DEPRECATED);
86+
7787
if (!is_resource($stream)) {
7888
throw new InvalidArgumentException('Input stream must be a valid resource.');
7989
}
@@ -84,10 +94,17 @@ public function setInputStream($stream)
8494
/**
8595
* Returns the helper's input stream.
8696
*
97+
* @deprecated since version 3.2, to be removed in 4.0. Use
98+
* StreamableInputInterface::getStream() instead.
99+
*
87100
* @return resource
88101
*/
89-
public function getInputStream()
102+
public function getInputStream($triggerDeprecationError = true)
90103
{
104+
if ($triggerDeprecationError) {
105+
@trigger_error(sprintf('The %s() method is deprecated since version 3.2 and will be removed in 4.0. Use %s:getStream() instead.', __METHOD__, StreamableInputInterface::class), E_USER_DEPRECATED);
106+
}
107+
91108
return $this->inputStream;
92109
}
93110

Collapse file

‎src/Symfony/Component/Console/Input/Input.php‎

Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Input/Input.php
+18-1Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,13 @@
2525
*
2626
* @author Fabien Potencier <fabien@symfony.com>
2727
*/
28-
abstract class Input implements InputInterface
28+
abstract class Input implements InputInterface, StreamableInputInterface
2929
{
3030
/**
3131
* @var InputDefinition
3232
*/
3333
protected $definition;
34+
protected $stream;
3435
protected $options = array();
3536
protected $arguments = array();
3637
protected $interactive = true;
@@ -233,4 +234,20 @@ public function escapeToken($token)
233234
{
234235
return preg_match('{^[\w-]+$}', $token) ? $token : escapeshellarg($token);
235236
}
237+
238+
/**
239+
* {@inheritdoc}
240+
*/
241+
public function setStream($stream)
242+
{
243+
$this->stream = $stream;
244+
}
245+
246+
/**
247+
* {@inheritdoc}
248+
*/
249+
public function getStream()
250+
{
251+
return $this->stream;
252+
}
236253
}
Collapse file

‎src/Symfony/Component/Console/Input/StreamableInputInterface.php‎

Copy file name to clipboard
+37Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
<?php
2+
3+
/*
4+
* This file is part of the Symfony package.
5+
*
6+
* (c) Fabien Potencier <fabien@symfony.com>
7+
*
8+
* For the full copyright and license information, please view the LICENSE
9+
* file that was distributed with this source code.
10+
*/
11+
12+
namespace Symfony\Component\Console\Input;
13+
14+
/**
15+
* StreamableInputInterface is the interface implemented by all input classes
16+
* that have an input stream.
17+
*
18+
* @author Robin Chalas <robin.chalas@gmail.com>
19+
*/
20+
interface StreamableInputInterface extends InputInterface
21+
{
22+
/**
23+
* Sets the input stream to read from when interacting with the user.
24+
*
25+
* This is mainly useful for testing purpose.
26+
*
27+
* @param resource $stream The input stream
28+
*/
29+
public function setStream($stream);
30+
31+
/**
32+
* Returns the input stream.
33+
*
34+
* @return resource|null
35+
*/
36+
public function getStream();
37+
}
Collapse file

‎src/Symfony/Component/Console/Tests/ApplicationTest.php‎

Copy file name to clipboardExpand all lines: src/Symfony/Component/Console/Tests/ApplicationTest.php
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1099,7 +1099,7 @@ public function testCanCheckIfTerminalIsInteractive()
10991099

11001100
$this->assertFalse($tester->getInput()->hasParameterOption(array('--no-interaction', '-n')));
11011101

1102-
$inputStream = $application->getHelperSet()->get('question')->getInputStream();
1102+
$inputStream = $tester->getInput()->getStream();
11031103
$this->assertEquals($tester->getInput()->isInteractive(), @posix_isatty($inputStream));
11041104
}
11051105
}

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.