Automated Drupal 10 compatibility fixes

I noticed this patch does not update the info.yml file so Upgrade Status still reports Transaction as Incompatible after applying the patch.

I created a merge request for #3299343-3: Automated Drupal 10 compatibility fixes → and the necessary updates to the .info.yml file.

Here's a patch which makes the module compatible with D10

RTBC +1 for #17

Would be really great to get a D10 compatible version.

Hi Folks!

I've added a few changes to the MR regarding the access checks. It might be ok to use ->accessCheck() in places where we show transactions in the UI, e.g. in the list builders. However, we can't rely on the currently logged in user having access to the transactions in places which are critical for the functionality of the module and there we should explicitly disable access checking with ->accessCheck(FALSE).

Transactions can be created in cron or via a drush command and in these scenarios there is no current_user, i.e. they run as anonymous. The anonymous user doesn't need to (and really shouldn't) have access to see the transactions, so the calls to critical methods like getLastExecutedTransaction can return wrong values.

Agree to #22

What else is needed for a D10 version?

Davituri → changed the visibility of the branch transaction-3299343-3299343-automated-drupal-10 to hidden.

Davituri → changed the visibility of the branch transaction-3299343-3299343-automated-drupal-10 to active.