My files menu item

sym - August 24, 2006 - 17:19
Project:e-Commerce
Version:5.x-3.3
Component:file
Category:support request
Priority:normal
Assigned:Unassigned
Status:active
Description

Does the my files menu item have to be a MENU_DYNAMIC_ITEM? I want to be able to remove it in admin/menu but because it's a dynamic item I can't.

I'll post a simple patch if it's not needed.

#1

sime - September 25, 2006 - 07:30

I'd accept a patch.

#2

designerbrent - June 15, 2007 - 14:22
Version:4.7.x-1.x-dev» 5.x-3.0
Status:active» needs review

Attached is a patch for this for version 3.

AttachmentSize
file-menuchange.diff.patch 405 bytes

#3

designerbrent - August 2, 2007 - 15:03
Version:5.x-3.0» 5.x-3.3

I'm attaching a second patch for the file menu to do the same thing as the previous patch, but for 5.x-3.3.

Is there a reason that this is not in the release?

Thanks... Brent

AttachmentSize
filemenuchange.diff 389 bytes

#4

gordon - August 3, 2007 - 02:02
Status:needs review» reviewed & tested by the community

No reason it was just missed. I will commit this later.

Please note, that we use soft tabs (2 spaces) for indentation.

#5

gordon - August 3, 2007 - 02:49
Status:reviewed & tested by the community» fixed

Commited to all current versions

#6

Anonymous - August 17, 2007 - 03:03
Status:fixed» closed

#7

bonzo meier - October 5, 2009 - 16:53
Assigned to:Anonymous» bonzo meier
Status:closed» active

in 5.x-3.5 the patch from #3 (designerbrent) is applied - still it doesn't work. the error is that you cant't move the menu item 'my Files' e.g. to another menu than the main navigation. if you do so (via ?q=admin/build/menu/item/edit) it is changed for the acting user (this case admin, uid1), but not for any other user. same for deactivation.

anyway, it might be useful if this link only appeared if files were available for download.

please fix this or post a hint - this bug is VERY disturbing...

thanks in advance,
bonzo

#8

bonzo meier - October 5, 2009 - 16:53
Assigned to:bonzo meier» Anonymous
 
 

Drupal is a registered trademark of Dries Buytaert.