Dynamic Filter [Support Thread]

[Design75]

Has anyone resolved or addressed this issue?

I think I solved the tax problem

In includes/templates/YOUR_TEMPLATE/sideboxes/tpl_dynamic_filter.php find

            if ($end < $max) $text = $currency_symbol . round($start * $conversion_rate) . ' -- ' . $currency_symbol . round($end * $conversion_rate);
            else $text = $currency_symbol . round($start * $conversion_rate) . ' and over'; 

and replace with

    if ($end < $max)
    {
      $text = $currency_symbol . round((zen_add_tax($start, $products_tax)) * $conversion_rate) . ' -- ' . $currency_symbol . round((zen_add_tax($end, $products_tax)) * $conversion_rate);
    }
    else
    {
      $text = $currency_symbol . round((zen_add_tax($start, $products_tax)) * $conversion_rate) . ' and over';
    } 

It works for me, but some further testing is needed, before releasing an update for this module

If this works, the next step is to replace the hard-coded text for language defines, to make this mod multilingual

[Design75]

Fixed. All very weird, bit of a clash with something else I use that needs jquery but seems one needs to load before CSS and one after.. Also both seem to need different flavours of jquery? Which I'm confused about, I thought putting the most up to date would cover both??

Hi philip,

I know you reported this a while back, but since i am updatig this module, I thought to give you an update on this.

The more/less button works fine when using jquery 1.4.4, but with higher versions it stops. Now we need to find out why :)

The other updates to be included for now are all the code bugfixes / improvements reported in this thread since version 1.1 was released.

[Kevin205]

Has anyone encountered similar errors with Dynamic Filter v1.0, when using ' for feet and " for inches and filtering for attribute size?
Example Attribute Size: with value 1'-4" x 2'-6"

[25-May-2013 18:09:05 America/Denver] PHP Fatal error: 1064:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '71"", GROUP_CONCAT(CONCAT(REPLACE(po.products_options_name, " ", ""), pov.produc' at line 7 :: SELECT DISTINCT p.products_id, p.products_type, p.master_categories_id,
p.manufacturers_id, p.products_price, p.products_tax_class_id, pd.products_description, IF(s.status = 1,
s.specials_new_products_price, NULL) as specials_new_products_price, IF(s.status =1, s.specials_new_products_price,
p.products_price) as final_price, p.products_sort_order, p.product_is_call, p.product_is_always_free_shipping,
p.products_qty_box_status FROM zen_products p LEFT JOIN zen_specials s on p.products_id = s.products_id LEFT JOIN zen_products_description pd on p.products_id = pd.products_id JOIN zen_products_to_categories p2c on p.products_id = p2c.products_id JOIN zen_products_attributes p2a on p.products_id = p2a.products_id JOIN zen_products_options po on p2a.options_id = po.products_options_id JOIN zen_products_options_values pov on p2a.options_values_id = pov.products_options_values_id WHERE p.products_status = 1
and pd.language_id = '1'
and p2c.categories_id = '17' GROUP BY p.products_id HAVING ( FIND_IN_SET("Size:51" x 71"", GROUP_CONCAT(CONCAT(REPLACE(po.products_options_name, " ", ""), pov.products_options_values_name)))) in /MyPath/includes/classes/db/mysql/query_factory.php on line 120

[Design75]

Has anyone encountered similar errors with Dynamic Filter v1.0, when using ' for feet and " for inches and filtering for attribute size?
Example Attribute Size: with value 1'-4" x 2'-6"

I entered a the value in a testshop and getting the same error. Like explained to you in other threads it has to do with the ' and " in the value name. I think the only solution right now is to use inch and ft instead

[Design75]

Hi philip,

I know you reported this a while back, but since i am updatig this module, I thought to give you an update on this.

The more/less button works fine when using jquery 1.4.4, but with higher versions it stops. Now we need to find out why :)

The other updates to be included for now are all the code bugfixes / improvements reported in this thread since version 1.1 was released.

Resolved

If using jquery 1.6.0 or newer, change the contents of file includes\templates\YOUR_TEMPLATE\jscript\dynamic_filter\jquery.dynamic_filter.mi n.js

$("div.dynamicfilterContent").eq(0).ready(function(){$('ul.dFilterExpand').each(function(){if($(this).attr("scrollHeight")>130)$(this).height(130).siblings('a.dFilterToggle').show();});$("a.dFilterToggle").click(function(){if($(this).siblings("ul.dFilterExpand").height()==130){$('.dFilterToggleImg',$(this)).attr('src',$('.dFilterToggleImg').attr('src').replace('_more','_less')).attr('alt','Less').attr('title','Less');$(this).html($(this).html().replace("More","Less"));$(this).siblings("ul.dFilterExpand").animate({height:$(this).siblings("ul.dFilterExpand").attr("scrollHeight")},"slow");}else{$('.dFilterToggleImg',$(this)).attr('src',$('.dFilterToggleImg').attr('src').replace('_less','_more')).attr('alt','More').attr('title','More');$(this).html($(this).html().replace("Less","More"));$(this).siblings("ul.dFilterExpand").animate({height:130},"slow");}
return false;});});

to

$("div.dynamicfilterContent").eq(0).ready(function(){$('ul.dFilterExpand').each(function(){if($(this).prop("scrollHeight")>130)$(this).height(130).siblings('a.dFilterToggle').show();});$("a.dFilterToggle").click(function(){if($(this).siblings("ul.dFilterExpand").height()==130){$('.dFilterToggleImg',$(this)).attr('src',$('.dFilterToggleImg').attr('src').replace('_more','_less')).attr('alt','Less').attr('title','Less');$(this).html($(this).html().replace("More","Less"));$(this).siblings("ul.dFilterExpand").animate({height:$(this).siblings("ul.dFilterExpand").prop("scrollHeight")},"slow");}else{$('.dFilterToggleImg',$(this)).attr('src',$('.dFilterToggleImg').attr('src').replace('_less','_more')).attr('alt','More').attr('title','More');$(this).html($(this).html().replace("Less","More"));$(this).siblings("ul.dFilterExpand").animate({height:130},"slow");}
return false;});});

.attr("scrollHeight") becomes .prop("scrollHeight") (2x)

[Kevin205]

I entered a the value in a testshop and getting the same error. Like explained to you in other threads it has to do with the ' and " in the value name. I think the only solution right now is to use inch and ft instead

I cannot use inch and ft, as the owner is insisting in getting size attribute in that particular format. I am trying to use escape or special character codes to resolve the issue; as suggested in the other threads.

Thank you for the reply and your solution.

[Design75]

I cannot use inch and ft, as the owner is insisting in getting size attribute in that particular format. I am trying to use escape or special character codes to resolve the issue; as suggested in the other threads.

Thank you for the reply and your solution.

I also tried using the escapes, and special character codes, but they are rewritten to the original character when saving the attribute name

[Kevin205]

I also tried using the escapes, and special character codes, but they are rewritten to the original character when saving the attribute name

Thank you for confirming that.

I think the error is getting from the following file at around line 54, I am trying to make sense out of it.

includes/modules/MyTemplate/dynamic_filter.php

[Design75]

Thank you for confirming that.

I think the error is getting from the following file at around line 54, I am trying to make sense out of it.

Basically what happens is that the single and double are entered in to the query, and thus causing errors. We need to figure a way to tell the script to add an escape to certain characters before adding them to the query.
It shouldn't be that difficult, but my skills are still limited, and am learning on the way :)

[Kevin205]

but my skills are still limited, and am learning on the way :)

Same here.

I tried escaping the " and or '. No luck.

The only thing that does not produce an error, is the following format:

1& #39;-2& #34; x 1& #39;-2& #34;

But try to tell a shop owner to look at the following and do your additions, deletions, edits and audits and etc.

5&# 39;-7& #34; x 3& #39;-11& #34;
1& #39;-2& #34; x 1& #39;-2& #34;
12& #39;-4& #34; x 6& #39;-6& #34;
1&# 39;-2& #34; x 1& #39;-2& #34;
2&# 39;-9& #34; x 5& #39;-3& #34;
.
.
.

How is that for head spin?

I am trying to figure out a way also.

Thanks.




ps. Spaces between & and # are left out, so it displays here.