Request

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

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

<?php

namespace \SimpleSAML\Module\monitor\DependencyInjection;

class Request

This class seems to be duplicated in your project.

{
    // The canonical list of acceptable request parameters
    /**
     * @var mixed
     */
    public $xml;

    /**
     * @param array $requestVars
     *
     * @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($requestVars)
    {
        $objectVars = get_object_vars($this);
        foreach ($requestVars 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?

    }
}