This patch fixes an error where If you set 'Display play menu tab' to 'No' under administer > settings > video, you don't get the video. Instead you get the "?q=node/" page.

Looks like the problem was that the menu item was never getting registered for node/$nid/play. This patch changed the menu hook so that this menu hook always registers that non-cached item, and changes it to a MENU_CALLBACK if we are not showing the tab.

CommentFileSizeAuthor
video_menu_fix.patch1.5 KBseanbfuller

Comments

fax8’s picture

fax8 commented
Assigned: Unassigned » fax8
Status: Needs review » Needs work

Thank you for pointing this out.

However your patch is not correct:

you are using 'MENU_LOCAL_TASK' instead of MENU_LOCAL_TASK .

fax8’s picture

fax8 commented
Status: Needs work » Fixed

fixed on video.module v1.50 .

Thank you.

Fabio

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)