1. In settings, omitted vocabularies are defined, but when deciding weather to display subscribe/unsubscribe links on content pages or not, selected vocabulary IDs (vid) are (improperly) compared to term IDs (tid), causing under certain conditions wrong display decisions. To correct the mistake, I've changed:
$taxa->tid
into:
$taxa->vid
in line 643.

2. On user settings page (categories / taxonomies), only term names are displayed (without a vocabulary name). In my case, I have the same (or similar) term names in different vocabularies, so I've had to enhance the display including both, vocabulary and term name. To get that, I've replaced:
$term->name
into:
$vocab->name . ": " . $term->name
in line 823.

Comments

dziemecki’s picture

dziemecki commented
Assigned: Unassigned » dziemecki
Status: Active » Fixed

Easy enough. Your suggestions have been added and committed.

jwilde’s picture

jwilde commented

Hi,

I just updated and this change, $taxa->tid into: $taxa->vid in line 643, wipes out the subscribe link(s) on nodes. I have not tested for any consequences.

Jim

dziemecki’s picture

dziemecki commented

Are you sure? I sure see them on my test instance.

jwilde’s picture

jwilde commented

Hi,

I'm using multiple vocabs (freetagging and a fixed one with one content type.) The problem is I only want to use subscriptions on the fixed one and so I omitted the freetagging in subscriptions settings. I didn't want to display all of the freetags in "my subscriptions/categories" but I do want users to be able to subscribe to the blog or blog post.

As a quick fix I removed lines 643 - 646.

if (!isset($node->taxonomy)){$node->taxonomy=array();}
foreach($node->taxonomy as $taxa){
if(in_array($taxa->vid,$omittaxa)){$taxaclear = false;}
}

Thanks,

Jim

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)