Merge pull request #92 from axwel13/master

fix deprecations and symfony 6

Author: mKnoop

src/Activator/TraceableChainActivator.php

@@ -23,7 +23,7 @@ class TraceableChainActivator extends ChainActivator
     /**
      * {@inheritdoc}
      */
-    public function isActive($name, Context $context)
+    public function isActive($name, Context $context): bool
     {
         $stack = [];
         $result = false;

src/DependencyInjection/Configuration.php

@@ -36,7 +36,7 @@ public function __construct(array $configurators)
     /**
      * {@inheritdoc}
      */
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('flagception');
         $rootNode = $treeBuilder->getRootNode();
@@ -108,7 +108,7 @@ public function getConfigTreeBuilder()
      *
      * @return NodeDefinition
      */
-    public function appendActivators()
+    public function appendActivators(): NodeDefinition
     {
         $builder = new TreeBuilder('activators');
         $node = $builder->getRootNode();

src/DependencyInjection/FlagceptionExtension.php

@@ -71,7 +71,7 @@ public function load(array $configs, ContainerBuilder $container)
      *
      * @throws Exception
      */
-    private function getConfigurators(ContainerBuilder $container)
+    private function getConfigurators(ContainerBuilder $container): array
     {
         $configurators = [];
 

src/Listener/AnnotationSubscriber.php

@@ -95,7 +95,7 @@ public function onKernelController(ControllerEvent $event)
     /**
      * {@inheritdoc}
      */
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             KernelEvents::CONTROLLER => 'onKernelController',

src/Listener/RoutingMetadataSubscriber.php

@@ -65,7 +65,7 @@ public function onKernelController(ControllerEvent $event)
     /**
      * {@inheritdoc}
      */
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             KernelEvents::CONTROLLER => 'onKernelController',

src/Profiler/FeatureDataCollector.php

@@ -95,7 +95,7 @@ public function collect(Request $request, Response $response, Throwable $excepti
                 ];
             }
 
-            $featureTrace = &$this->data['requests'][$trace['feature']];
+            $featureTrace = $this->data['requests'][$trace['feature']];
             $featureTrace['requests']++;
 
             if ($trace['result'] === true) {
@@ -117,6 +117,7 @@ public function collect(Request $request, Response $response, Throwable $excepti
                     $this->data['activators'][$activator]['inactiveRequests']++;
                 }
             }
+            $this->data['requests'][$trace['feature']] = $featureTrace;
         }
 
         $this->data['summary']['features'] = count($this->data['requests']);
@@ -136,7 +137,7 @@ public function collect(Request $request, Response $response, Throwable $excepti
      *
      * @return array
      */
-    public function getRequests()
+    public function getRequests(): array
     {
         return $this->data['requests'];
     }
@@ -146,7 +147,7 @@ public function getRequests()
      *
      * @return array
      */
-    public function getActivators()
+    public function getActivators(): array
     {
         return $this->data['activators'];
     }
@@ -156,7 +157,7 @@ public function getActivators()
      *
      * @return array
      */
-    public function getDecorators()
+    public function getDecorators(): array
     {
         return $this->data['decorators'];
     }
@@ -166,7 +167,7 @@ public function getDecorators()
      *
      * @return array
      */
-    public function getTrace()
+    public function getTrace(): array
     {
         return $this->data['trace'];
     }
@@ -176,15 +177,15 @@ public function getTrace()
      *
      * @return array
      */
-    public function getSummary()
+    public function getSummary(): array
     {
         return $this->data['summary'];
     }
 
     /**
      * {@inheritdoc}
      */
-    public function getName()
+    public function getName(): string
     {
         return 'flagception';
     }

src/Twig/ToggleExtension.php

@@ -41,7 +41,7 @@ public function __construct(FeatureManagerInterface $manager)
      *
      * @return bool
      */
-    public function isActive($name, array $contextValues = [])
+    public function isActive(string $name, array $contextValues = []): bool
     {
         $context = new Context();
         foreach ($contextValues as $contextKey => $contextValue) {
@@ -54,7 +54,7 @@ public function isActive($name, array $contextValues = [])
     /**
      * {@inheritDoc}
      */
-    public function getFunctions()
+    public function getFunctions(): array
     {
         return [
             new TwigFunction('feature', [$this, 'isActive']),
@@ -64,7 +64,7 @@ public function getFunctions()
     /**
      * {@inheritDoc}
      */
-    public function getTests()
+    public function getTests(): array
     {
         return [
             new TwigTest('active feature', [$this, 'isActive']),
@@ -77,7 +77,7 @@ public function getTests()
      *
      * @return string
      */
-    public function getName()
+    public function getName(): string
     {
         return 'flagception';
     }