πŸ‡¬πŸ‡§United Kingdom @j_edery

Account created on 5 October 2010, about 14 years ago
#

Merge Requests

Recent comments

πŸ‡¬πŸ‡§United Kingdom j_edery

I have added my fix to a new issue and created a merge request here: https://www.drupal.org/project/recently_read/issues/3407065#comment-1535... πŸ› PHP Fatal Error | Cron execution Entity queries must explicitly set whether the query should be access checked or not Needs review

πŸ‡¬πŸ‡§United Kingdom j_edery

Merge request issued. I can confirm after testing this fix resolves the issue.

πŸ‡¬πŸ‡§United Kingdom j_edery

I have issued another merge request to resolve the issue. I have tested and can confirm this resolves the PHP fatal error when running cron.

πŸ‡¬πŸ‡§United Kingdom j_edery

I have found another Drupal 10 compatibility issue with the code which results in a PHP error during testing.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

@tce in order to make use of the patch you need to install a composer plugin called "lenient"

Instructions:

setup the composer plugin:

composer require mglaman/composer-drupal-lenient

And add an exception for the vimeo_embed_field module:

composer config --merge --json extra.drupal-lenient.allowed-list '[ "vimeo_embed_field"]'

Add the patch to you'r composer files:

"drupal/vimeo_embed_field": {
"Issue #3290566: Drupal 10 compatibility fixes": "https://git.drupalcode.org/project/vimeo_embed_field/-/merge_requests/2...."
}

*Ensure you'r composer file is ssetup correctly to use patches.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

In the mean time users that need to use this module in a Drupal 10 site: https://git.drupalcode.org/project/simpler_quickedit/-/merge_requests/1....

πŸ‡¬πŸ‡§United Kingdom j_edery

Merge request created to address issues noted on upgrade_status please merge and issue a new release. Thanks

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

Patch failed I have issued a merge request with a patch that applies cleanly.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

Merge request issued with changes that resolve all the fixes required by upgrade status.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

According to upgrade status all issues are resolved after applying the following patch: https://git.drupalcode.org/project/commerce_br/-/merge_requests/1.diff

Generated from the merge request.

πŸ‡¬πŸ‡§United Kingdom j_edery

This module is currently compatible with Drupal 9 sites. This issue has been created to help provide support to get the module compatible with Drupal 10.

πŸ‡¬πŸ‡§United Kingdom j_edery

j_edery β†’ made their first commit to this issue’s fork.

πŸ‡¬πŸ‡§United Kingdom j_edery

For users that prefer the standard patch please see an updated patch that includes all the latest D10 fixes.

πŸ‡¬πŸ‡§United Kingdom j_edery

The patch above needs to remove the core key in the module info and needs amending.

Production build 0.71.5 2024