Moves logic to bottom

Author: deadlybutter

lib/modules/dosomething/dosomething_signup/dosomething_signup.module

@@ -243,13 +243,6 @@ function dosomething_signup_create($nid, $uid = NULL, $source = NULL, $timestamp
     return FALSE;
   }
 
-  if (module_exists('stathat')) {
-    // The source is often a string longer than just "reportback", this logic accounts for that.
-    if (strpos($source, 'reportback') === 0) {
-      stathat_send_ez_count('drupal - Signup - permalink signup', 1);
-    }
-  }
-
   $values = array(
     'nid' => $nid,
     'uid' => $uid,
@@ -264,6 +257,14 @@ function dosomething_signup_create($nid, $uid = NULL, $source = NULL, $timestamp
     // The SignupEntityController save method handles any NULL values.
     $entity->save();
     if (isset($entity->sid)) {
+
+      if (module_exists('stathat')) {
+        // The source is often a string longer than just "reportback", but always starts with it if the page is a permalink
+        if (strpos($source, 'reportback') === 0) {
+          stathat_send_ez_count('drupal - Signup - permalink signup', 1);
+        }
+      }
+
       return $entity->sid;
     }
     return FALSE;