Merge pull request #326 from Art4/add-phpstan

Add phpstan

Author: Art4

.github/workflows/tests.yml

@@ -36,6 +36,10 @@ jobs:
       - name: "Install Composer dependencies"
         uses: "ramsey/composer-install@v2"
 
+      - name: Run static code analysis
+        if: ${{ matrix.php-versions == '8.2' }}
+        run: composer run phpstan -- --error-format=github
+
       - name: Run tests
         run: vendor/bin/phpunit --coverage-clover .phpunit.cache/clover.xml
 

.phpstan.neon

@@ -0,0 +1,9 @@
+parameters:
+    level: 4
+
+    paths:
+        - src/
+        - tests/
+
+    scanDirectories:
+        - vendor

composer.json

@@ -32,7 +32,8 @@
         "friendsofphp/php-cs-fixer": "^3",
         "phpunit/phpunit": "^9 || 10.2.*",
         "guzzlehttp/psr7": "^2",
-        "php-mock/php-mock-phpunit": "^2.6"
+        "php-mock/php-mock-phpunit": "^2.6",
+        "phpstan/phpstan": "^1.10"
     },
     "autoload": {
         "psr-4": {
@@ -46,6 +47,7 @@
     },
     "scripts": {
         "coverage": "phpunit --coverage-html=\".phpunit.cache/code-coverage\"",
+        "phpstan": "phpstan analyze --memory-limit 512M --configuration .phpstan.neon",
         "test": "phpunit"
     }
 }

src/Redmine/Api/Group.php

@@ -63,7 +63,7 @@ public function listing($forceUpdate = false)
      *
      * @throws MissingParameterException Missing mandatory parameters
      *
-     * @return \SimpleXMLElement
+     * @return string|false
      */
     public function create(array $params = [])
     {

src/Redmine/Api/Issue.php

@@ -76,7 +76,7 @@ public function show($id, array $params = [])
      *
      * @param array $params the new issue data
      *
-     * @return \SimpleXMLElement
+     * @return string|false
      */
     public function create(array $params = [])
     {
@@ -174,6 +174,7 @@ public function removeWatcher($id, $watcherUserId)
      */
     public function setIssueStatus($id, $status)
     {
+        /** @var IssueStatus */
         $api = $this->client->getApi('issue_status');
 
         return $this->update($id, [
@@ -204,31 +205,37 @@ public function addNoteToIssue($id, $note, $privateNote = false)
     private function cleanParams(array $params = [])
     {
         if (isset($params['project'])) {
+            /** @var Project */
             $apiProject = $this->client->getApi('project');
             $params['project_id'] = $apiProject->getIdByName($params['project']);
             unset($params['project']);
             if (isset($params['category'])) {
+                /** @var IssueCategory */
                 $apiIssueCategory = $this->client->getApi('issue_category');
                 $params['category_id'] = $apiIssueCategory->getIdByName($params['project_id'], $params['category']);
                 unset($params['category']);
             }
         }
         if (isset($params['status'])) {
+            /** @var IssueStatus */
             $apiIssueStatus = $this->client->getApi('issue_status');
             $params['status_id'] = $apiIssueStatus->getIdByName($params['status']);
             unset($params['status']);
         }
         if (isset($params['tracker'])) {
+            /** @var Tracker */
             $apiTracker = $this->client->getApi('tracker');
             $params['tracker_id'] = $apiTracker->getIdByName($params['tracker']);
             unset($params['tracker']);
         }
         if (isset($params['assigned_to'])) {
+            /** @var User */
             $apiUser = $this->client->getApi('user');
             $params['assigned_to_id'] = $apiUser->getIdByUsername($params['assigned_to']);
             unset($params['assigned_to']);
         }
         if (isset($params['author'])) {
+            /** @var User */
             $apiUser = $this->client->getApi('user');
             $params['author_id'] = $apiUser->getIdByUsername($params['author']);
             unset($params['author']);

src/Redmine/Api/Membership.php

@@ -121,7 +121,7 @@ public function removeMember($projectId, $userId, array $params = [])
     {
         $memberships = $this->all($projectId, $params);
         if (!isset($memberships['memberships']) || !is_array($memberships['memberships'])) {
-            return;
+            return false;
         }
         $removed = false;
         foreach ($memberships['memberships'] as $membership) {

src/Redmine/Api/Project.php

@@ -106,7 +106,7 @@ public function show($id, array $params = [])
      *
      * @throws MissingParameterException
      *
-     * @return \SimpleXMLElement
+     * @return string|false
      */
     public function create(array $params = [])
     {

src/Redmine/Client/NativeCurlClient.php

@@ -4,7 +4,6 @@
 
 namespace Redmine\Client;
 
-use Redmine\Api;
 use Redmine\Exception\ClientException;
 
 /**
@@ -246,7 +245,9 @@ private function request(string $method, string $path, string $body = ''): bool
     /**
      * Prepare the request by setting the cURL options.
      *
-     * @return resource a cURL handle on success, <b>FALSE</b> on errors
+     * BC for PHP 7.4: Do not add the return type because CurlHandle was introduced in PHP 8.0
+     *
+     * @return \CurlHandle a cURL handle on success, <b>FALSE</b> on errors
      */
     private function createCurl(string $method, string $path, string $body = '')
     {
@@ -281,13 +282,13 @@ private function createCurl(string $method, string $path, string $body = '')
                     $curlOptions[CURLOPT_POSTFIELDS] = $filedata;
                     $curlOptions[CURLOPT_INFILE] = $file;
                     $curlOptions[CURLOPT_INFILESIZE] = $size;
-                } elseif (isset($body)) {
+                } elseif ($body !== '') {
                     $curlOptions[CURLOPT_POSTFIELDS] = $body;
                 }
                 break;
             case 'put':
                 $curlOptions[CURLOPT_CUSTOMREQUEST] = 'PUT';
-                if (isset($body)) {
+                if ($body !== '') {
                     $curlOptions[CURLOPT_POSTFIELDS] = $body;
                 }
                 break;

tests/Fixtures/MockClient.php

@@ -12,10 +12,15 @@
  * The runRequest method of this client class just returns the value of
  * the path, method and data or the $runRequestReturnValue value if set.
  */
-class MockClient implements Client
+final class MockClient implements Client
 {
     use ClientApiTrait;
 
+    public static function create()
+    {
+        return new self();
+    }
+
     /**
      * Return value the mocked runRequest method should return.
      *
@@ -34,22 +39,7 @@ class MockClient implements Client
     public $responseCodeMock;
     public $responseContentTypeMock;
 
-    private string $url;
-    private string $apikeyOrUsername;
-    private ?string $password;
-
-    /**
-     * $apikeyOrUsername should be your ApiKey, but it could also be your username.
-     * $password needs to be set if a username is given (not recommended).
-     */
-    public function __construct(
-        string $url,
-        string $apikeyOrUsername,
-        string $password = null
-    ) {
-        $this->url = $url;
-        $this->apikeyOrUsername = $apikeyOrUsername;
-        $this->password = $password;
+    private function __construct() {
     }
 
     /**

tests/Integration/GroupXmlTest.php

@@ -9,19 +9,10 @@
 
 class GroupXmlTest extends TestCase
 {
-    /**
-     * @var MockClient
-     */
-    private $client;
-
-    public function setup(): void
-    {
-        $this->client = new MockClient('http://test.local', 'asdf');
-    }
-
     public function testCreateBlank()
     {
-        $api = $this->client->getApi('group');
+        /** @var \Redmine\Api\Group */
+        $api = MockClient::create()->getApi('group');
         $this->assertInstanceOf('Redmine\Api\Group', $api);
 
         $this->expectException(MissingParameterException::class);
@@ -32,7 +23,9 @@ public function testCreateBlank()
 
     public function testCreateComplex()
     {
-        $res = $this->client->getApi('group')->create([
+        /** @var \Redmine\Api\Group */
+        $api = MockClient::create()->getApi('group');
+        $res = $api->create([
             'name' => 'Developers',
             'user_ids' => [3, 5],
         ]);
@@ -57,7 +50,8 @@ public function testCreateComplex()
 
     public function testUpdateNotImplemented()
     {
-        $api = $this->client->getApi('group');
+        /** @var \Redmine\Api\Group */
+        $api = MockClient::create()->getApi('group');
         $this->assertInstanceOf('Redmine\Api\Group', $api);
 
         $this->expectException(Exception::class);