I am using the V2.1 content module to filter content from only one K2 category. But for some reason, only when I am viewing the excluded K2 categories, articles from the excluded categories are also appearing in the module list.

So the filter is corrupt.

UPDATE: It appears that the problem lies within the module parameter for "Fetch items from children categories". If set to yes, then the filter gets confused and included items from excluded categories - even when these excluded categories are NOT children of the selected categories. When set to no, then the filter seems to behave properly.

Also, this behavior was observed when item ordering was using 'highest rated' and when items from all categories had no ratings.

Hi. I m trying to reproduce the bug but no luck yet. It works fine for me. Are you completely sure that you are using the latest version of K2? Also what's the PHP version you are using?

OK, double checking now. K2 component version is 2.1. Mod_k2_content is also version 2.1. PHP 5.2.10.

Try this: Set up two top-level K2 categories. Create a couple of items in each category, but do not post any ratings.

Set up your module to filter by just one of your categories. Set Fetch items from children categories to Yes. Set item ordering to Highest Rated. Publish it to all pages of the site.

Then from the front end, check it from a page in the excluded category to see if an item from the excluded category is listed in the module.

Lefteris Kavadas said:Hi. I m trying to reproduce the bug but no luck yet. It works fine for me. Are you completely sure that you are using the latest version of K2? Also what's the PHP version you are using?

Hi again. OK you are totally correct. The bug occurs when the module is on a category page. It has been fixed for the next release. Thanks for reporting this issue to us.

Thanks! Those of us who use extensions for free have an obligation to at least contribute through bug reporting, testing, and forum contribution.

Lefteris Kavadas said:It has been fixed for the next release. Thanks for reporting this issue to us.

Oh I got this bug also! I was looking for a solution but you Lefteris Kavadas said it is already fixed for the next release.. Hmm, can you upload this fix or just point us your svn where it is fixed? I'm quiting my job and I need my current site done until monday, so if I could use your fix I would be glad.

Thanks, and keep the good stuff comming ;)

You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with:

function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

Thank you man!
It worked flawessly! :D

Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with:
function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

Colin, tks for solution. This also solves a even more serious problem. When you set to fetch sub-categories, if you put 2 content modules in the same page, it only takes the articles of the last first category module.

Anyhow, taking the static array out, also solve the above problem.

Great job!

Kormann said:Thank you man! It worked flawessly! :D

Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with: function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

Hi. Thanks for posting this mod but i think that it will break in PHP 4. We have implemented a little different patch for this. You can always view the latest source from the Joomlaworks svn repository on google code.

Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with:
function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

Lefteris,

It would be better if you would either post the correct code here or release an updated version of the entire module.

Lefteris Kavadas said:Hi. Thanks for posting this mod but i think that it will break in PHP 4. We have implemented a little different patch for this. You can always view the latest source from the Joomlaworks svn repository on google code.
Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with: function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

You are probably right that it will break in PHP4. But after looking at the SVN repository I noticed that the second parameter is used when you only want to clear the array, thus you have to change all the occurrences where the function is called. You should make it default to true and when you make the recursive call throw in false.

Lefteris Kavadas said:Hi. Thanks for posting this mod but i think that it will break in PHP 4. We have implemented a little different patch for this. You can always view the latest source from the Joomlaworks svn repository on google code.
Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with: function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}

Thanks Colin!

I came here to post my fix to this issue, but realized your fix is much cleaner. +5 points for you!

Colin Salomons said:You have to edit function getCategoryChilds in components/com_k2/models/itemlist.php. The problem is that static array. Here is what I replaced it with:
function getCategoryChilds($catid, &$array=array()) {

$user = &JFactory::getUser();
$aid = $user->get('aid');
$db = &JFactory::getDBO();
$query = "SELECT * FROM #__k2_categories WHERE parent={$catid} AND published=1 AND trash=0 AND access<={$aid} ORDER BY ordering ";
$db->setQuery($query);
$rows = $db->loadObjectList();
foreach ($rows as $row) {
array_push($array, $row->id);
if (K2ModelItemlist::hasChilds($row->id)) {
K2ModelItemlist::getCategoryChilds($row->id, $array);
}

}
return $array;
}