Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Fix console mixed case optional value params #5951

Conversation

Thinkscape
Copy link
Member

This one slipped through.

When using optional value params in console routes, i.e.

do something [<withThisData>]

Given the following script invocation:

./zf2 do something "with foo"

We would receive the following route match:

// actual
[
  'withthisdata' => 'with foo'
]

Where it should be:

// expected
[
  'withThisData' => 'with foo'
]

Notice that withThisData parameter became lowercased withthisdata.

@weierophinney - we need to include a note on that in release notes, in case someone just ignored the bug and consumed all-lowercased parameter names.

weierophinney added a commit that referenced this pull request Mar 11, 2014
…tional-value-params

Fix console mixed case optional value params
weierophinney added a commit that referenced this pull request Mar 11, 2014
@weierophinney weierophinney merged commit e282acb into zendframework:develop Mar 11, 2014
@weierophinney weierophinney modified the milestones: 2.3.1, 2.3.0 Mar 11, 2014
@weierophinney weierophinney self-assigned this Mar 11, 2014
weierophinney added a commit to zendframework/zend-console that referenced this pull request May 15, 2015
…otfix/console-mixed-case-optional-value-params

Fix console mixed case optional value params
weierophinney added a commit to zendframework/zend-console that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants