Could we add support for the Inline Entity Form module? It's non-invasive and requires minimum integration.
https://www.drupal.org/project/inline_entity_form

CommentFileSizeAuthor
#3 2612602-3-asset-inline_entity_form_integration.patch2.17 KBgrndlvl
#2 2612602-2-asset-inline_entity_form_integration.patch2.23 KBgrndlvl
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

grndlvl created an issue. See original summary.

grndlvl’s picture

grndlvl CreditAttribution: grndlvl commented
Status: Active » Needs review
FileSize
2612602-2-asset-inline_entity_form_integration.patch2.23 KB
grndlvl’s picture

grndlvl CreditAttribution: grndlvl commented
FileSize
2612602-3-asset-inline_entity_form_integration.patch2.17 KB

Use asset_base_form() instead of the asset_wysiwysg_form().

Jody Lynn’s picture

Jody Lynn
she/her
Primary language English
CreditAttribution: Jody Lynn at Zivtech commented
Status: Needs review » Reviewed & tested by the community

Love it! Adding assets as entityreferences using inline entity form is great.

This patch works great- reviewed and tested.

We're running this patch in our install profile, Bear: https://www.drupal.org/commitlog/commit/46644/13b54672eab86338e4b6751913...

IRuslan’s picture

IRuslan CreditAttribution: IRuslan as a volunteer and at DrupalJedi commented
Status: Reviewed & tested by the community » Fixed

Pushed to dev, thanks for good job.

  • IRuslan committed 016fa1d on 7.x-1.x authored by grndlvl 
    Issue #2612602 by grndlvl: Add support for Inline Entity Form

Status: Fixed » Closed (fixed)

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