Mirrors/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell synced 2024-12-26 04:43:10 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix of multiple synonyms for apm command

Browse source 
`__fish_apm_using_command` was incorrectly taking lists of commands, new function added to support multiple a command having synonyms.

Simplify switch statement

Also remove superfluous function.

Allow for multiple completions after a command

Useful for removing packages, will complete for more than one.

Code improvements
This commit is contained in:
Dan Underwood 2015-09-27 19:48:12 +01:00 committed by Dan Underwood
parent 126c8c14b8
commit 37f4cbe3d7
1 changed files with 9 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
share/completions/apm.fish
View file

@ -18,21 +18,17 @@ function __fish_apm_needs_command
return 1
end
# Returns exit code of 0 if a command (argv[1]) appears once, exclusions can
# can be provided (argv[2..-1]) that are ignored, e.g. `get` and `set`
# Returns exit code of 0 if any command (argv[1..-1]) appears once, ignores flags.
function __fish_apm_using_command
set command $argv[1]
if test (count $argv) -gt 1
set exclusions $argv[2..-1]
end
set commands $argv
set cmd (commandline -opc)
if test (count $cmd) -gt 1
set -l command_seen_once 1
for c in $cmd[2..-1]
switch $c
case '--color' '--no-color' $exclusions
case '-*'
continue
case $argv
case $commands
# If the command is seen more than once then return 1
if test $command_seen_once -eq 1
set command_seen_once 0
@ -40,7 +36,11 @@ function __fish_apm_using_command
return 1
end
case '*'
return 1
if test $command_seen_once -eq 0
return 0
else
return 1
end
end
end
return $command_seen_once