Add revision support

It works for new ECK types created after patching.

-----------------------
Test for types created before the patching.

Drupal\Core\Entity\Query\QueryException: No revision table for eck_2_type, invalid query. in Drupal\Core\Entity\Query\Sql\Query->prepare() (line 99 of /var/www/html/local6site/web/core/lib/Drupal/Core/Entity/Query/Sql/Query.php).

Testing with workflow

If your existing eck type has content moderation workflow;

Drupal\Core\Database\DatabaseExceptionWrapper: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'mydatabase.dr_' doesn't exist: SELECT "base"."id" AS "id", "base"."type" AS "type", "base"."uuid" AS "uuid", "base"."langcode" AS "langcode" FROM "dr_eck_1_type" "base" INNER JOIN "dr_" "revision" ON "revision"."id" = "base"."id" AND "revision"."" IN (:revisionIds__0); Array ( [:revisionIds__0] => ) in Drupal\Core\Entity\Sql\SqlContentEntityStorage->doLoadMultipleRevisionsFieldItems() (line 624 of /var/www/html/local6site/web/core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php).

Tested with drupal 10.1.2 & PHP 8.1

There's been a lot of work on this over at https://www.drupal.org/project/eck/issues/2788507 ✨ Add revision support Needs review . Mind if we combine them?

This one shouldn't have been closed as duplicate for 2 reasons.
1. The other issue is for version 8.x-1.x-dev. This one is for version 2.0.0
2. The other issue patches didn't work for this version because of Drupal 10 changes and this is for version 2.0.0.

So this is unrelated to the other issue.

Sorry, I never got an email there were responses.

Oops! Good catch, thanks.

Created a MR based on the patch, will review there.

I left some comments in the MR. Additionally, a generic revision UI was added in Drupal 10.1 ( ✨ Implement a generic revision UI Fixed ), so we should probably rewrite some stuff to make use of that.

The changes from #2788507-116 ✨ Add revision support Needs review should still be added here as well.

Attaching a patch file that works for eck 2.0.0 on my local.

Hello,

I have created a patch based on changes available on #2788507-116 ✨ Add revision support Needs review for the 2.0.0.

Also the changes from #2788507-116 ✨ Add revision support Needs review do not work for existing eck entities. so I have modified the update_hook which will make the existing entities revisionable.

Note - This patch will make all the existing eck entities revisionable.

In our project, not all our custom entity types requires the revision enabled, however the patch #15 force all entity types revisionable, we can't enable turn it off, so I updated the patch to handle the revisionable entity types only.

dieterholvoet → credited audriusb → .

dieterholvoet → credited brianfisher → .

dieterholvoet → credited citlacom → .

dieterholvoet → credited damienmckenna → .

dieterholvoet → credited dawehner → .

dieterholvoet → credited legolasbo → .

dieterholvoet → credited MarieKirya → .

dieterholvoet → credited slydevil → .

dieterholvoet → credited T-lo → .

dieterholvoet → credited tetranz → .

Crediting the people who worked on ✨ Add revision support Needs review .

Please don't post new patches, work is being done in the MR. My comments there haven't been addressed yet, so setting back to Needs work.