- 🇺🇸United States DamienMcKenna NH, USA
One problem here is that there's an assumption on which field name should be used; should it be configurable or something? The main differences between #13 and #19 are in the image field name - #13 uses field_image whereas #19 uses field_media_image.
FWIW we've been using #13 on a production site for a few years.
- 🇺🇦Ukraine _shy Ukraine, Lutsk 🇺🇦
Please, reroll the patch to the latest branch version.
- Status changed to Needs work
about 1 year ago 7:36pm 25 November 2023 - 🇺🇸United States wstocker
Patch re-rolled and updated with Drupal core 9.5.x.
- 🇮🇳India sakthi_dev
The patch provided in #24 not patching the files correctly instead it's creating a patch file inside the directory. As per comment #22 added Needs Reroll to the tags.
- 🇺🇸United States wstocker
@sakthi_dev Whoops apologies wrong file. Here is the updated one.
- Status changed to Needs review
10 months ago 3:24pm 9 February 2024 - 🇺🇸United States wstocker
Patch 26 applied, but to Damiens' point there were PHP errors surrounding the new field name in #20.
We had been using patch #13 for several years which has field_image instead of field_media_image.
I think making that configurable is a good idea, but for now, I re-rolled and created patch #28 using patch #13 to start and the errors disappeared.
Patch applies clean to 8.x-1.x.
- 🇺🇸United States wstocker
Patch #28 is incompatible with D10 as it uses deprecated and removed file_create_url().
Refactored so it is compatible with D10 and 3.x.