Cannot see media settings page after node save [#2878763]

Problem/Motivation

I have Media 2.2 using the CKEditor CDN with the required plugins for media_ckeditor added locally.

I can insert an image into my body content and save it all okay - the image is rendered and alignment options respected.

When I come to edit the node (revisions enabled and Workbench moderation on) if I right click the image and try to select 'Media settings' I am directed to a 404 screen which the url /media/undefined/format-form cannot be found. Double clicking gives me the image options but I cannot get back to the Media settings window. Clicking the image then re-clicking the media button goes to 404 page.

Comment	File	Size	Author
#3	media-whitelist_regression_fields_undefined-2878763-3.patch	305 bytes	joseph.olstad
#3	7.x-2.x: PHP 5.6 & MySQL 5.5, D7 55 pass

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

16 May 2017 at 09:52

Andy_D created an issue. See original summary.

Comment #2

Andy_D CreditAttribution: Andy_D at Investis Digital commented 16 May 2017 at 10:37

Exception faults are

JS Error appears on line 23:

if (jQuery(data.node).is('[data-media-element]')) {

DOMException: Failed to execute 'querySelectorAll' on 'Element': '*, :x'

Console error is:
GET (sitename)/media/undefined/format-form?render=media-popup&fields=undefined 404 (Not Found)

(anonymous) @ VM546 jquery.min.js?v=1.10.2:5
domManip @ VM546 jquery.min.js?v=1.10.2:5
append @ VM546 jquery.min.js?v=1.10.2:5
x.fn.(anonymous function) @ VM546 jquery.min.js?v=1.10.2:5
_create @ jquery.ui.dialog.min.js:4
(anonymous) @ jquery.ui.widget.min.js:4
_createWidget @ jquery.ui.widget.min.js:4
e.(anonymous function).(anonymous function) @ jquery.ui.widget.min.js:4
(anonymous) @ jquery.ui.widget.min.js:4
each @ VM546 jquery.min.js?v=1.10.2:4
each @ VM546 jquery.min.js?v=1.10.2:4
e.fn.(anonymous function) @ jquery.ui.widget.min.js:4
Drupal.media.popups.mediaStyleSelector @ media.popups.js?v=7.x-2.2:233
invoke @ library.js?oq1h6y:32
exec @ /sites/all/modules/contrib/media_ckeditor/js/plugins/media/plugin.js?t=F969:59
exec @ ckeditor.js:172
execCommand @ ckeditor.js:235
exec @ /sites/all/modules/contrib/media_ckeditor/js/plugins/media/plugin.js?t=F969:67
exec @ ckeditor.js:172
execCommand @ ckeditor.js:235
onClick @ ckeditor.js:678
(anonymous) @ ckeditor.js:28
(anonymous) @ ckeditor.js:683
(anonymous) @ ckeditor.js:29
callFunction @ ckeditor.js:29
onclick @ VM376 edit:1

Comment #3

joseph.olstad

French

CreditAttribution: joseph.olstad commented 16 May 2017 at 17:50

Project:

Media CKEditor

» D7 Media

File	Size
media-whitelist_regression_fields_undefined-2878763-3.patch	305 bytes
7.x-2.x: PHP 5.6 & MySQL 5.5, D7 55 pass

moving this to 'media' queue

clues in bold:
(sitename)/media/undefined/format-form?render=media-popup&fields=undefined

attached is a patch for the media module

Comment #4

joseph.olstad

French

CreditAttribution: joseph.olstad commented 16 May 2017 at 17:54

Status:

Active

» Needs review

Comment #5

joseph.olstad

French

CreditAttribution: joseph.olstad commented 16 May 2017 at 18:20

Please test the patch out and get back to us. (apply it to the media module)
either way, let us know if this helps.

Comment #6

joseph.olstad

French

CreditAttribution: joseph.olstad commented 18 May 2017 at 02:07

@Andy_D , did the attached patch help you?

Comment #7

Andy_D CreditAttribution: Andy_D at Investis Digital commented 18 May 2017 at 08:26

@joseph.olstad - thanks for responding so quickly!

I reviewed my setup and found out that the CDN enabled version of CKEditor does not reference locally installed plugins. I therefore downloaded the full CKEditor package, added the suggested plugins and it all started working. Probably a good idea to have this patch anyway in case someone else gets in a similar pickle!

Thanks again.