ApiException::withPrevious() fix

phpstan.neon

@@ -7,9 +7,6 @@ parameters:
 		- '#^Variable property access on (\$this|static)\(Apitte\\Core\\Mapping\\(Request|Response)\\BasicEntity\)\.$#'
 		- '#^Variable property access on Apitte\\Core\\Mapping\\Request\\BasicEntity\.$#'
 
-		# Magic access
-		- '#^Access to private property \$previous of parent class Exception.$#'
-
 		# There is no apitte/negotiation dependency
 		- '#^Return typehint of method Apitte\\Core\\Http\\ApiResponse::getEntity\(\) has invalid type Apitte\\Negotiation\\Http\\AbstractEntity\.$#'
 		- '#^Parameter \$entity of method Apitte\\Core\\Http\\ApiResponse::withEntity\(\) has invalid typehint type Apitte\\Negotiation\\Http\\AbstractEntity\.$#'

ruleset.xml

@@ -15,6 +15,10 @@
         </properties>
     </rule>
 
+	<rule ref="SlevomatCodingStandard.Exceptions.ReferenceThrowableOnly.ReferencedGeneralException">
+		<exclude-pattern>src/Exception/ExceptionExtra.php</exclude-pattern>
+	</rule>
+
     <rule ref="PSR1.Classes.ClassDeclaration.MissingNamespace">
         <exclude-pattern>*tests/cases/*</exclude-pattern>
     </rule>

src/Exception/ExceptionExtra.php

@@ -3,6 +3,7 @@
 namespace Apitte\Core\Exception;
 
 use Exception;
+use ReflectionClass;
 use Throwable;
 
 /**
@@ -48,7 +49,11 @@ public function withMessage($message)
 	 */
 	public function withPrevious(Throwable $exception)
 	{
-		$this->previous = $exception;
+		$reflection = new ReflectionClass(Exception::class);
+		$property = $reflection->getProperty('previous');
+		$property->setAccessible(true);
+		$property->setValue($this, $exception);
+		$property->setAccessible(false);
 
 		return $this;
 	}
@@ -75,7 +80,6 @@ public function withTypedContext(string $type, $context)
 		return $this;
 	}
 
-
 	/**
 	 * @return mixed
 	 */

tests/cases/Exception/ExceptionExtra.phpt

@@ -0,0 +1,22 @@
+<?php declare(strict_types = 1);
+
+use Apitte\Core\Exception\Api\ClientErrorException;
+use Apitte\Core\Http\ApiResponse;
+use Tester\Assert;
+
+require_once __DIR__ . '/../../bootstrap.php';
+
+test(static function (): void {
+	$previous = new Exception('previous');
+
+	$exception = ClientErrorException::create()
+		->withCode(ApiResponse::S406_NOT_ACCEPTABLE)
+		->withMessage('test')
+		->withPrevious($previous)
+		->withTypedContext('foo', 'bar');
+
+	Assert::same(ApiResponse::S406_NOT_ACCEPTABLE, $exception->getCode());
+	Assert::same('test', $exception->getMessage());
+	Assert::same($previous, $exception->getPrevious());
+	Assert::same(['foo' => 'bar'], $exception->getContext());
+});