Merge pull request #416 from oat-sa/feat/TR-6514/php-84

fix: correct php 8.4 notices

Author: kilatib

src/qtism/common/ResolutionException.php

@@ -38,7 +38,7 @@ class ResolutionException extends RuntimeException
      * @param string $message A human-readable description of the exception.
      * @param Exception $previous An optional previous Exception that caused the exception to be thrown.
      */
-    public function __construct($message, Exception $previous = null)
+    public function __construct($message, ?Exception $previous = null)
     {
         parent::__construct($message, 0, $previous);
     }

src/qtism/common/beans/BeanException.php

@@ -87,7 +87,7 @@ class BeanException extends Exception
      * @param int $code An error code from the BeanException class constants.
      * @param Exception $previous An optional previous exception.
      */
-    public function __construct($message, $code = 0, Exception $previous = null)
+    public function __construct($message, $code = 0, ?Exception $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/qtism/common/datatypes/files/FileManagerException.php

@@ -46,7 +46,7 @@ class FileManagerException extends Exception
      * @param int $code A machine understandable error code (see class constants).
      * @param Exception $previous A possible previous Exception object.
      */
-    public function __construct($message, $code = 0, Exception $previous = null)
+    public function __construct($message, $code = 0, ?Exception $previous = null)
     {
         parent::__construct($message, $code, $previous);
     }

src/qtism/common/storage/MemoryStreamException.php

@@ -39,7 +39,7 @@ class MemoryStreamException extends StreamException
      * @param int $code A code describing the error.
      * @param Exception $previous An optional previous exception.
      */
-    public function __construct($message, IStream $source, $code = 0, Exception $previous = null)
+    public function __construct($message, IStream $source, $code = 0, ?Exception $previous = null)
     {
         parent::__construct($message, $source, $code, $previous);
     }

src/qtism/common/storage/StreamAccessException.php

@@ -61,7 +61,7 @@ class StreamAccessException extends Exception
      * @param int $code An exception code. See class constants.
      * @param Exception $previous An optional previously thrown exception.
      */
-    public function __construct($message, AbstractStreamAccess $source, $code = 0, Exception $previous = null)
+    public function __construct($message, AbstractStreamAccess $source, $code = 0, ?Exception $previous = null)
     {
         parent::__construct($message, $code, $previous);
         $this->setSource($source);

src/qtism/common/storage/StreamException.php

@@ -101,7 +101,7 @@ abstract class StreamException extends Exception
      * @param int $code A code describing the error.
      * @param Exception $previous An optional previous exception.
      */
-    public function __construct($message, IStream $source, $code = 0, Exception $previous = null)
+    public function __construct($message, IStream $source, $code = 0, ?Exception $previous = null)
     {
         parent::__construct($message, $code, $previous);
         $this->setSource($source);

src/qtism/data/AssessmentItem.php

@@ -538,7 +538,7 @@ public function getTemplateDeclarations(): TemplateDeclarationCollection
      *
      * @param TemplateProcessing $templateProcessing A TemplateProcessing object or null.
      */
-    public function setTemplateProcessing(TemplateProcessing $templateProcessing = null): void
+    public function setTemplateProcessing(?TemplateProcessing $templateProcessing = null): void
     {
         $this->templateProcessing = $templateProcessing;
     }
@@ -589,7 +589,7 @@ public function getStylesheets(): StylesheetCollection
      *
      * @param ItemBody $itemBody An ItemBody object or the NULL value to state that the item has no content.
      */
-    public function setItemBody(ItemBody $itemBody = null): void
+    public function setItemBody(?ItemBody $itemBody = null): void
     {
         $this->itemBody = $itemBody;
     }
@@ -629,7 +629,7 @@ public function getResponseProcessing(): ?ResponseProcessing
      *
      * @param ResponseProcessing $responseProcessing A ResponseProcessing object or null if no associated response processing.
      */
-    public function setResponseProcessing(ResponseProcessing $responseProcessing = null): void
+    public function setResponseProcessing(?ResponseProcessing $responseProcessing = null): void
     {
         $this->responseProcessing = $responseProcessing;
     }

src/qtism/data/AssessmentItemRef.php

@@ -89,7 +89,7 @@ class AssessmentItemRef extends SectionPart
      * @param IdentifierCollection $categories The categories to which the item belongs to.
      * @throws InvalidArgumentException If $href is not a string.
      */
-    public function __construct($identifier, $href, IdentifierCollection $categories = null)
+    public function __construct($identifier, $href, ?IdentifierCollection $categories = null)
     {
         parent::__construct($identifier);
 

src/qtism/data/AssessmentSection.php

@@ -203,7 +203,7 @@ public function getSelection(): ?Selection
      *
      * @param Selection $selection A selection rule.
      */
-    public function setSelection(Selection $selection = null): void
+    public function setSelection(?Selection $selection = null): void
     {
         $this->selection = $selection;
     }
@@ -234,7 +234,7 @@ public function getOrdering(): ?Ordering
      *
      * @param Ordering $ordering An Ordering object.
      */
-    public function setOrdering(Ordering $ordering = null): void
+    public function setOrdering(?Ordering $ordering = null): void
     {
         $this->ordering = $ordering;
     }

src/qtism/data/AssessmentTest.php

@@ -149,7 +149,7 @@ class AssessmentTest extends QtiComponent implements QtiIdentifiable
      * @param $title
      * @param TestPartCollection|null $testParts
      */
-    public function __construct($identifier, $title, TestPartCollection $testParts = null)
+    public function __construct($identifier, $title, ?TestPartCollection $testParts = null)
     {
         $this->setObservers(new SplObjectStorage());
 
@@ -303,7 +303,7 @@ public function getTimeLimits(): ?TimeLimits
      *
      * @param TimeLimits $timeLimits A TimeLimits object.
      */
-    public function setTimeLimits(TimeLimits $timeLimits = null): void
+    public function setTimeLimits(?TimeLimits $timeLimits = null): void
     {
         $this->timeLimits = $timeLimits;
     }
@@ -344,7 +344,7 @@ public function getOutcomeProcessing(): ?OutcomeProcessing
      *
      * @param OutcomeProcessing $outcomeProcessing An OutcomeProcessing object.
      */
-    public function setOutcomeProcessing(OutcomeProcessing $outcomeProcessing = null): void
+    public function setOutcomeProcessing(?OutcomeProcessing $outcomeProcessing = null): void
     {
         $this->outcomeProcessing = $outcomeProcessing;
     }