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

Fix for issue http://github.com/c3js/c3/issues/1691 #1694

Closed
wants to merge 1 commit into from

Conversation

kelthuzad
Copy link
Contributor

This is the bugfix for Fix for issue #1691
The bug was introduced with the implementation of #1471

See http://jsfiddle.net/L3obmfsn/ for a working example.

If an object has key that contains . or [], it is now correctly found
instead of searching for an inner object or array that does not exist.
Example:
var data = [{"key.withdot":"1"}]; // did not work before, works now
var data2 = [{"key":{"innerkey":"1"}}]; // still works

Updated test with new bugfix.

If an object has key that contains . or [], it is now correctly found
instead of searching for an inner object or array that does not exist.
Example:
var data = [{"key.withdot":"1"}]; // did not work before, works now
var data2 = [{"key":{"innerkey":"1"}}]; // still works

Updated test with new bugfix.
@tsileo
Copy link

tsileo commented Nov 8, 2016

Any update on this issue?

@DoomyTheFroomy
Copy link

After I have run into the same problem again, could you please let this pass and release a new version? This would be a great new years present 👍 🥇

@carmamir
Copy link

carmamir commented Aug 4, 2017

Would be great to include this bug fix into a main release as it is a big issue with any titles having dots (starting with domain names !!!)

@scherrs
Copy link

scherrs commented Dec 19, 2019

Is there an update on this issue?

@1Jesper1
Copy link

1Jesper1 commented Jan 7, 2020

Can we merge this pr?

@miladtai
Copy link

When will this problem be solved ?

@kt3k kt3k mentioned this pull request Feb 26, 2020
@kt3k
Copy link
Member

kt3k commented Feb 26, 2020

closing in favor of #2761

@kt3k kt3k closed this Feb 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants