Add database integrity checks for ContentContainer in the spacemodule

[marc-farre]

Is your feature request related to a problem? Please describe.

If a space is deleted in the database, but for some reason, the related ContentContainer is not deleted, running a DB integrity check should delete it.

E.g: https://community.humhub.com/content/perma?id=276362

Describe the solution you'd like

Add integrity checks in the space module similar to:

• https://github.com/humhub/humhub/blob/master/protected/humhub/modules/user/Events.php#L63-L70
• https://github.com/humhub/humhub/blob/master/protected/humhub/modules/user/Events.php#L144-L151

[marc-farre]

I'm not sure about this issue, because DB integrity check seams to have fixed the issue in fact: https://community.humhub.com/comment/perma?id=48400

When exploring the code, I still think this integrity check for the space module should be added, but maybe it's already somewhere else?

[luke-]

I can't find the check straight away either.

We should add a check here and loop over all "ContentContainers" and check whether the target is still correct.

https://github.com/humhub/humhub/blob/master/protected/humhub/modules/content/Events.php#L93

@marc-farre What do you think?

[marc-farre]

Yes, I also thought about checking in the content module (this is where I searched first).
In this case, we can remove so checks in the user module.