]> git.proxmox.com Git - pve-common.git/commitdiff
improve and fix some whitespace errors
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 18 Jun 2018 10:19:08 +0000 (12:19 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 18 Jun 2018 10:34:58 +0000 (12:34 +0200)
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/PVE/CLIHandler.pm

index 1b95d9a1fd499346fb36ef8555b16bebc0908dba..2fa7933ee00574baaf860cb87fcc0f34974d7e38 100644 (file)
@@ -409,18 +409,19 @@ my $print_bash_completion = sub {
 # formatopts element order defines column order (left to right)
 sub print_text_table {
     my ($formatopts, $data) = @_;
 # formatopts element order defines column order (left to right)
 sub print_text_table {
     my ($formatopts, $data) = @_;
-    my ($formatstring, @keys, @titles, %cutoffs, %defaults, $last_col);
 
 
-    $last_col = $formatopts->[$#{$formatopts}];
+    my ($formatstring, @keys, @titles, %cutoffs, %defaults);
+    my $last_col = $formatopts->[$#{$formatopts}];
+
     foreach my $col ( @$formatopts ) {
     foreach my $col ( @$formatopts ) {
-       my ($key, $title, $cutoff, $default) = @$col{ qw(key title cutoff default)};
+       my ($key, $title, $cutoff, $default) = @$col{qw(key title cutoff default)};
        $title //= $key;
 
        push @keys, $key;
        push @titles, $title;
        $defaults{$key} = $default;
 
        $title //= $key;
 
        push @keys, $key;
        push @titles, $title;
        $defaults{$key} = $default;
 
-       #calculate maximal print width and cutoff
+       # calculate maximal print width and cutoff
        my $titlelen = length($title);
 
        my $longest = $titlelen;
        my $titlelen = length($title);
 
        my $longest = $titlelen;
@@ -442,16 +443,17 @@ sub print_text_table {
 
     printf $formatstring, @titles;
 
 
     printf $formatstring, @titles;
 
-    foreach my $entry (sort { $a->{$keys[0]} cmp $b->{$keys[0]} } @$data){
+    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;
         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) {
     #TODO: handle objects/hashes as well
     foreach my $item (sort keys %$entry) {
-       if (ref($entry->{$item}) eq 'ARRAY'){
+       if (ref($entry->{$item}) eq 'ARRAY') {
            printf "%s: [ %s ]\n", $item, join(", ", @{$entry->{$item}});
        } else {
            printf "%s: %s\n", $item, $entry->{$item};
            printf "%s: [ %s ]\n", $item, join(", ", @{$entry->{$item}});
        } else {
            printf "%s: %s\n", $item, $entry->{$item};
@@ -463,8 +465,10 @@ sub print_entry {
 # and to have the results key of the API call defined.
 sub print_api_list {
     my ($props_to_print, $data, $returninfo) = @_;
 # and to have the results key of the API call defined.
 sub print_api_list {
     my ($props_to_print, $data, $returninfo) = @_;
-    my $formatopts;
+
     my $returnprops = $returninfo->{items}->{properties};
     my $returnprops = $returninfo->{items}->{properties};
+
+    my $formatopts = [];
     foreach my $prop ( @$props_to_print ) {
        my $propinfo = $returnprops->{$prop};
        my $colopts = {
     foreach my $prop ( @$props_to_print ) {
        my $propinfo = $returnprops->{$prop};
        my $colopts = {