DependencyInjectionFactory

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	11	3

<?php

namespace \SimpleSAML\Module\monitor;

abstract class DependencyInjectionFactory
{
    /**
     * @param array $vars
     *
     * @return Request
     */

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

    public function __construct($vars)
    {
        $objectVars = get_object_vars($this);
        foreach ($vars as $property => $value)
        {
            if (array_key_exists($property, $objectVars)) {
                $this->$property = $value;
            }
        }
        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }
}