From 54533c57a51560520bc364e4b80621bd25d5f557 Mon Sep 17 00:00:00 2001
From: Philipp Holzer <admin+github@philipp.info>
Date: Wed, 24 Jul 2019 11:17:55 +0200
Subject: [PATCH 1/3] friendica-7426 Database performance isn't measured
 anymore

---
 static/dependencies.config.php | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/static/dependencies.config.php b/static/dependencies.config.php
index c1ce4f3b56..7da1cac861 100644
--- a/static/dependencies.config.php
+++ b/static/dependencies.config.php
@@ -25,6 +25,11 @@ use Psr\Log\LoggerInterface;
  *
  */
 return [
+	'*' => [
+		// marks all class result as shared for other creations, so there's just
+		// one instance for the whole execution
+		'shared' => true,
+	],
 	'$basepath' => [
 		'instanceOf' => Util\BasePath::class,
 		'call' => [
@@ -36,7 +41,6 @@ return [
 		]
 	],
 	Util\BasePath::class => [
-		'shared'          => true,
 		'constructParams' => [
 			dirname(__FILE__, 2),
 			$_SERVER
@@ -53,15 +57,11 @@ return [
 		'call'       => [
 			['createCache', [], Dice::CHAIN_CALL],
 		],
-		'shared'     => true,
 	],
 	App\Mode::class => [
 		'call'   => [
 			['determine', [], Dice::CHAIN_CALL],
 		],
-		// marks the result as shared for other creations, so there's just
-		// one instance for the whole execution
-		'shared' => true,
 	],
 	Config\Configuration::class => [
 		'shared' => true,
@@ -71,14 +71,12 @@ return [
 		],
 	],
 	Config\PConfiguration::class => [
-		'shared' => true,
 		'instanceOf' => Factory\ConfigFactory::class,
 		'call' => [
 			['createPConfig', [], Dice::CHAIN_CALL],
 		]
 	],
 	Database::class => [
-		'shared' => true,
 		'constructParams' => [
 			[DICE::INSTANCE => \Psr\Log\NullLogger::class],
 			$_SERVER,
@@ -91,7 +89,6 @@ return [
 	 *   $baseURL = new Util\BaseURL($configuration, $_SERVER);
 	 */
 	Util\BaseURL::class => [
-		'shared'          => true,
 		'constructParams' => [
 			$_SERVER,
 		],
@@ -109,17 +106,15 @@ return [
 	 *    and is automatically passed as an argument with the same name
 	 */
 	LoggerInterface::class => [
-		'shared'     => true,
 		'instanceOf' => Factory\LoggerFactory::class,
 		'call'       => [
 			['create', [], Dice::CHAIN_CALL],
 		],
 	],
 	'$devLogger' => [
-		'shared'     => true,
 		'instanceOf' => Factory\LoggerFactory::class,
 		'call'       => [
 			['createDev', [], Dice::CHAIN_CALL],
 		]
-	]
+	],
 ];

From 6759e0c193b7ecc1d3e192a057ade4a50d65ea88 Mon Sep 17 00:00:00 2001
From: Philipp Holzer <admin+github@philipp.info>
Date: Wed, 24 Jul 2019 12:46:22 +0200
Subject: [PATCH 2/3] Remove superfluous "print"

---
 tests/src/Core/SystemTest.php | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tests/src/Core/SystemTest.php b/tests/src/Core/SystemTest.php
index 6b0781198a..a385d1b352 100644
--- a/tests/src/Core/SystemTest.php
+++ b/tests/src/Core/SystemTest.php
@@ -9,7 +9,6 @@ class SystemTest extends TestCase
 {
 	private function assertGuid($guid, $length, $prefix = '')
 	{
-		print $guid;
 		$length -= strlen($prefix);
 		$this->assertRegExp("/^" . $prefix . "[a-z0-9]{" . $length . "}?$/", $guid);
 	}

From 71bc37c1c1dbd2ba2b49e65abd2497ed1a02b671 Mon Sep 17 00:00:00 2001
From: Philipp Holzer <admin+github@philipp.info>
Date: Wed, 24 Jul 2019 12:48:29 +0200
Subject: [PATCH 3/3] Fix DI-check test

---
 tests/functional/DependencyCheckTest.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tests/functional/DependencyCheckTest.php b/tests/functional/DependencyCheckTest.php
index 4a1ab5afde..9fe469d5e8 100644
--- a/tests/functional/DependencyCheckTest.php
+++ b/tests/functional/DependencyCheckTest.php
@@ -83,6 +83,8 @@ class dependencyCheck extends TestCase
 			]
 		]);
 
+		// create new DI-library because of shared instance rule (so the Profiler wouldn't get created twice)
+		$this->dice = new Dice(include __DIR__ . '/../../static/dependencies.config.php');
 		$profiler = $this->dice->create(Profiler::class, [$configCache]);
 
 		$this->assertInstanceOf(Profiler::class, $profiler);