Fixed errant operator

+ Renamed the `getRawFile()` method to `getFileContents`
+ Updated the method to make it use the `/repos/{owner}/{repo}/contents/{filepath}` API route

src/Api/Repositories.php

@@ -112,28 +112,73 @@ class Repositories extends AbstractAllApiRequester
     }
 
     /**
-     * Get the raw contents of a file stored inside a repository
+     * Get a repository using its ID
+     *
+     * Example:
+     * ```
+     * $client->repositories()->getByID($repoId);
+     * ```
+     *
+     * @param string repoId The ID of the repository
+     * @return Repository
+     */
+    public function getById(int $repoId)
+    {
+        $client = $this->getClient();
+        try {
+            $response = $this->get("repositories/$repoId");
+            $statusCode = $response->getStatusCode();
+            $body = (string) $response->getBody();
+            if ($statusCode == 200) {
+                return Repository::fromJson(
+                    $client,
+                    $this,
+                    json_decode($body)
+                );
+            }
+            return false;
+
+        } catch (ServerException $serverError) {
+            return false;
+        }
+    }
+
+    /**
+     * Get the contents of a file in a certain in repository commit/branch/tag
      * using the repository's name and owner
      *
      * Example:
      * ```
-     * $client->repositories()->getRawFile($owner, $repoName, "README.md");
+     * $client->repositories()->getRawFile($owner, $repoName, "README.md", "v2.0.0");
      * ```
      *
      * @param string $owner The owner of the repository
      * @param string $repoName The name of the repository
-     * @param string $filepath The path to the raw file (relative to the repository root)
+     * @param string $filepath The path to the file (relative to the repository root)
+     * @param string $ref The name of the commit/branch/tag. Default the repository’s default branch (usually master)
      * @return string
      */
-    public function getRawFile(string $owner, string $repoName, string $filepath)
+    public function getFileContents(string $owner, string $repoName, string $filepath, string $ref="")
     {
         $client = $this->getClient();
         try {
-            $response = $this->get("repos/$owner/$repoName/raw/$filepath");
+            if ($ref !== "") {
+                $response = $this->get("repos/$owner/$repoName/contents/$filepath",[
+                    "ref" => $ref
+                ]);
+            } else {
+                $response = $this->get("repos/$owner/$repoName/contents/$filepath");
+            }
+
             $statusCode = $response->getStatusCode();
-            $body = (string) $response->getBody();
             if ($statusCode == 200) {
-                return $body;
+                $body = (string) $response->getBody();
+                $jsonObj = json_decode($body, true);
+                if (array_key_exists("content", $jsonObj)) {
+                    $base64FileContents = $jsonObj["content"];
+                    $fileContents = base64_decode($base64FileContents);
+                    return $fileContents;
+                }
             }
             return false;
 

src/Client.php

@@ -66,7 +66,7 @@ class Client implements RequestChainableInterface
     {
         // Append a slash to any URL that doesn't end in '/'
         if (!$this->endsWith($giteaURL, '/')) {
-            $giteaURL += "/";
+            $giteaURL .= "/";
         }
         $this->giteaURL = $giteaURL;
         $this->authToken = $authToken;

src/Model/Tag.php

@@ -44,9 +44,9 @@ class Tag extends AbstractApiModel {
         if (count($args) >= 2) {
             $id = $args[0];
             $name = $args[1];
-            if (!is_numeric($id)) {
+            if (!is_string($id)) {
                 $argumentType = gettype($id);
-                throw new InvalidArgumentException("The \"__construct()\" function requires the 3rd parameter to be of the integer type, but a \"$argumentType\" was passed in");
+                throw new InvalidArgumentException("The \"__construct()\" function requires the 3rd parameter to be of the string type, but a \"$argumentType\" was passed in");
             }
             if (!is_string($name)) {
                 $argumentType = gettype($name);
@@ -71,7 +71,7 @@ class Tag extends AbstractApiModel {
         $newTag = new static(
             $client,
             $caller,
-            isset($map->id) && is_numeric($map->id) ? $map->id : -1,
+            isset($map->id) && is_string($map->id) ? $map->id : -1,
             isset($map->name) && is_string($map->name) ? $map->name : ''
         );
         $newTag->setTarballURL(isset($map->tarball_url) && is_string($map->tarball_url) ? new Uri($map->tarball_url) : null);
@@ -104,7 +104,7 @@ class Tag extends AbstractApiModel {
      * Gets the tag identifier.
      * @return int The tag identifier.
      */
-    function getId(): int {
+    function getId(): string {
         return $this->id;
     }