Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to use decimal aatribute #27

Closed
roshnibharambe opened this issue Aug 27, 2015 · 7 comments
Closed

Unable to use decimal aatribute #27

roshnibharambe opened this issue Aug 27, 2015 · 7 comments
Labels

Comments

@roshnibharambe
Copy link

We have some attributes which haves decimal values. I have installed this module and it works fine for all values except decimal.

It gives me the following error.

Call to undefined method Smile_ElasticSearch_Helper_Data::getAttributeFieldName() in app/code/community/Smile/ElasticSearch/Model/Catalog/Layer/Filter/Decimal.php on line 136.

Is it possible to have decimal attribute values with this module?

@nicosomb
Copy link
Contributor

Hello @roshnibharambe,

When you talk about decimal, what do you mean? Can you give me an example please?
I tried with "2,5" and "2.5", it works fine.

@roshnibharambe
Copy link
Author

Hello @nicosomb ,

decimal means 1.2 , 0.01.

@nicosomb
Copy link
Contributor

OK, so it seems to be like my tests. Did you launch a reindex after your changes?

@roshnibharambe
Copy link
Author

Yes. i have re-indexed all data twice.

On Fri, Aug 28, 2015 at 2:40 PM, Nicolas Lœuillet notifications@github.com
wrote:

OK, so it seems to be like my tests. Did you launch a reindex after your
changes?


Reply to this email directly or view it on GitHub
#27 (comment)
.

Thanks & Regards,
Roshani Bharambe
+91 8879431195
Ambab Infotech Pvt Ltd.
http://www.ambab.com

@afoucret afoucret added the bug label Sep 14, 2015
@afoucret
Copy link
Contributor

The bug is well known.
If you want to fix it you have to disable faceting on this attribute. Then everything should be ok again.

I have to backport the patch a colleague of me has done for a project in order to be able to use it has a filter.

@afoucret
Copy link
Contributor

The patch will be available very soon. I need to clean one or twio things before it will be merged.

FYI, Decimal and integer attributes facets will be rendered just like price range filter as you can see in the following example :

capture d ecran 2015-09-22 a 17 26 15

@afoucret
Copy link
Contributor

This have been fixed since 79d980e.

If you are using ES > 1.7 (officially not supported), be careful that you should add this to your elasticsearch.yml config file :

script.inline: on
script.indexed: on

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants