Refactored savings percentage calculation.

Author: Evertrotshuizen

src/class-tiny-image.php

@@ -452,6 +452,12 @@ public static function get_optimization_statistics( $settings, $result = null )
 				);
 			}
 		}
+
+		if ( 0 != $stats['unoptimized-library-size'] ) {
+			$stats['display-percentage'] = round( 100 - ( $stats['optimized-library-size'] / $stats['unoptimized-library-size'] * 100 ), 1 );
+		} else {
+			$stats['display-percentage'] = 0;
+		}
 		return $stats;
 	}
 

src/js/dashboard-widget.js

@@ -52,7 +52,7 @@
 
   function renderWidget(data, container) {
     var stats = jQuery.parseJSON(data);
-    var savings = savingsPercentage(stats);
+    var savings = stats['display-percentage'];
     var libraryOptimized = optimizedPercentage(stats);
     renderContent(libraryOptimized, stats, savings);
     renderChart(savings);
@@ -98,14 +98,6 @@
     }
   }
 
-  function savingsPercentage(stats) {
-    if ( 0 != stats['unoptimized-library-size'] ) {
-      return 100 - Math.round((stats['optimized-library-size'] / stats['unoptimized-library-size'] * 1000))/ 10;
-    } else {
-      return 0
-    }
-  }
-
   function renderChart(savingsPercentage) {
     chart = chartOptions(savingsPercentage);
     jQuery('#optimization-chart svg circle.main').css('stroke-dasharray', chart['dash-array-size'] + ' ' + chart['circle-size'])

src/views/bulk-optimization-chart.php

@@ -3,11 +3,7 @@
 $chart = array();
 
 if ( isset( $stats ) ) {
-	if ( 0 != $stats['unoptimized-library-size'] ) {
-		$chart['percentage'] = round( 100 - ( $stats['optimized-library-size'] / $stats['unoptimized-library-size'] * 100 ), 1 );
-	} else {
-		$chart['percentage'] = 0;
-	}
+		$chart['percentage'] = $stats['display-percentage'];
 } else {
 	$chart['percentage'] = 0;
 };