Closed (fixed)
Project:
Video
Version:
4.7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
3 Aug 2006 at 20:49 UTC
Updated:
8 Sep 2006 at 22:45 UTC
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.
| Comment | File | Size | Author |
|---|---|---|---|
| video_menu_fix.patch | 1.5 KB | seanbfuller |
Comments
Comment #1
fax8 commentedThank you for pointing this out.
However your patch is not correct:
you are using 'MENU_LOCAL_TASK' instead of MENU_LOCAL_TASK .
Comment #2
fax8 commentedfixed on video.module v1.50 .
Thank you.
Fabio
Comment #3
(not verified) commented