The WYSIWYG integration doesn't always pick up all of the media items, in particular if there are multiple items embedded in a single field.

CommentFileSizeAuthor
#5 media_reference_revisions-n3004640-5.patch9.75 KBdamienmckenna
#2 media_reference_revisions-n3004640-2.patch8.59 KBdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes
StatusFileSize
new media_reference_revisions-n3004640-2.patch8.59 KB

Expanded test coverage.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: media_reference_revisions-n3004640-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Needs review
StatusFileSize
new media_reference_revisions-n3004640-5.patch9.75 KB

Fixed the bug. Woot.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: WYSIWYG integration doesn't always work » WYSIWYG integration breaks when multiple items present

  • DamienMcKenna committed affe433 on 8.x-1.x 
    Issue #3004640 by DamienMcKenna: WYSIWYG integration breaks when...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

Committed.

This might have been affecting reference fields too..

Status: Fixed » Closed (fixed)

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