http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692014-11-21T09:01:45ZTYPO3 ForgeTYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2371632014-11-21T09:01:45ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></ul><p>Patch set 1 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34425">http://review.typo3.org/34425</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2371672014-11-21T09:56:37ZGerrit Code Review
<ul></ul><p>Patch set 2 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34425">http://review.typo3.org/34425</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2371882014-11-21T12:19:37ZHelmut Hummeltypo3@helhum.io
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Rejected</i></li></ul><p>Thanks for the report and your efforts.<br />However it unfortunately does not fit in the planned changes of command controllers which will be integrated for 7LTS</p>
<p>You not always need to specify the dashed name at all. most of the time you can just pass arguments without names.<br />Besides that, I like to include the change from Flow which uses Symfony console, where instead of an exception to be thrown, the user can directly enter the required argument.<br />(all this is already implemented in typo3_console which will end up in the core in some release but not 7.0 any more)</p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2371912014-11-21T12:34:08ZHelmut Hummeltypo3@helhum.io
<ul></ul><p>To explain a bit more I take your example:</p>
<pre>
$ ./typo3/cli_dispatch.phpsh extbase message:greet
Required argument "sayHelloTo" is not set
$
</pre>
<p>Then you can provide the missing argument just like that to make it work:</p>
<pre>
$ ./typo3/cli_dispatch.phpsh extbase message:greet Helmut
Hello Helmut
$
</pre> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2371972014-11-21T12:53:35ZMathias Brodalambrodala@pagemachine.de
<ul></ul><p>I get what you mean. ;-) See my comment on the review server.</p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372122014-11-21T14:03:53ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>Rejected</i> to <i>Under Review</i></li></ul><p>Patch set 3 for branch <strong>master</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34425">http://review.typo3.org/34425</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372132014-11-21T14:13:35ZGerrit Code Review
<ul></ul><p>Patch set 1 for branch <strong>TYPO3_6-2</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34449">http://review.typo3.org/34449</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372152014-11-21T14:30:31ZMathias Brodalambrodala@pagemachine.de
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>50</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="[BUGFIX] Output dashed name for missing CLI arguments This changes the command argument name out..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/dcf2e68b4ef718ab5b4b2bc017af949b751df45c">dcf2e68b4ef718ab5b4b2bc017af949b751df45c</a>.</p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372172014-11-21T14:54:46ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Under Review</i></li></ul><p>Patch set 2 for branch <strong>TYPO3_6-2</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34449">http://review.typo3.org/34449</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372182014-11-21T14:55:54ZGerrit Code Review
<ul></ul><p>Patch set 3 for branch <strong>TYPO3_6-2</strong> of project <strong>Packages/TYPO3.CMS</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/34449">http://review.typo3.org/34449</a></p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=2372302014-11-21T16:30:22ZMathias Brodalambrodala@pagemachine.de
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li></ul><p>Applied in changeset <a class="changeset" title="[BUGFIX] Output dashed name for missing CLI arguments This changes the command argument name out..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/48f3fd9e295fb3d81b2d004994b40362e536493a">48f3fd9e295fb3d81b2d004994b40362e536493a</a>.</p> TYPO3 Core - Task #61630: Output dashed name for missing command arguments on CLIhttp://forge.typo3.org/issues/61630?journal_id=3824512018-10-02T11:52:46ZBenni Mackbenni@typo3.org
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>