Description of problem: It's better to return an error on empty task reference. Returning nil without any error means we need to check the returned task for nil every time and it also brings a bit of confusion to code. Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
Please help on steps to reproduce
More like a code debt / tech debt , cannot validate functional change . Hence moving to VERIFIED.
Alex provided info --these changes makes controller code safer
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2020:2409