Merge pull request #35 from launchdarkly/dr/allowBlankUserKey

Allow blank user key in evaluation.

Author: drichelson

src/LaunchDarkly/LDClient.php

@@ -111,11 +111,14 @@ public function variation($key, $user, $default = false) {
         }
 
         try {
-            if (is_null($user) || strlen($user->getKey()) === 0) {
+            if (is_null($user) || is_null($user->getKey()) ) {
                 $this->_sendFlagRequestEvent($key, $user, $default, $default);
-                $this->_logger->warn("Variation called with null user or null/empty user key! Returning default value");
+                $this->_logger->warn("Variation called with null user or null user key! Returning default value");
                 return $default;
             }
+            if (strlen($user->getKey()) === 0) {
+                $this->_logger->warn("User key is blank. Flag evaluation will proceed, but the user will not be stored in LaunchDarkly.");
+            }
             $flag = $this->_featureRequester->get($key);
 
             if (is_null($flag)) {