BUGFIX: Fix contentgraphintegrity:runviolationdetection

[mhsdesign]

The wiring for flow contentgraphintegrity:runviolationdetection was wrong and needed to be redone.

Upgrade instructions

Review instructions

Checklist

• Code follows the PSR-2 coding style
• Tests have been created, run and adjusted as needed
• The PR is created against the lowest maintained branch
• Reviewer - PR Title is brief but complete and starts with FEATURE|TASK|BUGFIX
• Reviewer - The first section explains the change briefly for change-logs
• Reviewer - Breaking Changes are marked with !!! and have upgrade-instructions

[kitsunet]

I would be fine merging this although silently different behavior due to different environments has bitten us a couple of times before, so IDK, maybe there is a way to make this query backwards compatible?

[mhsdesign]

Hmm im not that deep into json functions for maria db. I thought for a second that performance will not matter as this is just run for the behat tests but as i found out there is a cli command that is supposed to be run on large sets of data so better keep it high performing.

... we could add a switch to use php filtering if the json functions are not available but hard to test.

Id say this is just a hack and we can even drop it before final release, when we are sure what we want to support: #4337

Completely refactored ;)

[mhsdesign]

Okay this was rebased and refactored, this change now only contains minimal adjustments to rewire the command as otherwise it will crash (look at the code :D)