================================================================================
Focal Point, 2.0.3
Scanned on Fri, 04/26/2024 - 11:38

FILE: modules/contrib/focal_point/focal_point.info.yml

STATUS LINE MESSAGE
--------------------------------------------------------------------------------
Check manually 0 Value of core_version_requirement: ^9.3 || ^10 is not
compatible with the next major version of Drupal core. See
https://drupal.org/node/3070687.
--------------------------------------------------------------------------------

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

chandu7929 created an issue. See original summary.

chandu7929’s picture

rajan kumar@2026’s picture

Assigned: Unassigned » rajan kumar@2026
rajan kumar@2026’s picture

Status: Active » Needs review
StatusFileSize
new76.84 KB
new417 bytes

hii @chandu7929

this issue fixed "Automated Drupal 11 compatibility"

in your site composer.json file add "drush/drush": "^11 || ^12" then update composer file.

pls needs review

misterdidi made their first commit to this issue’s fork.

misterdidi’s picture

Can confirm that patch #4 works on D11. I created a fork with the patch.
Can't create a MR, though.

rajan kumar@2026’s picture

Assigned: rajan kumar@2026 » Unassigned

silvi.addweb’s picture

Status: Needs review » Reviewed & tested by the community

I've created the MR and tested the patch; it worked perfectly for me. I'm moving it to the RTBC state

avpaderno’s picture

Status: Reviewed & tested by the community » Needs review

Did anybody check the code is compatible with Drupal 11? Changing the value for core_version_requirement could not be sufficient.

skaught’s picture

Status: Needs review » Needs work

CRITICALLY --> (mid may/2024) Crop API itself is not D11 ready how is this being tested!!!!

lets start from the top down, please review: #3447620: Declare compatibility with Drupal 11 and lets get the parent into a release.

vishalkhode made their first commit to this issue’s fork.

vishalkhode’s picture

@Rajeshreeputra: The PHPUnit, PHPStan and PHPCS issues are fixed and CI is green now. Please review MR.

Note: The PHPUnit for Next Major will be passed, after Crop module changes are merged.
Thanks.

vishalkhode’s picture

Status: Needs work » Needs review
rajeshreeputra’s picture

Changes looks good to me, will wait for #3447620: Declare compatibility with Drupal 11 to get merged and released.

naveenvalecha’s picture

Title: Automated Drupal 11 compatibility fixes for focal_point » [PP-1] Automated Drupal 11 compatibility fixes for focal_point
Status: Needs review » Postponed

Postponing based on #15

vipin.mittal18’s picture

Status: Postponed » Reviewed & tested by the community

This ticket has suffice information for Drupal 11 compatibility and doesn't required any more info, its just wait for the release of CROP API module so setting up correct status.

vishalkhode’s picture

@Rajeshreeputra Can you review, merge and release as CROP module is released.

skaught’s picture

#18 Crop module 8.x-2.4 has now been released!!!!

rajeshreeputra’s picture

Status: Reviewed & tested by the community » Fixed

Merged, release to follow shortly!

rajeshreeputra’s picture

Released with 2.1.1 version.

naveenvalecha’s picture

Title: [PP-1] Automated Drupal 11 compatibility fixes for focal_point » Automated Drupal 11 compatibility fixes for focal_point

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.