For a module integration I am working on I need to modify the browserUrl query provided by media_browser_js(). This patch adds a drupal_alter().

CommentFileSizeAuthor
#1 media-browser-js-settings-alter-1984732-1.patch338 bytesqueenvictoria
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

queenvictoria’s picture

queenvictoria CreditAttribution: queenvictoria commented
FileSize
media-browser-js-settings-alter-1984732-1.patch338 bytes

Patch attached

queenvictoria’s picture

queenvictoria CreditAttribution: queenvictoria commented
Status: Active » Needs review

Should be a simple review.

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented
Issue summary: View changes
Status: Needs review » Needs work
Issue tags: +Needs reroll

The 6 year old patch in #1 does not apply to the latest media 7.x-2.x-dev and if still relevant needs to be rerolled

Checking patch includes/media.browser.inc...
error: while searching for:
      )
    ),
  );

  $js = array(
    'library' => array(

error: patch failed: includes/media.browser.inc:259
error: includes/media.browser.inc: patch does not apply
joseph.olstad’s picture

joseph.olstad
Primary language French
CreditAttribution: joseph.olstad commented
Status: Needs work » Closed (outdated)