Closed Bug 671837 Opened 13 years ago Closed 12 years ago

GCLI should not allow boolean parameters to be positional

Categories

(DevTools :: Console, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 17

People

(Reporter: jwalker, Unassigned)

Details

If we allow positional boolean parameters, then this is not only allowed, but what is generated by the dialog for a command with a single boolean param:

> foo true

Clicking on the checkbox again gives:

> foo false

Which is the same as 'foo' since all boolean params default to false. This is confusing, particularly given the following 2 synonym for 'foo true'

> foo --name
> foo -n

It seems sensible to just disallow positional boolean params, forcing use of 'foo --name' or 'foo -n'.
Blocks: GCLI-HELP
This won't show up if we don't ship commands that violate this rule. Move to GCLI-FUTURE
Blocks: GCLI-FUTURE
No longer blocks: GCLI-HELP
Moving GCLI bugs to Developer Tools: Console. Filter on 'baked beans are off'.
Component: Developer Tools → Developer Tools: Console
Triage. Filter on PEGASUS.
Priority: -- → P2
Triage. Filter on PEGASUS.
No longer blocks: GCLI-FUTURE
Target Milestone: --- → Firefox 14
GCLI Triage.
Target Milestone: Firefox 14 → Firefox 16
Target Milestone: Firefox 16 → Firefox 17
Triage.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.