implement call_method
authorDietmar Maurer <dietmar@proxmox.com>
Thu, 14 Jun 2018 06:16:31 +0000 (08:16 +0200)
committerDietmar Maurer <dietmar@proxmox.com>
Thu, 14 Jun 2018 07:16:02 +0000 (09:16 +0200)
pveclient

index a169066..a7e41ef 100755 (executable)
--- a/pveclient
+++ b/pveclient
@@ -12,6 +12,7 @@ use PVE::APIClient::CLIHandler;
 
 use PVE::APIClient::LWP;
 use PVE::APIClient::Helpers;
+use PVE::APIClient::Config;
 use PVE::APIClient::Commands::config;
 use PVE::APIClient::Commands::remote;
 use PVE::APIClient::Commands::list;
@@ -21,14 +22,22 @@ use PVE::APIClient::Commands::GuestStatus;
 use JSON;
 
 sub call_method {
-    my ($remote, $path, $method, $params) = @_;
+    my ($remote, $path, $method, $param) = @_;
 
     die "missing API path\n" if !defined($path);
 
+    my $config = PVE::APIClient::Config->load();
+
+    # test if api path exists
     my $info = PVE::APIClient::Helpers::lookup_api_method($path, $method);
-    print Dumper($params);
 
-    die "implement me";
+    my $conn = PVE::APIClient::Config->remote_conn($config, $remote);
+
+    my $res = $conn->call($method, "api2/json/$path", $param);
+    die "undefined result" if !defined($res);
+    die "undefined result data" if !exists($res->{data});
+
+    return $res->{data};
 }
 
 use base qw(PVE::APIClient::CLIHandler);
@@ -77,6 +86,15 @@ $path_properties->{api_path} = {
     },
 };
 
+my $format_result = sub {
+    my ($data, $format) = @_;
+
+    return if $path_returns->{type} eq 'null';
+
+    # TODO: implement different output formats ($format)
+    print to_json($data, {utf8 => 1, allow_nonref => 1, canonical => 1, pretty => 1 });
+};
+
 __PACKAGE__->register_method ({
     name => 'pveclient_get',
     path => 'pveclient_get',
@@ -165,11 +183,11 @@ our $cmddef = {
     stop => [ 'PVE::APIClient::Commands::GuestStatus', 'stop', ['remote', 'vmid']],
 
     api => {
-       get => [ __PACKAGE__, 'pveclient_get', ['remote', 'api_path']],
-       set => [ __PACKAGE__, 'pveclient_set', ['remote', 'api_path']],
-       create => [ __PACKAGE__, 'pveclient_create', ['remote', 'api_path']],
-       delete => [ __PACKAGE__, 'pveclient_delete', ['remote', 'api_path']],
-    }
+       get => [ __PACKAGE__, 'pveclient_get', ['remote', 'api_path'], {}, $format_result ],
+       set => [ __PACKAGE__, 'pveclient_set', ['remote', 'api_path'], {}, $format_result ],
+       create => [ __PACKAGE__, 'pveclient_create', ['remote', 'api_path'], {}, $format_result ],
+       delete => [ __PACKAGE__, 'pveclient_delete', ['remote', 'api_path'], {}, $format_result ],
+    },
 };