'right' and 'bottom' issue in Phaser.Ellipse.prototype

[hightopo]

https://github.com/photonstorm/phaser/blob/dev/src/geom/Ellipse.js#L196
this.width = this.x + value;
should be changed to:
this.width = value - this.x;

https://github.com/photonstorm/phaser/blob/dev/src/geom/Ellipse.js#L238
this.height = this.y + value;
should be changed to:
this.height = value - this.y;

[hightopo]

And i found the getBounds method has been defined twice in Phaser.Ellipse.prototype, and the first one below is wrong and useless.
https://github.com/photonstorm/phaser/blob/dev/src/geom/Ellipse.js#L76
getBounds: function () {
return new Phaser.Rectangle(this.x - this.width, this.y - this.height, this.width, this.height);
},

[photonstorm]

This class is straight from Pixi. When suggesting lines to fix please include the line number. You can get this by browsing to the file on github and clicking the line number in the left column, that will give you a URL you can paste in here that lets us find what you're talking about much quicker.

Always do it (and tests) on the dev branch though, not master, as they're far apart.

[hightopo]

Thank @photonstorm for pointing out, i have updated issue's description.

[photonstorm]

Perfect, thanks. I'll raise these on the Pixi repo.

[photonstorm]

Fixed locally in Phaser :)

[spayton]

The ".contains" function got broken with this update. Was there an intentional update here?

[photonstorm]

Split to a separate issue please.