Can Media Library Media Modify be used with Image Widget Crop?

Just updating this patch...

This has code seemingly ready to be committed so it is not only a support request anymore!

DrDam did you miss adding a new patch in your last comment five months ago? And how does this issue interplay with your new module https://www.drupal.org/project/media_contextual_crop → replacing multi_crop that you mentioned in comment 8?

Thanks!

Hi,

DrDam did you miss adding a new patch in your last comment five months ago?

There have been no changes since #7.
#13 are only an issue-meta update

How does this issue interplay with your new modules replacing multi_crop that you mentioned in comment 8 ?

In the same way multi_crop, I just implement this hook_alter in the "media_contextual_crop_field_formatter" module to add crop settings in context.
As soon as the patch are merge, i'll clean the composer.json of my module

@mlcn https://git.drupalcode.org/project/media_contextual_crop_field_formatter...

"drupal/media_library_media_modify": {
  "#3270150 Add alteration": "https://www.drupal.org/files/issues/2022-06-08/manage_crops_3270150-7.patch"
}

An alternate approach here would be for the module that needs the patch to decorate the media_library_media_modify service and make the changes it needs to \Drupal\media_library_media_modify\EntityReferenceOverrideService::getOverriddenValues - what speaks against this is the fact that \Drupal\media_library_media_modify\EntityReferenceOverrideService does not have an interface. Therefore I think that adding an alter seems okay and I guess it possible that other modules might need it too.

It would be wonderful if an MR was created and if we added a media_library_media_modify.api.php file was created that describes the hook and why it should be used.

I have made the MR on 1.x branch

drdam → changed the visibility of the branch 3270150- to hidden.

drdam → changed the visibility of the branch 3270150-can-media-library-2x to hidden.

I have create a second merge-request for 2.x branch