workbench_media_menu() contains the wrong file path. It should now be includes/media.pages.inc

CommentFileSizeAuthor
#1 file_path_fix-1167838-1.patch547 bytesstevector

Comments

stevector’s picture

stevector
he/him
commented
StatusFileSize
new file_path_fix-1167838-1.patch547 bytes

This fix is so small that I will go ahead and commit it. The module is completely broken without it.

stevector’s picture

stevector
he/him
commented
Status: Active » Fixed

http://drupalcode.org/project/workbench_media.git/commit/739b431506522b2...

Status: Fixed » Closed (fixed)

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

BarisW’s picture

BarisW commented

Can you please create a 1.2 release? This commit is still missing in the latest 1.x branch.

BarisW’s picture

BarisW commented
Issue summary: View changes
Status: Closed (fixed) » Active

Hmm, actually. This patch is in, however, the file being pointed to (includes/media.pages.inc), does not exist. Any idea's?

BarisW’s picture

BarisW commented

The problem is that the hook_menu in the 1.x branch is outdated. It should be the same as the 2.x branch (use the file_entity add files interface). I used a hook_menu_alter to fix it without having to patch the module. Like this:

<?php
/**
 * Implements hook_menu_alter().
 */
function MYMODULE_menu_alter(&$items) {
  $items['admin/workbench/media/add']['page arguments'] = array('file_entity_add_upload');
  $items['admin/workbench/media/add']['file'] = 'file_entity.pages.inc';
  $items['admin/workbench/media/add']['file path'] = drupal_get_path('module', 'file_entity');
}
?>
stevector’s picture

stevector
he/him
commented
Assigned: stevector » Unassigned

Removing the assignment since I haven't worked on this module in years.