shibdib / Dramiel

src/plugins/onTick/notifications.php

Spacing +1 added lines, -1 removed lines

@@ -172,7 +172,7 @@
                 $this->fuelChannel = $this->toDiscordChannel;
             }
             //Set timer for next key based on number of keys
-            $nextKey = (1900 / (int)$this->numberOfKeys) + time();
+            $nextKey = (1900 / (int) $this->numberOfKeys) + time();
             $nextKeyTime = gmdate("Y-m-d H:i:s", $nextKey);
             setPermCache("notificationsLastChecked", $nextKey);
             //Set cache timer for api key

src/plugins/onTick/evemails.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
         $cacheClr = $baseUnix - 13500;
 
         //Set timer for next key based on number of keys
-        $nextKey = (1900 / (int)$this->numberOfKeys) + time();
+        $nextKey = (1900 / (int) $this->numberOfKeys) + time();
         $nextKeyTime = gmdate("Y-m-d H:i:s", $nextKey);
         setPermCache("mailLastChecked", $nextKey);
 

src/plugins/onMessage/auth.php

Spacing +5 added lines, -5 removed lines

@@ -196,27 +196,27 @@
                     $allianceContacts = getContacts($allianceID);
                     $corpContacts = getContacts($corpID);
                     foreach ($roles as $role) {
-                        if ((@(int)$allianceContacts['standings'] === 5 || @(int)$corpContacts['standings'] === 5) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['plus5Role']) {
+                        if ((@(int) $allianceContacts['standings'] === 5 || @(int) $corpContacts['standings'] === 5) && (string) $role->name === (string) $this->config['plugins']['auth']['standings']['plus5Role']) {
                             $member->addRole($role);
                             $role = 'blue';
                             break;
                         }
-                        if ((@(int)$allianceContacts['standings'] === 10 || @(int)$corpContacts['standings'] === 10) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['plus10Role']) {
+                        if ((@(int) $allianceContacts['standings'] === 10 || @(int) $corpContacts['standings'] === 10) && (string) $role->name === (string) $this->config['plugins']['auth']['standings']['plus10Role']) {
                             $member->addRole($role);
                             $role = 'blue';
                             break;
                         }
-                        if ((@(int)$allianceContacts['standings'] === 0 || @(int)$corpContacts['standings'] === 0 || (@(int)$allianceContacts['standings'] && @(int)$corpContacts['standings'] === null || '')) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['neutralRole']) {
+                        if ((@(int) $allianceContacts['standings'] === 0 || @(int) $corpContacts['standings'] === 0 || (@(int) $allianceContacts['standings'] && @(int) $corpContacts['standings'] === null || '')) && (string) $role->name === (string) $this->config['plugins']['auth']['standings']['neutralRole']) {
                             $member->addRole($role);
                             $role = 'neut';
                             break;
                         }
-                        if ((@(int)$allianceContacts['standings'] === -5 || @(int)$corpContacts['standings'] === -5) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['minus5Role']) {
+                        if ((@(int) $allianceContacts['standings'] === -5 || @(int) $corpContacts['standings'] === -5) && (string) $role->name === (string) $this->config['plugins']['auth']['standings']['minus5Role']) {
                             $member->addRole($role);
                             $role = 'red';
                             break;
                         }
-                        if ((@(int)$allianceContacts['standings'] === -10 || @(int)$corpContacts['standings'] === -10) && (string)$role->name === (string)$this->config['plugins']['auth']['standings']['minus10Role']) {
+                        if ((@(int) $allianceContacts['standings'] === -10 || @(int) $corpContacts['standings'] === -10) && (string) $role->name === (string) $this->config['plugins']['auth']['standings']['minus10Role']) {
                             $member->addRole($role);
                             $role = 'red';
                             break;

src/plugins/onTick/authCheck.php

Spacing +5 added lines, -5 removed lines

@@ -199,20 +199,20 @@
 
                 //check if user authed based on standings
                 $standings = null;
-                if ((string)$role === 'blue' || 'neut' || 'red') {
+                if ((string) $role === 'blue' || 'neut' || 'red') {
                     $allianceContacts = getContacts($allianceID);
                     $corpContacts = getContacts($corporationID);
-                    if ((string)$role === 'blue' && ($allianceContacts['standings'] || $corpContacts['standings'] === 5 || 10 || '5' || '10')) {
+                    if ((string) $role === 'blue' && ($allianceContacts['standings'] || $corpContacts['standings'] === 5 || 10 || '5' || '10')) {
                         $standings = 1;
                     }
-                    if ((string)$role === 'red' && ($allianceContacts['standings'] || $corpContacts['standings'] === -5 || -10 || '-5' || '-10')) {
+                    if ((string) $role === 'red' && ($allianceContacts['standings'] || $corpContacts['standings'] === -5 || -10 || '-5' || '-10')) {
                         $standings = 1;
                     }
-                    if ((string)$role === 'neut' && ($allianceContacts['standings'] || $corpContacts['standings'] === 0 || '0' || (@(int)$allianceContacts['standings'] && @(int)$corpContacts['standings'] === null || ''))) {
+                    if ((string) $role === 'neut' && ($allianceContacts['standings'] || $corpContacts['standings'] === 0 || '0' || (@(int) $allianceContacts['standings'] && @(int) $corpContacts['standings'] === null || ''))) {
                         $standings = 1;
                     }
                 }
-                if (!in_array((int)$allianceID, $allianceArray) && !in_array((int)$corporationID, $corpArray) && null === $standings) {
+                if (!in_array((int) $allianceID, $allianceArray) && !in_array((int) $corporationID, $corpArray) && null === $standings) {
                     // Deactivate user in database
                     $sql = "UPDATE authUsers SET active='no' WHERE discordID='$discordID'";
                     $this->logger->addInfo("AuthCheck: {$eveName} account has been deactivated as they are no longer in a correct corp/alliance.");

src/lib/eveApi.php

Doc Comments +12 added lines, -1 removed lines

@@ -28,7 +28,7 @@ 
 
 /**
  * @param $url
- * @return SimpleXMLElement|null|string
+ * @return SimpleXMLElement|null
  */
 function makeApiRequest($url)
 {
@@ -117,6 +117,11 @@ 
  * @return mixed
  */
 ////Character name to ID
+/**
+ * @param string $characterName
+ *
+ * @return string
+ */
 function characterID($characterName)
 {
     $logger = new Logger('eveESI');
@@ -197,6 +202,9 @@ 
  * @return mixed
  */
 ////Corp name to ID
+/**
+ * @param string $corpName
+ */
 function corpID($corpName)
 {
     $logger = new Logger('eveESI');
@@ -364,6 +372,9 @@ 
 }
 
 ////Char/Object ID to name via CCP
+/**
+ * @param string $moonID
+ */
 function apiMoonName($moonID)
 {
     $url = "https://api.eveonline.com/eve/CharacterName.xml.aspx?IDs={$moonID}";

Spacing +9 added lines, -9 removed lines

@@ -101,7 +101,7 @@ 
 function characterName($characterID)
 {
     $character = characterDetails($characterID);
-    $name = (string)$character['name'];
+    $name = (string) $character['name'];
     if (null === $name || '' === $name) { // Make sure it's always set.
         $url = "https://api.eveonline.com/eve/CharacterName.xml.aspx?ids={$characterID}";
         $xml = makeApiRequest($url);
@@ -127,7 +127,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $id = (int)$data['character'][0];
+        $id = (int) $data['character'][0];
 
     } catch (Exception $e) {
         $logger->error('EVE ESI Error: ' . $e->getMessage());
@@ -178,7 +178,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $name = (string)$data['solar_system_name'];
+        $name = (string) $data['solar_system_name'];
 
     } catch (Exception $e) {
         $logger->error('EVE ESI Error: ' . $e->getMessage());
@@ -207,7 +207,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $id = (int)$data['corporation'][0];
+        $id = (int) $data['corporation'][0];
 
     } catch (Exception $e) {
         $logger->error('EVE ESI Error: ' . $e->getMessage());
@@ -230,7 +230,7 @@ 
 function corpName($corpID)
 {
     $corporation = corpDetails($corpID);
-    $name = (string)$corporation['corporation_name'];
+    $name = (string) $corporation['corporation_name'];
     if (null === $name || '' === $name) { // Make sure it's always set.
         $url = "https://api.eveonline.com/eve/CharacterName.xml.aspx?ids={$corpID}";
         $xml = makeApiRequest($url);
@@ -277,7 +277,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $name = (string)$data['alliance_name'];
+        $name = (string) $data['alliance_name'];
 
     } catch (Exception $e) {
         $url = "https://api.eveonline.com/eve/CharacterName.xml.aspx?ids={$allianceID}";
@@ -303,7 +303,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $id = (int)$data['solarsystem'][0];
+        $id = (int) $data['solarsystem'][0];
 
     } catch (Exception $e) {
         $logger->error('EVE ESI Error: ' . $e->getMessage());
@@ -329,7 +329,7 @@ 
     try {
         $json = file_get_contents($url);
         $data = json_decode($json, TRUE);
-        $name = (string)$data['type_name'];
+        $name = (string) $data['type_name'];
 
     } catch (Exception $e) {
         $logger->error('EVE ESI Error: ' . $e->getMessage());
@@ -354,7 +354,7 @@ 
     $url = "https://www.fuzzwork.co.uk/api/typeid.php?typename={$typeName}";
     $json = file_get_contents($url);
     $data = json_decode($json, TRUE);
-    $id = (int)$data['typeID'];
+    $id = (int) $data['typeID'];
 
     if (null === $id) { // Make sure it's always set.
         $id = 'Unknown';

src/plugins/admin/setAvatar.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
             $roles = $member->roles;
             foreach ($roles as $role) {
                 if (in_array($role->name, $adminRoles, true)) {
-                    $avatarURL = strtolower((string)$data['messageString']);
+                    $avatarURL = strtolower((string) $data['messageString']);
                     $ch = curl_init($avatarURL);
                     if (substr($avatarURL, -4) === '.jpg') {
                         $fp = fopen('/tmp/avatar.jpg', 'wb');