fix(Vectorizer): fix normalizePathData() to support zero-length arc t…

…o curve (#2598)
clientIO · Mar 27, 2024 · 62bfd32 · 62bfd32
1 parent 4622900
commit 62bfd32
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/packages/joint-core/src/V/index.mjs b/packages/joint-core/src/V/index.mjs
@@ -2292,6 +2292,10 @@ const V = (function() {
                 var ry2 = ry * ry;
 
                 var k = ((large_arc_flag == sweep_flag) ? -1 : 1) * sqrt(abs(((rx2 * ry2) - (rx2 * y * y) - (ry2 * x * x)) / ((rx2 * y * y) + (ry2 * x * x))));
+                if (!Number.isFinite(k)) {
+                    // Arc is a single point
+                    return [x1, y1, x2, y2, x2, y2];
+                }
 
                 var cx = ((k * rx * y) / ry) + ((x1 + x2) / 2);
                 var cy = ((k * -ry * x) / rx) + ((y1 + y2) / 2);

diff --git a/packages/joint-core/test/vectorizer/vectorizer.js b/packages/joint-core/test/vectorizer/vectorizer.js
@@ -1486,6 +1486,8 @@ QUnit.module('vectorizer', function(hooks) {
             assert.equal(V.normalizePathData('A 3 0 0 0 1 10 15'), 'M 0 0 L 10 15'); // 0 y radius
             assert.equal(V.normalizePathData('A 0 0 0 0 1 10 15'), 'M 0 0 L 10 15'); // 0 x and y radii
 
+            assert.equal(V.normalizePathData('M 3 7 A 7 7 0 0 1 3 7'), 'M 3 7 C 3 7 3 7 3 7'); // arc corresponding to a single point
+
             // Make sure this does not throw an error because of recursion in a2c() exceeding the maximum stack size
             V.normalizePathData('M 0 0 A 1 1 0 1 0 -1 -1');
             V.normalizePathData('M 14.4 29.52 a .72 .72 0 1 0 -.72 -.72 A .72 .72 0 0 0 14.4 29.52Z');