fix implicit null usage (#1362)

brettmc · web-flow · commit 0cba875ea195 · 2024-08-21T10:29:20.000+10:00
php8.4 deprecates implicit null (now checked by phan)
diff --git a/Propagation/MultiTextMapPropagator.php b/Propagation/MultiTextMapPropagator.php
@@ -32,14 +32,14 @@ public function fields(): array
         return $this->fields;
     }
 
-    public function inject(&$carrier, PropagationSetterInterface $setter = null, ContextInterface $context = null): void
+    public function inject(&$carrier, ?PropagationSetterInterface $setter = null, ?ContextInterface $context = null): void
     {
         foreach ($this->propagators as $propagator) {
             $propagator->inject($carrier, $setter, $context);
         }
     }
 
-    public function extract($carrier, PropagationGetterInterface $getter = null, ContextInterface $context = null): ContextInterface
+    public function extract($carrier, ?PropagationGetterInterface $getter = null, ?ContextInterface $context = null): ContextInterface
     {
         $context ??= Context::getCurrent();
 

Original file line number	Diff line number	Diff line change
`@@ -32,14 +32,14 @@ public function fields(): array`
`32`	`32`	`return $this->fields;`
`33`	`33`	`}`
`34`	`34`
`35`		`- public function inject(&$carrier, PropagationSetterInterface $setter = null, ContextInterface $context = null): void`
	`35`	`+ public function inject(&$carrier, ?PropagationSetterInterface $setter = null, ?ContextInterface $context = null): void`
`36`	`36`	`{`
`37`	`37`	`foreach ($this->propagators as $propagator) {`
`38`	`38`	`$propagator->inject($carrier, $setter, $context);`
`39`	`39`	`}`
`40`	`40`	`}`
`41`	`41`
`42`		`- public function extract($carrier, PropagationGetterInterface $getter = null, ContextInterface $context = null): ContextInterface`
	`42`	`+ public function extract($carrier, ?PropagationGetterInterface $getter = null, ?ContextInterface $context = null): ContextInterface`
`43`	`43`	`{`
`44`	`44`	`$context ??= Context::getCurrent();`
`45`	`45`