Code cleanup

Author: Synchro

.scrutinizer.yml

@@ -1,5 +1,6 @@
 tools:
-    php_sim: true
+    php_sim:
+        min_mass: 36
     php_pdepend: true
     php_analyzer: true
     php_code_coverage: false

SmartmessagesAPI.class.php

@@ -912,8 +912,7 @@ public function deletetemplate($templateid)
      *      or an ISO-format UTC date ('yyyy-mm-dd hh:mm:ss')
      * @param boolean $continuous Is this a continuous mailshot? (never completes, existing subs are ignored,
      *      new subscriptions are sent a message immediately, ideal for 'welcome' messages)
-     * @param array $elements For future expansion; ignore for now
-     * @return integer of the new mailshot id, or false on failure
+     * @return integer|bool ID of the new mailshot id, or false on failure
      * @access public
      */
     public function sendmailshot(
@@ -926,8 +925,7 @@ public function sendmailshot(
         $fromname = '',
         $replyto = '',
         $when = 'now',
-        $continuous = false,
-        $elements = array()
+        $continuous = false
     ) {
         $res = $this->dorequest(
             'sendmailshot',
@@ -988,8 +986,8 @@ protected function dorequest(
         //Make the request (must have fopen wrappers enabled)
         if ($this->debug) {
             echo "<h1>$verb Request (" . htmlspecialchars($command) . "):</h1><p>" . htmlspecialchars(
-                    $url
-                ) . "</p>\n";
+                $url
+            ) . "</p>\n";
         }
         if ($post) {
             $response = $this->doPostRequest($url, $params, $files);