- Posts: 49
[SOLVED] mod_flexicontent category scope behaviour is ANY category, without option to configure REQUIRE ALL
9 years 10 months ago - 9 years 10 months ago #60608
by fortino
Replied by fortino on topic mod_flexicontent category scope behaviour is ANY category, without option to configure REQUIRE ALL
WELL, this is defnitely not a bug but the way this module works.
I modified the helper and the xml to have an option about the "query's logic" [AND / OR].
modules/mod_flexicontent/helper.php
row 1692 to 1731, replace with this:
modules/mod_flexicontent/mod_flexicontent.xml
INSERT this in after row 235 and before 236:
Hope this will help someone!
Please don't be shy and suggest me a better query if you feel to. I'm not a db wizard!
I modified the helper and the xml to have an option about the "query's logic" [AND / OR].
modules/mod_flexicontent/helper.php
row 1692 to 1731, replace with this:
Code:
if( $params->get('cat_and_or', 0) ==0 ){
$items_query = 'SELECT '
. ' i.id '
. (in_array('commented', $ordering) ? $select_comments : '')
. (in_array('rated', $ordering) ? $select_rated : '')
. ' FROM #__flexicontent_items_tmp AS i'
. ' JOIN #__flexicontent_items_ext AS ie on ie.item_id = i.id'
. ' JOIN #__flexicontent_types AS ty on ie.type_id = ty.id'
. ' JOIN #__flexicontent_cats_item_relations AS rel ON rel.itemid = i.id'
. ' JOIN #__categories AS c ON c.id = rel.catid'
. ' JOIN #__categories AS mc ON mc.id = i.catid'
. $joinaccess
. $join_favs
. $join_date
. (in_array('commented', $ordering) ? $join_comments : '')
. (in_array('rated', $ordering) ? $join_rated : '')
. $orderby_join
. $join_field_filters
. $where .' '. ($apply_config_per_category ? '__CID_WHERE__' : '')
. $where_field_filters
. ' GROUP BY i.id'
. $orderby
;
}
else{
$items_query = 'SELECT '
. ' i.id '
. (in_array('commented', $ordering) ? $select_comments : '')
. (in_array('rated', $ordering) ? $select_rated : '')
. ' FROM '
. ' (SELECT COUNT(rel2.itemid), rel2.itemid FROM rl2k16_flexicontent_cats_item_relations AS rel2
WHERE catid IN ('.implode(",",array_merge($catids, $catids_arr)).')
GROUP BY rel2.itemid HAVING COUNT(*) > '.(count(array_merge($catids, $catids_arr))-1).') as guzzo,'
. ' #__flexicontent_items_tmp AS i'
. ' JOIN #__flexicontent_items_ext AS ie on ie.item_id = i.id'
. ' JOIN #__flexicontent_types AS ty on ie.type_id = ty.id'
. ' JOIN #__flexicontent_cats_item_relations AS rel ON rel.itemid = i.id'
. ' JOIN #__categories AS c ON c.id = rel.catid'
. ' JOIN #__categories AS mc ON mc.id = i.catid'
. $joinaccess
. $join_favs
. $join_date
. (in_array('commented', $ordering) ? $join_comments : '')
. (in_array('rated', $ordering) ? $join_rated : '')
. $orderby_join
. $join_field_filters
. $where .' '. ($apply_config_per_category ? '__CID_WHERE__' : '')
. ' AND i.id = guzzo.itemid '
. $where_field_filters
. ' GROUP BY i.id'
. $orderby
;
}
modules/mod_flexicontent/mod_flexicontent.xml
INSERT this in after row 235 and before 236:
Code:
<field name="cat_and_or" type="list" default="0" label="AND / OR?" description="AND / OR?" class="method_cat_ops" >
<option value="0">OR</option>
<option value="1">AND</option>
</field>
Hope this will help someone!
Please don't be shy and suggest me a better query if you feel to. I'm not a db wizard!
Last edit: 9 years 10 months ago by ggppdk.
Please Log in or Create an account to join the conversation.
9 years 10 months ago - 9 years 10 months ago #60664
by ggppdk
-- Flexicontent is Free but involves a big effort on our part.
Like the our support? (for a bug-free FC, despite having a long list of functions) Like the features? Like the ongoing development and future commitment to FLEXIcontent?
-- Add your voice to the FLEXIcontent JED listing with a 5-star review. Thanks!
Replied by ggppdk on topic mod_flexicontent category scope behaviour is ANY category, without option to configure REQUIRE ALL
Hello
the count-having is a bit heavy,
thus it is better to optimize by doing an
AND i.id IN (.... count-having....)
like we do for tags
PS: the TAGS require ALL was broken because of wrong parameter name
Here is the new files (fixed TAGs require ALL and added CATS require ALL)
github.com/FLEXIcontent/flexicontent-cck...501033f28483cd3b7499
the count-having is a bit heavy,
thus it is better to optimize by doing an
AND i.id IN (.... count-having....)
like we do for tags
PS: the TAGS require ALL was broken because of wrong parameter name
Here is the new files (fixed TAGs require ALL and added CATS require ALL)
github.com/FLEXIcontent/flexicontent-cck...501033f28483cd3b7499
-- Flexicontent is Free but involves a big effort on our part.
Like the our support? (for a bug-free FC, despite having a long list of functions) Like the features? Like the ongoing development and future commitment to FLEXIcontent?
-- Add your voice to the FLEXIcontent JED listing with a 5-star review. Thanks!
Last edit: 9 years 10 months ago by ggppdk.
Please Log in or Create an account to join the conversation.
