Desperately seeking taxonomy [#963856]

Comment	File	Size	Author
#33	963856-33-taxonomy.patch	13.45 KB	johnalbin
#32	963856-32-taxonomy.patch	11.48 KB	johnalbin
#6	menu-position-taxonomy-963856.patch	13.39 KB	artis.bajars
#4	taxo.patch	10.49 KB	johnalbin

Comment #1

ericduran commented 23 November 2010 at 04:49

Hi @JohnAlbin,

I started working on this but I haven't finish. You need to use the entity api to get the data. I've already got the taxonomy form working. I'll try to post a complete patch tomorrow.

Regarding getting the vids of the current node you can do this:

<?php

function menu_position_entity_load($entities, $type) {
// if you're going to do a taxonomy term page maybe a hook_entity_view is better than load
  if ($type == 'taxonomy_term') {
        foreach ($entities as $entity) {
        dsm($entities);
      }  
  }
}
?>

Hope this helps, I plan on submitted a completely patch, but the entity API is also new to me, So I'm learning as I go. I'll probably try to put up a documentation page after I finish.

Log in or register to post comments

Comment #2

johnalbin

they/他

English

Taipei, Taiwan

commented 23 November 2010 at 09:10

Sweet! The entity API is totally new to me as well. Thanks for the help!

Log in or register to post comments

Comment #3

johnalbin

they/他

English

Taipei, Taiwan

commented 17 December 2010 at 15:26

The d6 version has a taxonomy plug-in now.

Log in or register to post comments

Comment #4

johnalbin

they/他

English

Taipei, Taiwan

commented 22 December 2010 at 21:21

Status:

Active

» Needs work

Status	File	Size
new	taxo.patch	10.49 KB

Here's a not-yet complete patch for a taxonomy patch.

The menu_position_menu_position_rule_taxonomy_form() and menu_position_taxonomy_autocomplete() should be working. The other functions are only partially finished.

Log in or register to post comments

Comment #5

binford2k commented 22 February 2011 at 23:48

subscribe

Log in or register to post comments

Comment #6

artis.bajars commented 2 March 2011 at 15:09

Status:

Needs work

» Needs review

Status	File	Size
new	menu-position-taxonomy-963856.patch	13.39 KB

Here's a patch, continued upon the work started in #4, that gets taxonomy conditions working. Should apply cleanly to 7.x-1.x-dev. Please test.

Log in or register to post comments

Comment #7

rvilar

Catalan

Montcada i Reixac, Catalonia

commented 14 March 2011 at 13:22

Status:

Needs review

» Needs work

This works for me, but when I attempt to use the autocomplete widget to enter the taxonomy term an error ocur.

Log in or register to post comments

Comment #8

mrharolda commented 24 March 2011 at 11:17

The patch in #6 doesn't apply anymore on the current -dev. Any progress on getting this completed and merged into the module?

Log in or register to post comments

Comment #9

ropaolle commented 27 March 2011 at 10:16

subscribe

Log in or register to post comments

Comment #10

duvrai commented 31 March 2011 at 23:22

subscribe

Log in or register to post comments

Comment #11

valderama commented 7 April 2011 at 09:02

subscribe

Log in or register to post comments

Comment #12

joakimekroos commented 26 April 2011 at 10:59

+1

Log in or register to post comments

Comment #13

vikfroberg commented 28 April 2011 at 10:24

+1

Log in or register to post comments

Comment #14

akanouras commented 10 May 2011 at 08:55

Subscribing

Log in or register to post comments

Comment #15

Shadlington commented 14 May 2011 at 16:03

Subbing

Log in or register to post comments

Comment #16

itangalo commented 20 May 2011 at 11:28

For information: It is possible to use conditions on taxonomy terms by using this module's functionality from Rules.

For more information, check out #1163662: Functionality ported as a Rules action.

Log in or register to post comments

Comment #17

Anonymous (not verified) commented 16 June 2011 at 22:15

The patch from #6 worked for me, however I had to apply the patch using the old method "patch < patchname.patch" rather than the new git method described on the Drupal patch documentation page. I also had to fix a single merge failure, but no big deal.

Now things are working perfectly. Works just like the 6.x version did and allows me to really fine tune where menu items go. I haven't had any issues with the autocomplete field either. There is a slight issue where upon adding a term from the autocomplete field the taxonomy tab only shows what I typed, not what the actual term is, until I save the form. Then it shows the full term correctly. Not a huge deal, but might be slightly confusing.

Getting this committed to the dev branch would be great.

Log in or register to post comments

Comment #18

meus commented 25 June 2011 at 23:07

subscribe

Log in or register to post comments

Comment #19

jonasdowney commented 27 June 2011 at 15:42

subscribe

Log in or register to post comments

Comment #20

jlenni commented 20 July 2011 at 08:44

subscribe

Log in or register to post comments

Comment #21

rogical commented 21 July 2011 at 09:57

+1 seems not ok to set menu position for taxonomy term pages yet

Log in or register to post comments

Comment #22

Mark Theunissen commented 23 July 2011 at 08:28

Subscribe

Log in or register to post comments

Comment #23

m3n0R commented 26 July 2011 at 09:03

Subscribe

Log in or register to post comments

Comment #24

verdaechtig commented 27 July 2011 at 09:44

Subscribe

Log in or register to post comments

Comment #25

valderama commented 27 July 2011 at 17:30

if you are using menu_block this module could be an alternative..

http://drupal.org/project/context_menu_block

Log in or register to post comments

Comment #26

itangalo commented 30 July 2011 at 01:03

As pointed out in #16, it is possible to use Rules with Menu position to activate a menu item only if a node has a specified taxonomy term.

Here's a quick screencast showing how to go about it: http://nodeone.se/node/934
Here's an issue about getting more integration between Menu position and Rules: #1163662: Functionality ported as a Rules action

Log in or register to post comments

Comment #27

petrovichby commented 25 August 2011 at 15:55

Subscribe

Log in or register to post comments

Comment #28

waynedpj commented 27 September 2011 at 10:04

subscribe

Log in or register to post comments

Comment #29

attiks commented 2 October 2011 at 19:44

sub

Log in or register to post comments

Comment #30

cangeceiro commented 12 October 2011 at 14:00

according to the commit logs this was added on oct 5th http://drupalcode.org/project/menu_position.git/commit/25ea476 but i dont see this in the latest beta or -dev branches that were released on the 10th and 11th respectively.

Log in or register to post comments

Comment #31

johnalbin

they/他

English

Taipei, Taiwan

commented 12 October 2011 at 14:26

@cangeceiro The drupalcode.org repository viewer doesn't do a good of showing this, but that commit is on a temporary feature branch called 7.x-1.x-963856. http://drupalcode.org/project/menu_position.git/shortlog/refs/heads/7.x-...