Immediately publishing a node fails when no content type have content moderation applied

Updated patch a bit, the patch does the job, RTBC

Thanks for reporting this. Can you fix it for coding standards/linting, the test result of "PHPLint Failed" needs to be sorted before we can consider the change.

I see, I used some features that are available since php 7.4, but this module should still support php 7.3 based on tests, ok, I will change it

@jonathan1055,

Let's see if it works now for php 7.3 test

Rebased for 2.x since 8.x-1.x is no longer supported.

Going to merge as it does solve the issue. 2.x branch is currently broken so that's not showing correct info.

The fix added

$entity = $event->getNode();

but immediately after the new conditional check, we already had

$entity = $event->getEntity();

Is there a reason why we have to use getNode()? It seems wrong (a) to have both calls, and (b) it should be content-type generic, not just for nodes.

Putting this back to 'active' to that it does not get 'closed (fixed)' after two weeks.