Skip to content

Fix exception when epic has no author

John Hope requested to merge fix_link_to_member_for_nil into master

What does this MR do and why?

Describe in detail what your merge request does and why.

Related to #350653 (closed)

The link_to_member method looks like it should support a null author, it even has a guard clause to return a String. But this is underneath an attempt to call #id on null so it raises an exception.

This is stopping users who deleted users from seeing their epics, as these epics now have no author. This helper is used in the rendering of things like the epics list, so it's important that it can work with no author.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports