Closed (fixed)
Project:
D7 Media
Version:
7.x-2.x-dev
Component:
Media Browser
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Mar 2013 at 20:48 UTC
Updated:
25 May 2017 at 05:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
Anonymous (not verified) commentedAny traction on this? I'm having the same problem.
Comment #2
acrosmanI just retired this with the Dev release from 7/17/2013, and while I now see references to the page numbers in the links, it continues to load just the first page.
If I load the link from the pager directly in its own window, I get the right page, but the pager there does not reload correctly either.
Link for page 3 in pager:
http://example.org/media/browser/page/4/0?filename=&type=All&sort_by=timestamp&sort_order=DESC&types[image]=image&types[video]=video&enabledPlugins[upload]=upload&enabledPlugins[media_default--media_browser_1]=media_default--media_browser_1&enabledPlugins[media_internet]=media_internet&schemes[kaltura]=kaltura&schemes[vimeo]=vimeo&schemes[youtube]=youtube&schemes[public]=public&schemes[private]=private&file_directory=signature_media&file_extensions=jpg%20jpeg%20gif%20png&uri_scheme=public&plugins=undefined&render=media-popup&activePlugins=&max_filesize=Comment #3
gmclelland commentedI'm not seeing this behavior on the latest media and file_entity - 2.x-devs. I tested on two different sites. Are you getting any javascript errors in your console?
Does this happen on a fresh install? Did you modify the media browser view?
Comment #4
damienmckennaI've experiencing this with the latest -dev for both file_entity and media, will see what I can do to dig into it.
Comment #5
damienmckennaMarked #2079155: Unable to select media:youtube video if there's no URL alias as a duplicate.
Comment #6
damienmckennaA part of this problem also happens when filtering the list - it updates the display but does not update Drupal.settings.media.files, so any files that were not part of the original list are no longer selectable, it just gives the olde "You have not selected anything" error.
Comment #7
damienmckennaFYI previewing the view itself via the Views UI, and it works just fine. Go figure.
So, on with the debugging.
Comment #8
damienmckennaWhen loading a page via the media browser page (/media/browser?render=media-popup&types%5B%5D=image&types%5B%5D=video&activePlugins=&enabledPlugins%5B%5D=upload&enabledPlugins%5B%5D=media_default--media_browser_1&enabledPlugins%5B%5D=media_internet&file_directory=&id=media_wysiwyg&plugins=undefined), the XHR call returns the following:
https://drupal.org/files/media-n1956620-8-browser.json_.txt
When loading it from the Views UI it returns the following:
https://drupal.org/files/media-n1956620-8-viewsui.json_.txt
The thick plotten.
Comment #9
larowlanThis fixes the data returned, you can't have two pagers on the same page with the same pager id.
But it doesn't fix the tab issue, ie the wrong tab is loaded after the ajax finishes loading (but the data is right once you switch tabs)
Comment #10
larowlanThis fixes the js too, there are some .error.clientside-error elements on each tab, which means the tabs just keep borking.
The new code filters the .clientside-error ones out first.
Comment #11
damienmckennaThere was a typo, you had errorTabId instead of errorTabID. Also I reformatted those few lines of code to match the formatting used elsewhere in the file.
FYI this still doesn't solve the problem for my site.
Comment #12
aaron commentedFWIW, I do not experience any of the issues mentioned here. The patch looks innocuous enough.
Comment #13
acrosmanI applied the patch in #11 to the dev from August (current), and it also does not resolve the problem for me.
Comment #14
damienmckennaRelated: #2104493: Media Browser only shows 1 page of results (Media v1)
Comment #15
samhassell commentedPatch no longer applies, and the lines it effect seem to have disappeared.
Comment #16
samhassell commentedThis code was reworked in #1802026: Ajax callbacks in the media overlay cause the active tab to switch tab unexpectedly
Comment #17
grom358 commentedFix javascript errors from @larowlan patch. Patched against alpha3.
Comment #18
larowlanComment #19
larowlanThis is no longer an issue.