restenv: remove unused variable 'ctime'

[pve-common.git] / src / PVE / RESTEnvironment.pm

diff --git a/src/PVE/RESTEnvironment.pm b/src/PVE/RESTEnvironment.pm
index 6c1ed59ae51c662f2894761e7a2cb1abf3d13428..15eef90c97fbae3dcf5a9a9cf83c82e0e222692e 100644 (file)
--- a/src/PVE/RESTEnvironment.pm
+++ b/src/PVE/RESTEnvironment.pm
@@ -60,8 +60,7 @@ my $log_task_result = sub {
        $msg = $task->{status};
     }
 
-    $rest_env->log_cluster_msg($pri, $user, "end task $upid $msg")
-       if $rest_env;
+    $rest_env->log_cluster_msg($pri, $user, "end task $upid $msg");
 };
 
 my $worker_reaper = sub {
@@ -210,11 +209,11 @@ sub set_user {
 }
 
 sub get_user {
-    my ($self) = @_;
+    my ($self, $noerr) = @_;
 
-    die "user name not set\n" if !$self->{user};
+    return $self->{user} if defined($self->{user}) || $noerr;
 
-    return $self->{user};
+    die "user name not set\n";
 }
 
 sub is_worker {
@@ -319,7 +318,6 @@ sub active_workers  {
        # we try to reduce the amount of data
        # list all running tasks and task and a few others
        # try to limit to 25 tasks
-       my $ctime = time();
        my $max = 25 - scalar(@$tlist);
         foreach my $task (@ta) {
            last if $max <= 0;
@@ -687,7 +685,9 @@ sub check_api2_permissions {
 sub init_request {
     my ($self, %params) = @_;
 
-    # implement in subclass
+    $self->{result_attributes} = {}
+
+    # if you nedd more, implement in subclass
 }
 
 1;