I would like to be able to assign the navigation vocabulary to some existing/shared vocabulary. I have done this successfully by just altering the variables table to point to an already existing vid, but I think it would be a nice feature to have in the configuration menu.

CommentFileSizeAuthor
#13 weblinks18.patch4.12 KBnancydru

Comments

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

http://drupal.org/node/266383 is a duplicate.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

taxonomy_get_vocabularies('weblinks') would pick up all vocabularies that are used for the weblinks content type.

rmiddle’s picture

rmiddle commented

Ok this looks like a pretty simple fix will need to look into this.

Thanks
Robert

rmiddle’s picture

rmiddle commented
Status: Active » Closed (duplicate)

Mark this one a dupe on an earlier ticket.

rmiddle’s picture

rmiddle commented
Status: Closed (duplicate) » Active
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

Hmm... not quite that simple. "_weblinks_get_vocid" sets the "module" column in the vocabulary table to "weblinks" (opening the way to "weblinks_term_path"). Setting a different vocabulary would probably prevent this. How important is it? I don't know.

rmiddle’s picture

rmiddle commented

Yea I looked into this earlier. That function is used in 90% of the code. I might require a reactor to make this change.

Thanks
Robert

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Assigned: Unassigned » nancydru

Not really. I've already got this coded; I'm just trying to set a test.

rmiddle’s picture

rmiddle commented

Nancy,

That is great. I am still learning the taxonomy system some but if you can make the change without having to redue large chunks of code great as there have been several requests for this.

Thanks
Robert

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Active » Needs review

This patch adds a vocabulary selection at the top of the settings page if there is more than one vocabulary assigned to the weblinks content type. If there is only one, the existing function continues as it always has.

Please visit the settings page after applying this because I also moved all radios to single lines. You may or may not like that, but it does shrink the settings page a bit.

rmiddle’s picture

rmiddle commented

Comment attachment is have issues today I see. Need to re-post the patch.

Thanks
Robert

rmiddle’s picture

rmiddle commented
Version: 5.x-1.8 » master
nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
StatusFileSize
new weblinks18.patch4.12 KB

Maybe I need another glass of wine...

rmiddle’s picture

rmiddle commented
Status: Needs review » Patch (to be ported)

Works good overall. Committing to CVS.

Note to self

Not sure about the category list on the bottom of this page? They don't change until you save the changes. That will confuse people need to think of a better way to handle that area.

Thanks
Robert

rmiddle’s picture

rmiddle commented

Note:
Also need to do testing in none is selected how that is handled.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

If none is selected, it should continue as it does now.

rmiddle’s picture

rmiddle commented

Yea that is no category list. I need to add code to alert or update this.

rmiddle’s picture

rmiddle commented
Status: Patch (to be ported) » Fixed

Backported to 5. Committed to CVS.

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented
Status: Fixed » Closed (fixed)

didn't close automatically