Bug #30493

CLI argument handling chokes on = use

Added by Karsten Dambekalns almost 4 years ago. Updated over 3 years ago.

Status:Resolved Start date:2011-10-01
Priority:Should have Due date:
Assigned To:Christian Müller % Done:

100%

Category:Command
Target version:-
PHP Version: Complexity:
Has patch: Affected Flow version:FLOW3 1.0.0

Description

When a value for an argument is given like --name=value the result is an uncaught exception #1297759968.


Related issues

related to TYPO3.Neos - Bug #30267: site:import task broken Closed 2011-09-25

Associated revisions

Revision 578b580d
Added by Christian Müller over 3 years ago

[BUGFIX] (CLI): CLI arguments should be usable with equals sign (=)

With this fix the last argument may use the equals sign (=)
again to separate between argument name and value.

Besides writing an additional test case which reproduces the issue,
the other test cases have been strengthened by replacing assertEquals()
by assertSame().

Change-Id: I15c7ca0ba50fc612b7328573ebce9f748c46fb2f
Resolves: #30493
Release: 1.0, 1.1

Revision 2b137a04
Added by Christian Müller over 3 years ago

[BUGFIX] CLI arguments should be usable with equals sign (=)

With this fix the last argument may use the equals sign (=)
again to separate between argument name and value.

Besides writing an additional test case which reproduces the issue,
the other test cases have been strengthened by replacing assertEquals()
by assertSame().

Change-Id: Iacfeef7487bf59852fcc992f4d1d6fc9ff96c07d
Fixes: #30493
Release: 1.0, 1.1

History

#1 Updated by Christian Müller almost 4 years ago

  • Assigned To set to Christian Müller

#2 Updated by Mr. Hudson almost 4 years ago

  • Status changed from New to Under Review

Patch set 1 of change I15c7ca0ba50fc612b7328573ebce9f748c46fb2f has been pushed to the review server.
It is available at http://review.typo3.org/5724

#3 Updated by Mr. Hudson almost 4 years ago

Patch set 2 of change I15c7ca0ba50fc612b7328573ebce9f748c46fb2f has been pushed to the review server.
It is available at http://review.typo3.org/5724

#4 Updated by Karsten Dambekalns almost 4 years ago

  • Target version deleted (1230)

#5 Updated by Karsten Dambekalns almost 4 years ago

  • Affected Flow version set to FLOW3 1.0.0

#6 Updated by Mr. Hudson over 3 years ago

Patch set 3 of change I15c7ca0ba50fc612b7328573ebce9f748c46fb2f has been pushed to the review server.
It is available at http://review.typo3.org/5724

#7 Updated by Christian Müller over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF