Add a check for '@' in safeId()

CottageLabs / facetview2

a pure javascript frontend for ElasticSearch search indices.

MIT License

78 stars 31 forks source link

Add a check for '@' in safeId() #15

Closed wonzbak closed 8 years ago

wonzbak commented 8 years ago

Logstash has field name with @, which disrupts jQuery selectors.

This PR fix the bug when a fieldname of a facet has a '@'. With a '@' facet values was not displayed.

richard-jones commented 8 years ago

Sorry to be a pain, as the change seems good, but there's a massive amount of whitespace diff. Any chance you can clean up the patch before I pull it?

wonzbak commented 8 years ago

Hi,

I cleanup all whitespaces diff, and update my commit.

richard-jones commented 8 years ago

Sorry, I lost track of this, but merged now