I want to make some changes to the params used by the Media Browser using the hook, but nothing I do there has any effect.

It looks like this is a one liner: replace drupal_get_query_parameters() with media_get_browser_params in media.media.inc

CommentFileSizeAuthor
#1 use-api-function-for-retrieving-browser-params-1942990-1.patch1.22 KBDevin Carlson
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Issue summary: View changes
Status: Active » Needs review
FileSize
use-api-function-for-retrieving-browser-params-1942990-1.patch1.22 KB

A patch to use media_get_browser_params().

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs review » Fixed

Tested #1 and committed it to Media 7.x-2.x.

morenstrat’s picture

morenstrat
he/him
Primary language German
Location Bochum, Germany
CreditAttribution: morenstrat commented
Status: Fixed » Needs work

Paging does not work for me with this patch. I get a

PHP Fatal error: Call to undefined function media_get_browser_params()

Adding the following line right before calling media_get_browser_params() fixes the problem for me:
module_load_include('inc', 'media', 'includes/media.browser');

Devin Carlson’s picture

Devin Carlson CreditAttribution: Devin Carlson commented
Status: Needs work » Postponed (maintainer needs more info)

@dunix, have you done a git checkout of the latest commit hash?

That issue should be fixed with #2209537: media browser library broken since latest commits.

morenstrat’s picture

morenstrat
he/him
Primary language German
Location Bochum, Germany
CreditAttribution: morenstrat commented
Status: Postponed (maintainer needs more info) » Fixed

No, sorry, I applied the patch from this issue manually.

Status: Fixed » Closed (fixed)

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