Update d3pie.js

[sebinkuriakose]

Fixing 'ReferenceError: pie is not defined'

[benkeen]

Hi @sebinkuriakose, thanks for the PR! Under what circumstances does that error occur? Could you provide a few more details on how to reproduce?

[kashiif]

Thanks for the awesome work.

I faced this with following stack trace:

Uncaught ReferenceError: pie is not defined
    at Object.maybeCloseOpenSegment (core-libs.min-a77ef77….js:8)
    at Object.openSegment (core-libs.min-a77ef77….js:8)
    at SVGGElement.<anonymous> (core-libs.min-a77ef77….js:8)
    at SVGGElement.<anonymous> (d3.min.js:2)

To reproduce, click on a segment of a piechart. (Also reproducible on website charts as well)

[Spekou]

Hello,

I am facing the same issue. I have several charts shown in the same page so I can't name them all "pie".

When will this be available in master version ?

[irimiab]

Facing the same issue. It can be reproduced when you are trying to use the d3pie object on a div with another ID than "pie":

        var pie = new d3pie("mypie", {
		header: {
			title: {
				text: "A Very Simple Pie",
				fontSize: 30
			}
		},
		data: {
			content: [
				{ label: "JavaScript", value: 264131 },
				{ label: "Ruby", value: 218812 },
				{ label: "Java", value: 157618}
			]
		}
	});

[CDluznie]

Same problem, any news when the PR will be merged ?

[gabemeola]

👍🏾