cs fixes

Karaka-Management · Sep 21, 2023 · 1775636 · 1775636
1 parent 5babc18
commit 1775636
Show file tree

Hide file tree

Showing 7 changed files with 117 additions and 5 deletions.
diff --git a/Algorithm/Rating/Glicko2.php b/Algorithm/Rating/Glicko2.php
@@ -121,7 +121,7 @@ public function rating(
 
         // Step 0:
         $rdOld /= self::Q;
-        $elo   = ($elo - $this->DEFAULT_ELO) / self::Q;
+        $elo    = ($elo - $this->DEFAULT_ELO) / self::Q;
 
         foreach ($oElo as $idx => $value) {
             $oElo[$idx] = ($value - $this->DEFAULT_ELO) / self::Q;

diff --git a/DataStorage/Database/Mapper/DeleteMapper.php b/DataStorage/Database/Mapper/DeleteMapper.php
@@ -171,7 +171,7 @@ private function deleteHasMany(\ReflectionClass $refClass, object $obj, mixed $o
 
             $objIds  = [];
             $refProp = $refClass->getProperty($member);
-            $values = $refProp->isPublic() ? $obj->{$member} : $refProp->getValue($obj);
+            $values  = $refProp->isPublic() ? $obj->{$member} : $refProp->getValue($obj);
 
             if (!\is_array($values)) {
                 // conditionals

diff --git a/DataStorage/Database/Mapper/WriteMapper.php b/DataStorage/Database/Mapper/WriteMapper.php
@@ -262,7 +262,7 @@ private function createHasMany(\ReflectionClass $refClass, object $obj, mixed $o
             }
 
             $property = $refClass->getProperty($propertyName);
-            $values = $property->isPublic() ? $obj->{$propertyName} : $property->getValue($obj);
+            $values   = $property->isPublic() ? $obj->{$propertyName} : $property->getValue($obj);
 
             /** @var class-string<DataMapperFactory> $mapper */
             $mapper       = $this->mapper::HAS_MANY[$propertyName]['mapper'];

diff --git a/Message/Cli/CliHeader.php b/Message/Cli/CliHeader.php
@@ -153,9 +153,15 @@ public function has(string $key) : bool
      */
     public function generate(int $code) : void
     {
-        $this->generate500();
+        switch ($code) {
+            default:
+                $this->generate500();
+        }
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function getRequestTime() : int
     {
         return $this->timestamp;

diff --git a/Message/HeaderAbstract.php b/Message/HeaderAbstract.php
@@ -150,6 +150,13 @@ abstract public function get(string $key = null) : array;
      */
     abstract public function has(string $key) : bool;
 
+    /**
+     * Get time of the request
+     *
+     * @return int
+     *
+     * @since 1.0.0
+     */
     abstract public function getRequestTime() : int;
 
     /**

diff --git a/Message/Http/HttpHeader.php b/Message/Http/HttpHeader.php
@@ -145,22 +145,47 @@ public function getProtocolVersion() : string
         return $_SERVER['SERVER_PROTOCOL'] ?? 'HTTP/1.1';
     }
 
+    /**
+     * Get the referer link
+     *
+     * @return string
+     *
+     * @since 1.0.0
+     */
     public function getReferer() : string
     {
         return $_SERVER['HTTP_REFERER'] ?? '';
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function getRequestTime() : int
     {
         return (int) ($_SERVER['REQUEST_TIME'] ?? $this->timestamp);
     }
 
+    /**
+     * Get the ip of the requester
+     *
+     * @return string
+     *
+     * @since 1.0.0
+     */
     public function getRequestIp() : string
     {
         return $_SERVER['HTTP_X_FORWARDED_FOR'] ?? $_SERVER['REMOTE_ADDR'] ?? '';
     }
 
-    public function getBrowserName(): string {
+    /**
+     * Get the browser/agent name of the request
+     *
+     * @return string
+     *
+     * @since 1.0.0
+     */
+    public function getBrowserName(): string
+    {
         $userAgent = $_SERVER['HTTP_USER_AGENT'];
 
         if (\strpos($userAgent, 'Opera') !== false || \strpos($userAgent, 'OPR/') !== false) {

diff --git a/Message/Http/ImpressionStat.php b/Message/Http/ImpressionStat.php
@@ -27,26 +27,93 @@
  */
 class ImpressionStat
 {
+    /**
+     * Request ip
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $address = '';
 
+    /**
+     * Request language
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $language = ISO639x1Enum::_EN;
 
+    /**
+     * Request country
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $country = ISO3166TwoEnum::_XXX;
 
+    /**
+     * Request time
+     *
+     * @var \DateTime
+     * @since 1.0.0
+     */
     public \DateTime $datetime;
 
+    /**
+     * Request host/domain
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $host = '';
 
+    /**
+     * Request path
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $path = '';
 
+    /**
+     * Request full uri
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $uri = '';
 
+    /**
+     * Request referer link
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $referer = '';
 
+    /**
+     * Request browser/agent
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public string $userAgent = '';
 
+    /**
+     * Additional custom meta data to be stored
+     *
+     * @var string
+     * @since 1.0.0
+     */
     public array $meta = [];
 
+    /**
+     * Constructor.
+     *
+     * @param HttpRequest $request Http request object
+     *
+     * @since 1.0.0
+     */
     public function __construct(HttpRequest $request)
     {
         $this->language  = $request->header->l11n->language;
@@ -60,6 +127,13 @@ public function __construct(HttpRequest $request)
         $this->userAgent = $request->header->getBrowserName();
     }
 
+    /**
+     * Turn object to array
+     *
+     * @return array
+     *
+     * @since 1.0.0
+     */
     public function toArray() : array
     {
         return [