X-Git-Url: https://git.proxmox.com/?p=pve-common.git;a=blobdiff_plain;f=src%2FPVE%2FCLIHandler.pm;h=810467d46eb28db8ac2a997901a0a22cb1b787ea;hp=9bbc15627bff5a385d4198fc80b77e643b3b81d4;hb=57d5edee2fe1d27c6ae9fa5ddef6f6a17dd87861;hpb=24ccf353c76bcb346bf86761358b638e3105e298 diff --git a/src/PVE/CLIHandler.pm b/src/PVE/CLIHandler.pm index 9bbc156..810467d 100644 --- a/src/PVE/CLIHandler.pm +++ b/src/PVE/CLIHandler.pm @@ -54,7 +54,9 @@ my $expand_command_name = sub { return $cmd if exists $def->{$cmd}; # command is already complete - my @expanded = grep { /^\Q$cmd\E/ } keys %$def; + my $is_alias = sub { ref($_[0]) eq 'HASH' && exists($_[0]->{alias}) }; + my @expanded = grep { /^\Q$cmd\E/ && !$is_alias->($def->{$_}) } keys %$def; + return $expanded[0] if scalar(@expanded) == 1; # enforce exact match return undef; @@ -67,6 +69,24 @@ my $get_commands = sub { my $complete_command_names = sub { $get_commands->($cmddef) }; +my $standard_mappings = { }; + +sub get_standard_mapping { + my ($name, $base) = @_; + + my $std = $standard_mappings->{$name}; + die "no such standard mapping '$name'\n" if !$std; + + my $res = $base || {}; + + foreach my $opt (keys %$std) { + next if defined($res->{$opt}); + $res->{$opt} = $std->{$opt}; + } + + return $res; +} + # traverses the command definition using the $argv array, resolving one level # of aliases. # Returns the matching (sub) command and its definition, and argument array for @@ -314,7 +334,7 @@ my $print_bash_completion = sub { shift @$args; # no need for program name my $print_result = sub { foreach my $p (@_) { - print "$p\n" if $p =~ m/^$cur/; + print "$p\n" if $p =~ m/^\Q$cur\E/; } }; @@ -397,6 +417,92 @@ my $print_bash_completion = sub { &$print_result(@option_list); }; +# prints a formatted table with a title row. +# $formatopts is an array of hashes, with the following keys: +# 'key' - key of $data element to print +# 'title' - column title, defaults to 'key' - won't get cutoff +# 'cutoff' - maximal (print) length of this column values, if set +# the last column will never be cutoff +# 'default' - optional default value for the column +# formatopts element order defines column order (left to right) +sub print_text_table { + my ($formatopts, $data) = @_; + + my ($formatstring, @keys, @titles, %cutoffs, %defaults); + my $last_col = $formatopts->[$#{$formatopts}]; + + foreach my $col ( @$formatopts ) { + my ($key, $title, $cutoff, $default) = @$col{qw(key title cutoff default)}; + $title //= $key; + + push @keys, $key; + push @titles, $title; + $defaults{$key} = $default; + + # calculate maximal print width and cutoff + my $titlelen = length($title); + + my $longest = $titlelen; + foreach my $entry (@$data) { + my $len = length($entry->{$key}) // 0; + $longest = $len if $len > $longest; + } + + $cutoff = (defined($cutoff) && $cutoff < $longest) ? $cutoff : $longest; + $cutoffs{$key} = $cutoff; + + my $printalign = $cutoff > $titlelen ? '-' : ''; + if ($col == $last_col) { + $formatstring .= "%${printalign}${titlelen}s\n"; + } else { + $formatstring .= "%${printalign}${cutoff}s "; + } + } + + printf $formatstring, @titles; + + foreach my $entry (sort { $a->{$keys[0]} cmp $b->{$keys[0]} } @$data) { + printf $formatstring, map { substr(($entry->{$_} // $defaults{$_}), 0 , $cutoffs{$_}) } @keys; + } +} + +sub print_entry { + my $entry = shift; + + #TODO: handle objects/hashes as well + foreach my $item (sort keys %$entry) { + if (ref($entry->{$item}) eq 'ARRAY') { + printf "%s: [ %s ]\n", $item, join(", ", @{$entry->{$item}}); + } else { + printf "%s: %s\n", $item, $entry->{$item}; + } + } +} + +# prints the result of an API GET call returning an array +# and to have the results key of the API call defined. +sub print_api_list { + my ($props_to_print, $data, $returninfo) = @_; + + die "can only print array result" if $returninfo->{type} ne 'array'; + + my $returnprops = $returninfo->{items}->{properties}; + + my $formatopts = []; + foreach my $prop ( @$props_to_print ) { + my $propinfo = $returnprops->{$prop}; + my $colopts = { + key => $prop, + title => $propinfo->{title}, + default => $propinfo->{default}, + cutoff => $propinfo->{print_width} // $propinfo->{maxLength}, + }; + push @$formatopts, $colopts; + } + + print_text_table($formatopts, $data); +} + sub verify_api { my ($class) = @_; @@ -489,7 +595,7 @@ my $handle_cmd = sub { } # checked special commands, if def is still a hash we got an incomplete sub command - $abort->("incomplete command '$cmd_str'") if ref($def) eq 'HASH'; + $abort->("incomplete command '$cmd_str'", $args) if ref($def) eq 'HASH'; &$preparefunc() if $preparefunc; @@ -498,7 +604,10 @@ my $handle_cmd = sub { my $res = $class->cli_handler($cmd_str, $name, $cmd_args, $arg_param, $uri_param, $read_password_func, $param_mapping_func); - &$outsub($res) if $outsub; + if (defined $outsub) { + my $returninfo = $class->map_method_by_name($name)->{returns}; + $outsub->($res, $returninfo); + } }; my $handle_simple_cmd = sub { @@ -533,7 +642,10 @@ my $handle_simple_cmd = sub { my $res = $class->cli_handler($name, $name, \@ARGV, $arg_param, $uri_param, $read_password_func, $param_mapping_func); - &$outsub($res) if $outsub; + if (defined $outsub) { + my $returninfo = $class->map_method_by_name($name)->{returns}; + $outsub->($res, $returninfo); + } }; sub run_cli_handler {