Merge pull request #47 from benpickles/zero-sum-pies

Don’t blow up if a pie/donut’s sum is less than zero.
benpickles · Oct 22, 2014 · d99db2c · d99db2c
2 parents 5fe8a9e + 9bdc79c
commit d99db2c
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 0 deletions.
diff --git a/jquery.peity.js b/jquery.peity.js
@@ -124,6 +124,12 @@
         sum += values[i]
       }
 
+      if (!sum) {
+        length = 2
+        sum = 1
+        values = [0, 1]
+      }
+
       var diameter = opts.radius * 2
         , width = opts.width || diameter
         , height = opts.height || diameter

diff --git a/test/charts.json b/test/charts.json
@@ -175,6 +175,12 @@
     "width": 16,
     "height": 16
   },
+  "donut12": {
+    "type": "donut",
+    "text": "0,0,0,0,0",
+    "width": 16,
+    "height": 16
+  },
   "line1": {
     "type": "line",
     "text": "5,3,9,6,5,9,7,3,5,2",
@@ -328,5 +334,11 @@
     "text": "-2/3",
     "width": 16,
     "height": 16
+  },
+  "pie11": {
+    "type": "pie",
+    "text": "0,0,0,0,0",
+    "width": 16,
+    "height": 16
   }
 }
diff --git a/test/fixtures/donut12.png b/test/fixtures/donut12.png
diff --git a/test/fixtures/pie11.png b/test/fixtures/pie11.png