- Hide Total Counts button if empty and "Hide Empty Counts" setting i…

…s enabled, closes #44
jaredatch · Apr 18, 2019 · a7bd220 · a7bd220
1 parent 0170abd
commit a7bd220
Show file tree

Hide file tree

Showing 4 changed files with 142 additions and 130 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -14,6 +14,7 @@ All notable changes to this project will be documented in this file, formatted v
 - Pass post_id to `needs_updating` method.
 - Removed support for Google+ and StumbleUpon (RIP).
 - Default `letter-spacing` to normal on button labels.
+- Hide Total Counts button if empty and "Hide Empty Counts" setting is enabled.
 
 ### Fixed
 - Pinterest JS API conflict.

diff --git a/includes/class-shared-counts-front.php b/includes/class-shared-counts-front.php
@@ -647,6 +647,12 @@ public function link( $types = 'facebook', $id = false, $echo = true, $style = '
 				$show_count = false;
 			}
 
+			// Hide Total Counts button if empty and "Hide Empty Counts" setting
+			// is enabled.
+			if ( 'included_total' === $type && 0 === absint( $link['count'] ) && 'false' === $show_empty ) {
+				continue;
+			}
+
 			// Build button output.
 			if ( 'included_total' === $type ) {
 				$elements['wrap_open']  = sprintf(