From e4ad6f82366ef9f90d605fa140f3f177b5daeda5 Mon Sep 17 00:00:00 2001
From: qgarnier <garnier.quentin@gmail.com>
Date: Fri, 14 Apr 2023 11:46:58 +0200
Subject: [PATCH] (plugin) apps::backup::commvault::commserve::restapi - mode
 jobs change limit (#4351)

---
 src/apps/backup/commvault/commserve/restapi/custom/api.pm | 4 +++-
 src/apps/backup/commvault/commserve/restapi/mode/jobs.pm  | 4 ++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/apps/backup/commvault/commserve/restapi/custom/api.pm b/src/apps/backup/commvault/commserve/restapi/custom/api.pm
index 6702b84ff..127934ee4 100644
--- a/src/apps/backup/commvault/commserve/restapi/custom/api.pm
+++ b/src/apps/backup/commvault/commserve/restapi/custom/api.pm
@@ -236,6 +236,7 @@ sub request_internal {
     my $content = $self->{http}->request(
         url_path => $self->{url_path} . $options{endpoint},
         get_param => $options{get_param},
+        header => $options{header},
         warning_status => '',
         unknown_status => '',
         critical_status => ''
@@ -337,7 +338,8 @@ sub request_jobs {
 
     my $response = $self->request_internal(
         endpoint => $options{endpoint},
-        get_param => ['completedJobLookupTime=' . $lookup_time]
+        get_param => ['completedJobLookupTime=' . $lookup_time],
+        header => ['limit: 10000']
     );
 
     $self->create_cache_file(type => 'jobs', response => $response)
diff --git a/src/apps/backup/commvault/commserve/restapi/mode/jobs.pm b/src/apps/backup/commvault/commserve/restapi/mode/jobs.pm
index 706540558..2b1b7cfde 100644
--- a/src/apps/backup/commvault/commserve/restapi/mode/jobs.pm
+++ b/src/apps/backup/commvault/commserve/restapi/mode/jobs.pm
@@ -199,9 +199,9 @@ sub manage_selection {
             $self->{output}->output_add(long_msg => "skipping job '" . $policy_name . "/" . $job->{jobId} . "': no matching filter type.", debug => 1);
             next;
         }
-        if (defined($job->{clientGroups}) && defined($self->{option_results}->{filter_client_name}) && $self->{option_results}->{filter_client_name} ne '') {
+        if (defined($self->{option_results}->{filter_client_group}) && $self->{option_results}->{filter_client_group} ne '' && defined($job->{clientGroups}) && ref($job->{clientGroups}) eq 'ARRAY') {
             my $matched = 0;
-            foreach (@$job->{clientGroups}) {
+            foreach (@{$job->{clientGroups}}) {
                 if ($_->{clientGroupName} =~ /$self->{option_results}->{filter_client_group}/) {
                     $matched = 1;
                     last;