diff --git a/src/applications/auth/xaction/PhabricatorAuthFactorProviderNameTransaction.php b/src/applications/auth/xaction/PhabricatorAuthFactorProviderNameTransaction.php
--- a/src/applications/auth/xaction/PhabricatorAuthFactorProviderNameTransaction.php
+++ b/src/applications/auth/xaction/PhabricatorAuthFactorProviderNameTransaction.php
@@ -17,12 +17,12 @@
     $old = $this->getOldValue();
     $new = $this->getNewValue();
 
-    if (!strlen($old)) {
+    if (!phutil_nonempty_string($old)) {
       return pht(
         '%s named this provider %s.',
         $this->renderAuthor(),
         $this->renderNewValue());
-    } else if (!strlen($new)) {
+    } else if (!phutil_nonempty_string($new)) {
       return pht(
         '%s removed the name (%s) of this provider.',
         $this->renderAuthor(),