Skip to content

bugfix: without $self this is a function call, not a method call

Marek Szuba requested to merge bugfix/selfless_stats_sync into version/2.5

Created by: muffato

Description

I've found this whilst working on a fix for negative job counts. $self-> is missing here, which makes this line a function call, not a method call. The subroutine then gets $stats in $self and the other arguments (it expects $stats and $has_refresh_just_been_done) are both undefined. Because it can deal with undefined $stats and returns immediately, it's never raised an error, but this means this call was doing nothing.

Use case

I guess this only means that failures were not reported with the freshest stats, which people may not even notice.

Possible Drawbacks

None

Testing

Have you added/modified unit tests to test the changes?

No

If so, do the tests pass/fail?

N/A

Have you run the entire test suite and no regression was detected?

Yes. OK

Merge request reports