Merge pull request #214 from hadfl/issue

citrus-it · web-flow · commit 68974ce733c6 · 2024-04-24T12:05:50.000+01:00
issue: only limit issues if maxissue is set
diff --git a/lib/Fenix/Model/Handler/Issue.pm b/lib/Fenix/Model/Handler/Issue.pm
@@ -48,7 +48,8 @@ sub process_p($self, $chan, $from, $msg, $mentioned = 0) {
         return Mojo::Promise->resolve([]) if !@issues;
 
         # limit the maximum number of issues fenix handles for one request
-        @issues = splice @issues, 0, $self->config->{maxissue};
+        @issues = splice @issues, 0, $self->config->{maxissue}
+            if $self->config->{maxissue};
 
         return $self->handler->{$hd}->process_p(\@issues, $opts);
     }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,8 @@ sub process_p($self, $chan, $from, $msg, $mentioned = 0) {`
`48`	`48`	`return Mojo::Promise->resolve([]) if !@issues;`
`49`	`49`
`50`	`50`	`# limit the maximum number of issues fenix handles for one request`
`51`		`- @issues = splice @issues, 0, $self->config->{maxissue};`
	`51`	`+ @issues = splice @issues, 0, $self->config->{maxissue}`
	`52`	`+ if $self->config->{maxissue};`
`52`	`53`
`53`	`54`	`return $self->handler->{$hd}->process_p(\@issues, $opts);`
`54`	`55`	`}`