Support text_long data type

Created on 24 August 2021, over 3 years ago
Updated 8 August 2023, over 1 year ago

Problem/Motivation

Thanks for this great module for migration. In our case we also use entity_embed for media in the field with text_long type.

Steps to reproduce

- Create a field with text_long type to the content type "page"
- Insert a media via entity_embed + entity_browser and save the page
- Install the module convert_entity_media_embed
- As you can see there is no queue item has been created for this page as text_long field is not supported

Proposed resolution

- support text_long

Remaining tasks

Fix in the PR

User interface changes

N/A

API changes

N/A

Data model changes

N/A

✨ Feature request
Status

Fixed

Version

1.0

Component

Code

Created by

πŸ‡¦πŸ‡ΊAustralia eric.chenchao

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States DamienMcKenna NH, USA

    Seems straight forward.

  • πŸ‡ΊπŸ‡ΈUnited States pthurmond Overland Park, KS

    Ok, so I ran this and kept running into an error that said "'' not found" (that is two single-quotes). So I created a new version of this to address this with a few additional checks. I tested these checks via the Devel PHP execute module. The error is gone and it populates the arrays it needs to build this queue now (it didn't do that before). Obviously my patch includes the changes from the original patch.

  • πŸ‡ΊπŸ‡ΈUnited States pthurmond Overland Park, KS

    It should be noted that this requires cron jobs to be run to complete. Locally I don't have cron tasks scheduled to run. So I had to do so manually. That should probably be pointed out in the README. Along with the fact that you have to go to "Admin > Reports > Status Report" and scroll down to the "Checked" section to find the status of this.

    With that in mind, I did just that with this next patch. Because documentation is good.

  • πŸ‡³πŸ‡±Netherlands eelkeblok Netherlands πŸ‡³πŸ‡±

    Thanks all, committed. @patrick.thrumond, thanks for your suggestions. You'll find the issue with the empty bundle key has been resolved in the latest alpha, there was a different issue for that. I've chosen not to adopt your changes to the README (yet) because this is not what the issue is about and it's just so much easier to merge a merge request than to locally apply a patch and commit that. I'll create a new issue and have a look at your suggestions there.

  • Status changed to Fixed over 1 year ago
  • πŸ‡³πŸ‡±Netherlands eelkeblok Netherlands πŸ‡³πŸ‡±
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024