Prevent fatal error if the revision has no author

Patch still applies and we have been using it in production sites for a while.

Would be great to have it merged.

Marking as RTBC.

Thanks for the work on this one. I've recently taken up maintainership of this project and am looking through the RTBC issues.

This fix looks good.

To get this in, I'll need an MR rebased against the latest 8.x-1.x code with tests added.

Thanks!

silvi.addweb → made their first commit to this issue’s fork.

Hello, I have raised MR for the same.

Don't remove the tag if tests haven't been added please.

Fixed failing MR.
Added the Required Test.

Tests are failing.

acbramley → closed merge request !59

acbramley → changed the visibility of the branch 8.x-1.x to hidden.

acbramley → changed the visibility of the branch 3088274-prevent-fatal-error to hidden.

Thanks for everyone's contributions to this, however i have taken the solution in a different direction.

The type, title and url keys on this array were completely unnecessary, template_preprocess_username doesn't use any of them. We can simply remove those lines to allow falling back to an Anonymous user display when the revision user is NULL. Also added much simpler test coverage using entity_test