Merge pull request #9183 from nupplaphil/Minor-deprecation

Fix composer notices
2020-09-12 07:59:50 -04:00 · 2020-09-12 07:59:50 -04:00 · 28864d8fdd
parent b63b024f3f 61014feec3
commit 28864d8fdd
13 changed files with 74 additions and 65 deletions
--- a/composer.json
+++ b/composer.json
@ -128,7 +128,7 @@
 		"mikey179/vfsstream": "^1.6",
 		"mockery/mockery": "^1.2",
 		"johnkary/phpunit-speedtrap": "1.1",
-		"jakub-onderka/php-parallel-lint": "^1.0"
+		"php-parallel-lint/php-parallel-lint": "^1.2"
 	},
 	"scripts": {
 		"test": "phpunit"
--- a/composer.lock
+++ b/composer.lock
@ -4,7 +4,7 @@
        "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
        "This file is @generated automatically"
    ],
-    "content-hash": "7d1fe40c28d815b56d0b5cb323860b26",
+    "content-hash": "27203cc7da01ff668ba15e03fd17dc94",
    "packages": [
        {
            "name": "asika/simple-console",
@ -3382,55 +3382,6 @@
            ],
            "time": "2016-01-20T08:20:44+00:00"
        },
-        {
-            "name": "jakub-onderka/php-parallel-lint",
-            "version": "v1.0.0",
-            "source": {
-                "type": "git",
-                "url": "https://github.com/JakubOnderka/PHP-Parallel-Lint.git",
-                "reference": "04fbd3f5fb1c83f08724aa58a23db90bd9086ee8"
-            },
-            "dist": {
-                "type": "zip",
-                "url": "https://api.github.com/repos/JakubOnderka/PHP-Parallel-Lint/zipball/04fbd3f5fb1c83f08724aa58a23db90bd9086ee8",
-                "reference": "04fbd3f5fb1c83f08724aa58a23db90bd9086ee8",
-                "shasum": ""
-            },
-            "require": {
-                "php": ">=5.3.3"
-            },
-            "require-dev": {
-                "jakub-onderka/php-console-highlighter": "~0.3",
-                "nette/tester": "~1.3",
-                "squizlabs/php_codesniffer": "~2.7"
-            },
-            "suggest": {
-                "jakub-onderka/php-console-highlighter": "Highlight syntax in code snippet"
-            },
-            "bin": [
-                "parallel-lint"
-            ],
-            "type": "library",
-            "autoload": {
-                "classmap": [
-                    "./"
-                ]
-            },
-            "notification-url": "https://packagist.org/downloads/",
-            "license": [
-                "BSD-2-Clause"
-            ],
-            "authors": [
-                {
-                    "name": "Jakub Onderka",
-                    "email": "ahoj@jakubonderka.cz"
-                }
-            ],
-            "description": "This tool check syntax of PHP files about 20x faster than serial check.",
-            "homepage": "https://github.com/JakubOnderka/PHP-Parallel-Lint",
-            "abandoned": "php-parallel-lint/php-parallel-lint",
-            "time": "2018-02-24T15:31:20+00:00"
-        },
        {
            "name": "johnkary/phpunit-speedtrap",
            "version": "v1.1.0",
@ -3635,6 +3586,59 @@
            ],
            "time": "2017-10-19T19:58:43+00:00"
        },
+        {
+            "name": "php-parallel-lint/php-parallel-lint",
+            "version": "v1.2.0",
+            "source": {
+                "type": "git",
+                "url": "https://github.com/php-parallel-lint/PHP-Parallel-Lint.git",
+                "reference": "474f18bc6cc6aca61ca40bfab55139de614e51ca"
+            },
+            "dist": {
+                "type": "zip",
+                "url": "https://api.github.com/repos/php-parallel-lint/PHP-Parallel-Lint/zipball/474f18bc6cc6aca61ca40bfab55139de614e51ca",
+                "reference": "474f18bc6cc6aca61ca40bfab55139de614e51ca",
+                "shasum": ""
+            },
+            "require": {
+                "ext-json": "*",
+                "php": ">=5.4.0"
+            },
+            "replace": {
+                "grogy/php-parallel-lint": "*",
+                "jakub-onderka/php-parallel-lint": "*"
+            },
+            "require-dev": {
+                "nette/tester": "^1.3 || ^2.0",
+                "php-parallel-lint/php-console-highlighter": "~0.3",
+                "squizlabs/php_codesniffer": "~3.0"
+            },
+            "suggest": {
+                "php-parallel-lint/php-console-highlighter": "Highlight syntax in code snippet"
+            },
+            "bin": [
+                "parallel-lint"
+            ],
+            "type": "library",
+            "autoload": {
+                "classmap": [
+                    "./"
+                ]
+            },
+            "notification-url": "https://packagist.org/downloads/",
+            "license": [
+                "BSD-2-Clause"
+            ],
+            "authors": [
+                {
+                    "name": "Jakub Onderka",
+                    "email": "ahoj@jakubonderka.cz"
+                }
+            ],
+            "description": "This tool check syntax of PHP files about 20x faster than serial check.",
+            "homepage": "https://github.com/php-parallel-lint/PHP-Parallel-Lint",
+            "time": "2020-04-04T12:18:32+00:00"
+        },
        {
            "name": "phpdocumentor/reflection-common",
            "version": "1.0.1",
--- a/src/Module/Api/Twitter/ContactEndpoint.php
+++ b/src/Module/Api/Twitter/ContactEndpoint.php
@ -73,7 +73,7 @@ abstract class ContactEndpoint extends BaseApi
 				throw new HTTPException\NotFoundException(DI::l10n()->t('User not found'));
 			}

-			$uid = $user['uid'];
+			$uid = (int)$user['uid'];
 		}

 		return $uid;
@ -111,7 +111,7 @@ abstract class ContactEndpoint extends BaseApi
 			'next_cursor_str' => $return['next_cursor_str'],
 			'previous_cursor' => $return['previous_cursor'],
 			'previous_cursor_str' => $return['previous_cursor_str'],
-			'total_count' => $return['total_count'],
+			'total_count' => (int)$return['total_count'],
 		];

 		return $return;
@ -153,7 +153,7 @@ abstract class ContactEndpoint extends BaseApi
 				'pending' => false
 			];

-			$total_count = DBA::count('contact', $condition);
+			$total_count = (int)DBA::count('contact', $condition);

 			if ($cursor !== -1) {
 				if ($cursor > 0) {
@ -171,7 +171,7 @@ abstract class ContactEndpoint extends BaseApi
 			// Cursor is on the user-specific contact id since it's the sort field
 			if (count($ids)) {
 				$previous_cursor = -$ids[0];
-				$next_cursor = $ids[count($ids) -1];
+				$next_cursor = (int)$ids[count($ids) -1];
 			}

 			// No next page
--- a/src/Object/Api/Twitter/User.php
+++ b/src/Object/Api/Twitter/User.php
@ -89,7 +89,7 @@ class User extends BaseEntity
 	 */
 	public function __construct(array $publicContact, array $apcontact = [], array $userContact = [], $skip_status = false, $include_user_entities = true)
 	{
-		$this->id                      = $publicContact['id'];
+		$this->id                      = (int)$publicContact['id'];
 		$this->id_str                  = (string) $publicContact['id'];
 		$this->name                    = $publicContact['name'];
 		$this->screen_name             = $publicContact['nick'] ?: $publicContact['name'];
@ -143,10 +143,10 @@ class User extends BaseEntity
 		$this->notifications                  = false;

 		// Friendica-specific
-		$this->uid                   = $userContact['uid'] ?? 0;
-		$this->cid                   = $userContact['id'] ?? 0;
-		$this->pid                   = $publicContact['id'];
-		$this->self                  = $userContact['self'] ?? false;
+		$this->uid                   = (int)$userContact['uid'] ?? 0;
+		$this->cid                   = (int)$userContact['id'] ?? 0;
+		$this->pid                   = (int)$publicContact['id'];
+		$this->self                  = (boolean)$userContact['self'] ?? false;
 		$this->network               = $publicContact['network'];
 		$this->statusnet_profile_url = $publicContact['url'];
 	}
--- a/src/Util/PidFile.php
+++ b/src/Util/PidFile.php
--- a/src/Worker/CleanWorkerQueue.php
+++ b/src/Worker/CleanWorkerQueue.php
--- a/tests/include/ApiTest.php
+++ b/tests/include/ApiTest.php
@ -3,7 +3,7 @@
 * ApiTest class.
 */

-namespace Friendica\Test;
+namespace Friendica\Test\legacy;

 use Friendica\App;
 use Friendica\Core\Config\IConfig;
@ -11,6 +11,7 @@ use Friendica\Core\PConfig\IPConfig;
 use Friendica\Core\Protocol;
 use Friendica\DI;
 use Friendica\Network\HTTPException;
+use Friendica\Test\FixtureTest;
 use Friendica\Util\Temporal;
 use Monolog\Handler\TestHandler;

@ -49,6 +50,10 @@ class ApiTest extends FixtureTest
 	 */
 	protected function setUp()
 	{
+		global $API, $called_api;
+		$API = [];
+		$called_api = [];
+
 		parent::setUp();

 		/** @var IConfig $config */
--- a/tests/src/Content/Text/BBCode/VideoTest.php
+++ b/tests/src/Content/Text/BBCode/VideoTest.php
@ -19,7 +19,7 @@
 *
 */

-namespace Friendica\Test\Content\Text\BBCode;
+namespace Friendica\Test\src\Content\Text\BBCode;

 use Friendica\Content\Text\BBCode\Video;
 use Friendica\Test\MockedTest;
--- a/tests/src/Core/L10nTest.php
+++ b/tests/src/Core/L10nTest.php
@ -19,7 +19,7 @@
 *
 */

-namespace Friendica\Test\src\Core\L10n;
+namespace Friendica\Test\src\Core;

 use Friendica\Core\L10n;
 use Friendica\Test\MockedTest;
--- a/tests/src/Core/Lock/DatabaseLockDriverTest.php
+++ b/tests/src/Core/Lock/DatabaseLockDriverTest.php
--- a/tests/src/Module/Api/Twitter/ContactEndpointTest.php
+++ b/tests/src/Module/Api/Twitter/ContactEndpointTest.php
@ -234,7 +234,7 @@ class ContactEndpointTest extends FixtureTest
 			'uid' => 42,
 			'cid' => 44,
 			'pid' => 45,
-			'self' => 0,
+			'self' => false,
 			'network' => 'dfrn',
 			'statusnet_profile_url' => 'http://localhost/profile/friendcontact',
 		];
--- a/tests/src/Protocol/ActivityTest.php
+++ b/tests/src/Protocol/ActivityTest.php
@ -19,7 +19,7 @@
 *
 */

-namespace Friendica\Test\Protocol;
+namespace Friendica\Test\src\Protocol;

 use Friendica\Protocol\Activity;
 use Friendica\Protocol\ActivityNamespace;
--- a/tests/src/Util/JsonLDTest.php
+++ b/tests/src/Util/JsonLDTest.php