Make tab work with taxomony/term View override

Oliver Coleman - March 28, 2008 - 03:33
Project:Edit term
Version:5.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Oliver Coleman
Status:closed
Description

I need to use a View to display terms to allow ordering nodes by something other than sticky/creation date. Views kindly provides a default taxonomy/term view. Enabling this causes the "Edit Term" menu tab to break, however. I think this is because Views also uses "/edit" on the end of the address (to edit the view). Renaming the "Edit Term" menu tab URL from ".../edit" to ".../edit-term" seems to do the trick (two lines in edit_term.module). Attached is a patch for brevity. :)

AttachmentSize
edit_term.module.diff851 bytes

#1

dman - April 6, 2008 - 14:46

Interesting.
I guess this is a useful fix. Certainly more practically useful than my minor objection to a messy name change workaround.
Hm. OK, if you are viewing a view, then it's the view that deserves to own the 'edit' noun I guess :)
OK. +1
I'll roll it next time I've touching those things...

#2

catch - June 10, 2008 - 11:04
Status:needs review» reviewed & tested by the community

This is a worthwhile patch in lieue of a better solution, tested and it works.

#3

christefano - August 18, 2008 - 22:02

#296629 is a duplicate of this issue.

#4

dman - August 19, 2008 - 00:11
Status:reviewed & tested by the community» fixed

Committed to DRUPAL-5
Sorry for the delay folks, it just dropped of the to-do list :-B

#5

Anonymous (not verified) - September 2, 2008 - 00:12
Status:fixed» closed

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

#6

christefano - September 22, 2008 - 02:39
Status:closed» active

Thanks! I think this change is significant enough to push out a new release, though. Changing back to active.

#7

dman - September 27, 2008 - 05:05

Release created. Now the recommended for 5.x.
Not sure if there was any upgrade action needed if installing this version? Just flushing the menus should have done it, so I guess visiting upgrade.php will help, if the chenge doesn't show up.

http://drupal.org/node/314111

#8

dman - September 27, 2008 - 05:05
Status:active» fixed

o yeh. fixed.

#9

Anonymous (not verified) - October 11, 2008 - 05:22
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.