Contrib.social

Add support to create new taxonomy terms (enitties) during bulk upload

Open on Drupal.org →
Created on 9 August 2022, over 2 years ago
Updated 10 February 2023, about 2 years ago

Problem/Motivation

We've added a taxonomy term field to the bulk upload so users can tag the uploaded files with theme. This works fine when users select a taxonomy term that already exists, but it leads to problems when users want to create a new tag.

This results in only the first file being uplaoded and a lot of errors in watchdog (Integrity constraint violation: 1062 Duplicate entry).

Steps to reproduce

Add a taxonomy field to the bulk upload form and create a new term when uploading multiple files.

Proposed resolution

I've added a patch with the solution I've used to fix this issue.

The solution I came up with was to loop through all the fields during the submit function of the form itself. A refrence field value can have 2 options. It either contains a target_id to an existing entity or a new entity object that hasn't been stored yet.
If we encounter the later we save the entity and replace the form_state values with a target_id to the just created entity.

✨ Feature request
Status

Active

Version

3.0

Component

Code

Created by

🇳🇱Netherlands Da_Cloud

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.

  • First commit to issue fork.
  • @codebymikey opened merge request.
  • Status changed to Needs review about 2 years ago
  • Comment about 2 years ago →
    🇺🇸United States Kristen Pol Santa Cruz, CA, USA

    @Da_Cloud Thanks. After adding a patch, please move to "Needs review" so people know there is something to code review and manually test.

  • Comment over 1 year ago →
    🇦🇹Austria daniel.pernold

    Reroll patch for 3.0.2.

  • Comment 10 months ago →
    🇩🇪Germany stefan.korn Jossgrund

    We encountered the same issue and the patch from #5 fixes it. Thanks.

    I change to "Bug report". And imho this is kind of a serious bug. If Media Bulk Upload does not fully support taxonomy entities this should be mentioned somewhere.

  • Assigned to Kristen Pol
  • Status changed to RTBC about 1 month ago
  • Comment about 1 month ago →
    🇺🇸United States Kristen Pol Santa Cruz, CA, USA

    Sounds like @stefan.korn should have moved this to RTBC based on the comment.

    Assigning to me to review for next release.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024