Repeatable: Always

Steps to repeat:
1. Download and enable Media and File entity
2. In admin/config/regional/entity_translation set the file entity translatable
3. Add an image field to a content type and use the "Media file selector" widget
4. Create a node of this content type and add an image.
5. Modify the created node and edit the attached image

Expected Results:
You can edit the attache image in a dialog

Actual Results:
A dialog with "page not found" error is showed

CommentFileSizeAuthor
#1 et-edit_tabs.patch3.34 KBpeximo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

peximo’s picture

Status: Active » Needs review
FileSize
3.34 KB

The problem is that ET add a menu path to show the language tabs; this menu path conflicts with that defined by media to show the edit dialog.
Attached patch adds a property to not generate the tab and should fix the problem.

plach’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me. Let's test this for a while before commit.

plach’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed, thanks.

Status: Fixed » Closed (fixed)

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

Marty2081’s picture

Status: Closed (fixed) » Needs work

I still have the issue that is described in the summary. I've updated the Media and File Entity modules to the latest version and installed the development version of entity_translation (which includes the changed from the patch provided in this issue). When I disable the entity_translation module the issue is gone, so entity_translation is definitely the cause.

  • Commit 248cef7 on 7.x-1.x, revisions authored by peximo, committed by plach:
    Issue #2130091 by peximo: Media edit dialog integration bug.
    

  • Commit 248cef7 on 7.x-1.x, revisions, workbench authored by peximo, committed by plach:
    Issue #2130091 by peximo: Media edit dialog integration bug.