Fixed bug in DisplayObject where it was using PI_2 instead of PI2.

This commit is contained in:
Richard Davey 2016-11-22 19:44:55 +00:00
parent 03fe5ef076
commit 8f4858c705
2 changed files with 12 additions and 11 deletions

View file

@ -40,7 +40,7 @@ Grab the source and join in the fun!
> In Development > In Development
This is the development branch, where we are building Phaser 3.0.0 This is the development branch, where we are building Phaser 2.7.1
Check the [Change Log](#change-log) to see what we've done so far. Or switch to the master branch for the current stable release. Check the [Change Log](#change-log) to see what we've done so far. Or switch to the master branch for the current stable release.
@ -99,7 +99,7 @@ Install via [bower](http://bower.io)
Install via [npm](https://www.npmjs.com) Install via [npm](https://www.npmjs.com)
`npm install phaser` `npm install phaser@2.7.1`
Using Browserify? Please [read this](#browserify). Using Browserify? Please [read this](#browserify).
@ -107,15 +107,15 @@ Using Browserify? Please [read this](#browserify).
[jsDelivr](http://www.jsdelivr.com/#!phaser) is a "super-fast CDN for developers". Include the following in your html: [jsDelivr](http://www.jsdelivr.com/#!phaser) is a "super-fast CDN for developers". Include the following in your html:
`<script src="//cdn.jsdelivr.net/phaser/3.0.0/phaser.js"></script>` `<script src="//cdn.jsdelivr.net/phaser/2.7.1/phaser.js"></script>`
or the minified version: or the minified version:
`<script src="//cdn.jsdelivr.net/phaser/3.0.0/phaser.min.js"></script>` `<script src="//cdn.jsdelivr.net/phaser/2.7.1/phaser.min.js"></script>`
[cdnjs.com](https://cdnjs.com/libraries/phaser) also offers a free CDN service. They have all versions of Phaser and even the custom builds: [cdnjs.com](https://cdnjs.com/libraries/phaser) also offers a free CDN service. They have all versions of Phaser and even the custom builds:
`<script src="https://cdnjs.cloudflare.com/ajax/libs/phaser/3.0.0/phaser.js"></script>` `<script src="https://cdnjs.cloudflare.com/ajax/libs/phaser/2.7.1/phaser.js"></script>`
### Phaser Sandbox ### Phaser Sandbox
@ -305,7 +305,7 @@ You can read all about the philosophy behind Lazer [here](http://phaser.io/news/
![Change Log](http://phaser.io/images/github/div-change-log.png "Change Log") ![Change Log](http://phaser.io/images/github/div-change-log.png "Change Log")
<a name="change-log"></a> <a name="change-log"></a>
## Version 3.0.0 - "Shadow Coast" - In Development ## Version 2.7.1 - "Shadow Coast" - In Development
### New Features ### New Features
@ -362,6 +362,7 @@ You can read all about the philosophy behind Lazer [here](http://phaser.io/news/
* Text.width and Text.height now divide the result by the Text.resolution, to avoid incorrect dimensions on High DPI devices (thanks @mattahj #2146) * Text.width and Text.height now divide the result by the Text.resolution, to avoid incorrect dimensions on High DPI devices (thanks @mattahj #2146)
* If you called Video.changeSource, and then immediately called Video.play after it, it would fire the `onComplete` event twice (thanks @jaraiza #2543) * If you called Video.changeSource, and then immediately called Video.play after it, it would fire the `onComplete` event twice (thanks @jaraiza #2543)
* The Video.playing property didn't check to see if the Video existed, and would throw the error `Uncaught TypeError: Cannot read property 'paused' of null` if you called it after destroying the video (thanks @Tetley #2740) * The Video.playing property didn't check to see if the Video existed, and would throw the error `Uncaught TypeError: Cannot read property 'paused' of null` if you called it after destroying the video (thanks @Tetley #2740)
* Fixed bug in DisplayObject where it was using `PI_2` instead of `PI2`.
### Pixi Updates ### Pixi Updates
@ -417,10 +418,10 @@ All rights reserved.
[![Analytics](https://ga-beacon.appspot.com/UA-44006568-2/phaser/index)](https://github.com/igrigorik/ga-beacon) [![Analytics](https://ga-beacon.appspot.com/UA-44006568-2/phaser/index)](https://github.com/igrigorik/ga-beacon)
[get-js]: https://github.com/photonstorm/phaser/releases/download/v3.0.0/phaser.js [get-js]: https://github.com/photonstorm/phaser/releases/download/v2.7.1/phaser.js
[get-minjs]: https://github.com/photonstorm/phaser/releases/download/v3.0.0/phaser.min.js [get-minjs]: https://github.com/photonstorm/phaser/releases/download/v2.7.1/phaser.min.js
[get-zip]: https://github.com/photonstorm/phaser/archive/v3.0.0.zip [get-zip]: https://github.com/photonstorm/phaser/archive/v2.7.1.zip
[get-tgz]: https://github.com/photonstorm/phaser/archive/v3.0.0.tar.gz [get-tgz]: https://github.com/photonstorm/phaser/archive/v2.7.1.tar.gz
[clone-http]: https://github.com/photonstorm/phaser.git [clone-http]: https://github.com/photonstorm/phaser.git
[clone-ssh]: git@github.com:photonstorm/phaser.git [clone-ssh]: git@github.com:photonstorm/phaser.git
[clone-svn]: https://github.com/photonstorm/phaser [clone-svn]: https://github.com/photonstorm/phaser

View file

@ -331,7 +331,7 @@ PIXI.DisplayObject.prototype = {
var a, b, c, d, tx, ty; var a, b, c, d, tx, ty;
// so if rotation is between 0 then we can simplify the multiplication process.. // so if rotation is between 0 then we can simplify the multiplication process..
if (this.rotation % Phaser.Math.PI_2) if (this.rotation % Phaser.Math.PI2)
{ {
// check to see if the rotation is the same as the previous render. This means we only need to use sin and cos when rotation actually changes // check to see if the rotation is the same as the previous render. This means we only need to use sin and cos when rotation actually changes
if (this.rotation !== this.rotationCache) if (this.rotation !== this.rotationCache)