IBX-11038: Content rendering in BO fails after deleting user that created the content

[Sztig]

🎫 Issue	IBX-11059

Description:

This is a regression coming from #1697
author in VersionInfo must be nullable so we can still display content of some deleted user.
I have also reverted in AuthorsTab retrieving ownerId to be directly accessed from contentInfo instead of quering the owner as it might be no longer in the database.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Sztig]

Since playing around initializing the lazy proxy didn't quite work out I decided to use try catch for the problematic user

[Sztig]

I have reverted the fix to the pre regression state, I didn't want to go with option 2 as it wouldn't work without try catch since getOwner method returns proxy that is not a null even if the fetched user doesn't exist anymore, so we would have to fetch the ownerId of this object and handle the exception anyway

[konradoboza]

I decided to go for proxy resolving to avoid silencing null value for uninitialized proxy when non-existent user is accessed. This however needs to be deeper investigated concerning owner property coming from core VO.

[barbaragr]

When I delete the editor and then go to the content tree, the error message box is displayed. Console shows: Failed to load resource: the server responded with a status of 404 (Not Found).

[konradoboza]

Well, it doesn't happen on my end. Is there any specific subtree you visit? Besides, wouldn't this be caught on the Behat tests' level?

[barbaragr]

Well, it doesn't happen on my end. Is there any specific subtree you visit? Besides, wouldn't this be caught on the Behat tests' level?

Tested on fresh and clean installation of 5.0.x-dev.

Steps to reproduce:

1. As admin create new Editor user.
2. Log in the BO as the editor.
3. Add a folder content.
4. Logout and login as admin.
5. Go to Users/Editors
6. Remove the Editor user.
7. Go to the Content/Content structure

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[konradoboza]

Thank you, a long list of content objects on my instance kinda made me overconfident. I hope we are good now but please re-test @barbaragr. 😄

[barbaragr]

Approved ✅