subj

Comments

firebee’s picture

firebee commented

I think I'm seeing this... The statistic counter no longer counting after I set this up. Took me a while to figure out what happen

Sorix’s picture

Sorix commented
Priority: Normal » Critical
Anonymous’s picture

Anonymous (not verified) commented

I believe i too have encountered this issue, but had not even considered Node Breadcrumbs Just taken a site fron d5.6 to d6.12 My first thought was it was all none "page" types but now realise Node Breadcrumbs are generated for affected pages.

One thing I did notice was that if you type the node/nid the counter did update but the node breadcrumb is still added , maybe a clue

Anonymous’s picture

Anonymous (not verified) commented

Forgot the obvious. Tried disabling Node Breadcrumbs and the counter started to work again

Emmental’s picture

Emmental commented

I also have this problem. However, what it is actually doing is incrementing the view counter of the parent page linked by the breadcrumb.

For example, if I use the module to link the node "Game 1" to the parent "Game Reviews", for every refresh of node "Game 1" the counter for the node "Game Reviews" is incremented.

I have had occasions where the correnct counter has updated which have usually involved a login or logout but I can't reproduce that exactly.

edhel’s picture

edhel commented
Status: Active » Fixed

fixed in beta3

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Active

The new version appears to cause another issue

After uploading, I lost the View Edit Revisions Tabs for nodes using Node Breadcrumb. reverting to the previous version resolved

Anonymous’s picture

Anonymous (not verified) commented

On further investigation i can tell you its this bit of code that makes the tabs dissapear

  $add_active_class = variable_get('node_breadcrumb_add_active_class', FALSE);
  $menu_item = menu_link_load($mid);
  menu_set_item($_GET['q'], $menu_item);
  if ($add_active_class) {
	$q = $_GET['q'];
	$_GET['q'] = $menu_item['link_path'];
  }

What it does is beyond me :)

edhel’s picture

edhel commented
Version: 6.x-1.0-beta2 » 6.x-1.0-beta3
Status: Active » Fixed

Add before line:

menu_set_item($_GET['q'], $menu_item);

These two line:

menu_local_tasks(0);
menu_local_tasks(1);

Fixed in dev-version (will be later at night).

Status: Fixed » Closed (fixed)

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