francis94c / ci-rest

models/RESTModel.php

Spacing +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
    * @param  string $apiKey [description]
    * @return array          [description]
    */
-  public function getAPIKeyData(string $apiKey):?array {
+  public function getAPIKeyData(string $apiKey): ?array {
     // Preliminary Check.
     if ($this->api_key_table == null || $this->api_key_column == null) return null;
     // Query.
@@ -120,7 +120,7 @@ 
    * @param  string $group  [description]
    * @return [type]         [description]
    */
-  public function getLimitData(string $client, string $group):?array {
+  public function getLimitData(string $client, string $group): ?array {
     $sql = 'SELECT count, start, (`start` + INTERVAL (1 - TIMESTAMPDIFF(HOUR, UTC_TIMESTAMP(), NOW())) HOUR) AS reset_epoch FROM rest_api_rate_limit WHERE client = ? AND _group = ?';
     $query = $this->db->query($sql, [$client, $group]);
     if (!is_scalar($query) && $query->num_rows() > 0) return $query->result_array()[0];

Braces +18 added lines, -6 removed lines

@@ -81,10 +81,14 @@ 
       $this->db->or_where($this->users_username_column, $username);
     }
     $query = $this->db->get();
-    if ($query->num_rows() == 0) return false;
+    if ($query->num_rows() == 0) {
+     return false;
+    }
     // Authenticate.
     if (password_verify($password, $query->result()[0]->{$this->users_password_column})) {
-      if ($this->users_id_column != null) $context->userId = $query->result()[0]->{$this->users_id_column};
+      if ($this->users_id_column != null) {
+       $context->userId = $query->result()[0]->{$this->users_id_column};
+      }
       return true;
     }
     return false;
@@ -96,15 +100,21 @@ 
    */
   public function getAPIKeyData(string $apiKey):?array {
     // Preliminary Check.
-    if ($this->api_key_table == null || $this->api_key_column == null) return null;
+    if ($this->api_key_table == null || $this->api_key_column == null) {
+     return null;
+    }
     // Query.
     $this->db->select($this->api_key_column);
-    if ($this->api_key_limit_column != null) $this->db->select($this->api_key_limit_column);
+    if ($this->api_key_limit_column != null) {
+     $this->db->select($this->api_key_limit_column);
+    }
     $this->db->from($this->api_key_table);
     $this->db->where($this->api_key_column, $apiKey);
     $query = $this->db->get();
     // Process Result.
-    if ($query->num_rows() > 0) return $query->result_array()[0];
+    if ($query->num_rows() > 0) {
+     return $query->result_array()[0];
+    }
     return null;
   }
   /**
@@ -123,7 +133,9 @@ 
   public function getLimitData(string $client, string $group):?array {
     $sql = 'SELECT count, start, (`start` + INTERVAL (1 - TIMESTAMPDIFF(HOUR, UTC_TIMESTAMP(), NOW())) HOUR) AS reset_epoch FROM rest_api_rate_limit WHERE client = ? AND _group = ?';
     $query = $this->db->query($sql, [$client, $group]);
-    if (!is_scalar($query) && $query->num_rows() > 0) return $query->result_array()[0];
+    if (!is_scalar($query) && $query->num_rows() > 0) {
+     return $query->result_array()[0];
+    }
     return null;
   }
   /**

phpunit/RESTTest.php

Spacing +4 added lines, -4 removed lines

@@ -24,10 +24,10 @@ 
    */
   public static function setUpBeforeClass(): void {
     echo "IN TEST";
-    self::$ci =& get_instance();
+    self::$ci = & get_instance();
     self::$ci->load->database('mysqli://root@localhost/test_db');
     self::$ci->load->helper("url");
-    $queries = explode("#@@@", file_get_contents(FCPATH . 'application/splints/' . self::PACKAGE . '/phpunit/database.sql'));
+    $queries = explode("#@@@", file_get_contents(FCPATH.'application/splints/'.self::PACKAGE.'/phpunit/database.sql'));
     self::assertTrue(count($queries) > 0);
     self::$ci->load->database();
     foreach ($queries as $query) {
@@ -43,7 +43,7 @@ 
     // However, for the purpose of this test, we are going to Hack Code CodeIgniter
     // with a Splint Config variable to allow us load config files from where
     // ever we want. This happens below.
-    self::$ci->load->add_package_path(APPPATH . 'splints/' . self::PACKAGE . "/phpunit/");
+    self::$ci->load->add_package_path(APPPATH.'splints/'.self::PACKAGE."/phpunit/");
     //self::$ci->config->set_item('st_config_path_prefix', '../splints/' . self::PACKAGE . "/phpunit/config/");
   }
   /**
@@ -55,7 +55,7 @@ 
     // Simulate Basic Authorization
     $_SERVER['PHP_AUTH_USER'] = "francis94c";
     $_SERVER['PHP_AUTH_PW'] = "0123456789";
-    self::$ci->load->splint(self::PACKAGE, '+REST', null, 'rest_' . $this->obj_count++);
+    self::$ci->load->splint(self::PACKAGE, '+REST', null, 'rest_'.$this->obj_count++);
     $this->assertEquals(1, self::$ci->{'rest_'.($this->obj_count - 1)}->userId);
   }
   /**

phpunit/config/rest.php

Spacing +4 added lines, -4 removed lines

@@ -9,19 +9,19 @@
 
 $config['auth_callbacks'] = [
 
-  RESTAuth::CUSTOM('X-APP-ID')    => function (&$context, $value):bool {
+  RESTAuth::CUSTOM('X-APP-ID')    => function(&$context, $value):bool {
     return true;
   },
 
-  RESTAuth::CUSTOM('X-DEVICE-ID') => function (&$context, $value):bool {
+  RESTAuth::CUSTOM('X-DEVICE-ID') => function(&$context, $value):bool {
     return true;
   },
 
-  RESTAuth::BEARER                => function (&$context, $token):bool {
+  RESTAuth::BEARER                => function(&$context, $token):bool {
     return true;
   },
 
-  RESTAuth::OAUTH2                => function (&$context, $token):bool {
+  RESTAuth::OAUTH2                => function(&$context, $token):bool {
     return true;
   }