From 2b3d3852e2a59417dd199192f6ebf6aabdabf602 Mon Sep 17 00:00:00 2001 From: Robert Kowalski Date: Fri, 17 Dec 2021 16:05:25 -0500 Subject: [PATCH] Much cleaner fix. --- src/geometry/Vertices.js | 28 ++++++---------------------- 1 file changed, 6 insertions(+), 22 deletions(-) diff --git a/src/geometry/Vertices.js b/src/geometry/Vertices.js index 33b4c89..9ce840d 100644 --- a/src/geometry/Vertices.js +++ b/src/geometry/Vertices.js @@ -244,30 +244,14 @@ var Common = require('../core/Common'); point = point || Vertices.centre(vertices); - var vertex, - delta; - // Reverse the order from counterclockwise to clockwise when scaleX or scaleY is negative, but not both. - if (scaleX < 0 && scaleY > 0 || scaleX > 0 && scaleY < 0) { - var pastVertices = []; + if (scaleX < 0 && scaleY > 0 || scaleX > 0 && scaleY < 0) + vertices.reverse(); - for (var i = 0, len = vertices.length; i < len; i++) - pastVertices[i] = { x: vertices[i].x, y: vertices[i].y }; - - for (var i = 0, j = pastVertices.length - 1, len = pastVertices.length; i < len; i++) { - j = pastVertices.length - i - 1; - vertex = pastVertices[j]; - delta = Vector.sub(vertex, point); - vertices[i].x = point.x + delta.x * scaleX; - vertices[i].y = point.y + delta.y * scaleY; - } - } else { - for (var i = 0, len = vertices.length; i < len; i++) { - vertex = vertices[i]; - delta = Vector.sub(vertex, point); - vertices[i].x = point.x + delta.x * scaleX; - vertices[i].y = point.y + delta.y * scaleY; - } + for (var i = 0, delta; i < vertices.length; i++) { + delta = Vector.sub(vertices[i], point); + vertices[i].x = point.x + delta.x * scaleX; + vertices[i].y = point.y + delta.y * scaleY; } return vertices;