Documented the support of iterators in write() and writeln() #9776

javiereguiluz · 2018-05-15T14:02:36Z

This fixes #9634.

linaori · 2018-05-17T10:29:06Z

console.rst

@@ -101,6 +101,10 @@ messages to the console)::
            '',
        ]);

+        // the value returned by someMethod() can be an iterator that
+        // generates and returns the messages with the 'yield' PHP keyword
+        $output->writeln($this->someMethod());


Would it be an idea to mention the Iterator? http://php.net/manual/en/class.iterator.php Could hint towards yield support next to that, but yield is a more complex concept than just an iterator

…n() (javiereguiluz) This PR was squashed before being merged into the 4.1 branch (closes #9776). Discussion ---------- Documented the support of iterators in write() and writeln() This fixes #9634. Commits ------- f9b9c10 Documented the support of iterators in write() and writeln()

Documented the support of iterators in write() and writeln()

92a468a

javiereguiluz added Missing Documentation Console labels May 15, 2018

javiereguiluz added this to the 4.1 milestone May 15, 2018

carsonbot added the Status: Needs Review label May 15, 2018

linaori reviewed May 17, 2018

View reviewed changes

Added a link to PHP's iterator article

6e2b88c

javiereguiluz added Status: Reviewed and removed Status: Needs Review labels May 22, 2018

javiereguiluz closed this May 22, 2018

javiereguiluz mentioned this pull request May 22, 2018

[Console] Support iterable in some methods #9634

Closed

javiereguiluz deleted the fix_9634 branch May 24, 2018 16:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Documented the support of iterators in write() and writeln() #9776

Documented the support of iterators in write() and writeln() #9776

javiereguiluz commented May 15, 2018

linaori May 17, 2018

Documented the support of iterators in write() and writeln() #9776

Documented the support of iterators in write() and writeln() #9776

Conversation

javiereguiluz commented May 15, 2018

linaori May 17, 2018

Choose a reason for hiding this comment