Ajax error Page Not Found on opening media browser modal popup [#1835244]

We are experiencing some strange behavior after updating to Manual Crop version 7.x-1.4. When clicking the "Edit Media" button on the media file selector widget we would expect the Media modal popup window to appear. However we get an AJAX error mentioning the callback path could not be found.

So i started investigating why this isn't working and by accident found out that the same button was working in the English administration interface which has 'en' as a prefix, our default language is Dutch. So i compared the URI's that are used in both languages.

With the Dutch administration interface (no prefix) the edit URI is:

/media/10815/edit/nojs/media/10808/edit/nojs/node/blog_article/field_blog_image

With the English administration interface ('en' as prefix) the edit URI is:

/en/media/10815/edit/nojs/en/media/10808/edit/nojs/node/blog_article/field_blog_image

The first URI returns a page not found, the second URI is working just fine. After a bit of debugging i found out the URI without the first 'en' prefix was also working:

/media/10815/edit/nojs/en/media/10808/edit/nojs/node/blog_article/field_blog_image

I can actually replace 'en' in the above link with the prefix for any other active language and it will also work.

The URI for the edit button is modified by the 'manualcrop_media_element_process()' function where '/media/10815/edit/nojs/' is modified to the links you see above.

So this can either be an issue with the Manual Crop module or the Media module but i have no idea where to look next.

Comment	File	Size	Author
#5	manual-crop-ajax_error_page_not_found-1835244.patch	550 bytes	ericmulder1980
#4	manual-crop-ajax_error_page_not_found-1835244.patch	0 bytes	ericmulder1980

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Matthijs

Dutch

CreditAttribution: Matthijs commented 8 November 2012 at 19:20

Hi Eric,

Thanks for your report! This looks like a weird issue, my first guess would be that it's a problem with the Media module, but I should investigate it to give you a final/good answer...

Instead of spending my time on this, I would like to ask you if you mind updating to the last dev version? I've rewritten the link altering to use query variables instead of appending addition url elements, so this might fix your issue.

Could you give it a try and let me know?

Thanks!

Matthijs

Comment #2

ericmulder1980 CreditAttribution: ericmulder1980 commented 9 November 2012 at 09:05

Hi Martijn,

Thank you so much for answering so quickly.

As you've asked i have installed the latest dev version of manual crop and tried again. Unfortunately the error is still the same, the URL for the Edit media button did change however

/media/898/edit/ajax/media/898/edit/nojs%3Fmanualcrop%5Bentity_type%5D%3Dnode%26manualcrop%5Bbundle%5D%3Dpage%26manualcrop%5Bfield_name%5D%3Dfield_page_attachment

Hope this helps.

Comment #3

ericmulder1980 CreditAttribution: ericmulder1980 commented 12 November 2012 at 11:42

So i've tested some more and checked the part where the edit link is modified.

As far as i can see this is the URL you want

http://www.example.com/media/[file_id]/edit/nojs?[parameters]

With the current modifications you get a duplicate entry of the "/media/[file_id]/edit/nojs" part.

So if you change

$element['edit']['#href'] .= url('media/' . $element['fid']['#value'] . '/edit/nojs', array('query' => $data));

$element['edit']['#href'] = url('media/' . $element['fid']['#value'] . '/edit/nojs', array('query' => $data, 'absolute' => TRUE));

it should give you the URL you are looking for.

Comment #4

ericmulder1980 CreditAttribution: ericmulder1980 commented 12 November 2012 at 13:39

Status:

Active

» Needs review

File	Size
manual-crop-ajax_error_page_not_found-1835244.patch	0 bytes

Added a patch to create the situation mentioned in previous post. I hope it's the proper solution to this problem.

Comment #5

ericmulder1980 CreditAttribution: ericmulder1980 commented 12 November 2012 at 13:46

File	Size
manual-crop-ajax_error_page_not_found-1835244.patch	550 bytes

Previous patch did not upload correctly.

Comment #6

Matthijs

Dutch

CreditAttribution: Matthijs commented 12 November 2012 at 21:29

Status:

Needs review

» Fixed

Hi Eric,

Thanks a lot for your work on this! The I've tested and applied your patch, the ".=" was a typo, odd I didn't notice it before (everything worked fine here)...

Thanks again!

Matthijs