CMS MADE SIMPLE FORGE

News Module

 
Back Back to List

[#5884] node.depth not calculated properly

avatar
Created By: Jack Skiba (nidus)
Date Submitted: Thu Dec 23 16:14:02 -0500 2010

Assigned To:
Version: None
CMSMS Version: None
Severity: Major
Resolution: Fixed
State: Closed
Summary:
node.depth not calculated properly
Detailed Description:
function get_categories
$depth= count(explode('\.', $row['hierarchy']));
should be:
$depth= count(explode('.', $row['hierarchy']));
no need for a \


History

Comments
avatar 
Date: 2011-08-30 03:08
Posted By: Fred Polizo (fredp)

I just discovered the same bug in the latest News versions: 
  News 2.11.4 (in CMSMS 1.9.4.3) on line 845 of modules/News/News.module.php
News 2.12 (in CMSMS 1.10-beta2) on line 77 of
modules/News/lib/class.news_ops.php

This bug causes all "depth" and "prevdepth" fields, of the items returned, to
have values of 1; since no cms_module_news_categories.hierarchy column value
will contain the '\.' substring.
avatar 
Date: 2011-08-31 04:23
Posted By: Rolf (rolf1)

Fixed in SVN, will be shipped with 1.10
avatar 
Date: 2011-10-22 14:03
Posted By: Ronny Krijt (ronnyk)

1.10 is released.
Updates

Updated: 2011-10-22 14:03
state: Open => Closed

Updated: 2011-08-31 04:23
resolution_id: => 7
cmsms_version_id: => -1