Fixed AbstractJob::resume() assumes getStatusMessage non-nullable return value

[Xon]

Affected version

2.3.7

XF2.3 added a method JobResult::newResume() which had typed arguments and the AbstractJob::resume() assumes getStatusMessage() a stringable but the getStatusMessage method was never type hinted for a return type and I've seen it return null in a few of my add-ons and a few 3rd party add-ons.

I think the best fix would be for ?? '' to be added in the resume() method and a ?string return type hint added to getStatusMessage() method.

 

[XF Bug Bot]

Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.3.8).

Change log:

Add handling for null status message values when resuming jobs

There may be a delay before changes are rolled out to the XenForo Community.