diff --git a/src/applications/herald/action/HeraldAction.php b/src/applications/herald/action/HeraldAction.php
--- a/src/applications/herald/action/HeraldAction.php
+++ b/src/applications/herald/action/HeraldAction.php
@@ -298,10 +298,9 @@
         $no_permission[] = $phid;
         unset($targets[$phid]);
       }
-    }
-
-    if ($no_permission) {
-      $this->logEffect(self::DO_STANDARD_PERMISSION, $no_permission);
+      if ($no_permission) {
+        $this->logEffect(self::DO_STANDARD_PERMISSION, $no_permission);
+      }
     }
 
     return $targets;