bug #36696 [Console] don't check tty on stdin, it breaks with "data lost during stream conversion" (nicolas-grekas)

fabpot · fabpot · commit cf6e499baf4a · 2020-05-05T13:55:38.000+02:00
This PR was merged into the 4.4 branch. Discussion ---------- [Console] don't check tty on stdin, it breaks with "data lost during stream conversion" | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - /cc @ostrolucky FYI Commits ------- 6d089ac [Console] fix "data lost during stream conversion" with QuestionHelper
diff --git a/src/Symfony/Component/Console/Helper/QuestionHelper.php b/src/Symfony/Component/Console/Helper/QuestionHelper.php
@@ -462,10 +462,6 @@ private function validateAttempts(callable $interviewer, OutputInterface $output
         $error = null;
         $attempts = $question->getMaxAttempts();
 
-        if (null === $attempts && !$this->isTty()) {
-            $attempts = 1;
-        }
-
         while (null === $attempts || $attempts--) {
             if (null !== $error) {
                 $this->writeError($output, $error);
@@ -477,6 +473,8 @@ private function validateAttempts(callable $interviewer, OutputInterface $output
                 throw $e;
             } catch (\Exception $error) {
             }
+
+            $attempts = $attempts ?? -(int) $this->isTty();
         }
 
         throw $error;
@@ -517,7 +515,7 @@ private function isTty(): bool
             return stream_isatty($inputStream);
         }
 
-        if (!\function_exists('posix_isatty')) {
+        if (\function_exists('posix_isatty')) {
             return posix_isatty($inputStream);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -462,10 +462,6 @@ private function validateAttempts(callable $interviewer, OutputInterface $output`
`462`	`462`	`$error = null;`
`463`	`463`	`$attempts = $question->getMaxAttempts();`
`464`	`464`
`465`		`- if (null === $attempts && !$this->isTty()) {`
`466`		`- $attempts = 1;`
`467`		`- }`
`468`		`-`
`469`	`465`	`while (null === $attempts \|\| $attempts--) {`
`470`	`466`	`if (null !== $error) {`
`471`	`467`	`$this->writeError($output, $error);`
`@@ -477,6 +473,8 @@ private function validateAttempts(callable $interviewer, OutputInterface $output`
`477`	`473`	`throw $e;`
`478`	`474`	`} catch (\Exception $error) {`
`479`	`475`	`}`
	`476`	`+`
	`477`	`+ $attempts = $attempts ?? -(int) $this->isTty();`
`480`	`478`	`}`
`481`	`479`
`482`	`480`	`throw $error;`
`@@ -517,7 +515,7 @@ private function isTty(): bool`
`517`	`515`	`return stream_isatty($inputStream);`
`518`	`516`	`}`
`519`	`517`
`520`		`- if (!\function_exists('posix_isatty')) {`
	`518`	`+ if (\function_exists('posix_isatty')) {`
`521`	`519`	`return posix_isatty($inputStream);`
`522`	`520`	`}`
`523`	`521`