Aimeos\MW\View\Helper\Request\Laravel5

Complexity

Total Complexity

8

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	3
Bugs	0	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	getBody()	0	4	1
A	getClientAddress()	0	4	1
A	getTarget()	0	6	2
B	createUploadedFiles()	0	24	3

<?php

/**
 * @license LGPLv3, http://opensource.org/licenses/LGPL-3.0
 * @copyright Aimeos (aimeos.org), 2015
 * @package MW
 * @subpackage View
 */


namespace Aimeos\MW\View\Helper\Request;


/**
 * View helper class for retrieving request data.
 *
 * @package MW
 * @subpackage View
 */
class Laravel5
	extends \Aimeos\MW\View\Helper\Request\Standard

The extends keyword must be on the same line as the class name

Expected 0 spaces between "Standard" and comma; 1 found

	implements \Aimeos\MW\View\Helper\Request\Iface

The implements keyword must be on the same line as the class name

{
	private $request;


	/**
	 * Initializes the request view helper.
	 *
	 * @param \\Aimeos\MW\View\Iface $view View instance with registered view helpers
	 * @param \Illuminate\Http\Request $request Laravel request object
	 */
	public function __construct( \Aimeos\MW\View\Iface $view, \Illuminate\Http\Request $request )

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

	{
		\Aimeos\MW\View\Helper\Request\Standard::__construct( $view, null, null, null, $request->file() );

		$this->request = $request;
	}


	/**
	 * Returns the request body.
	 *
	 * @return string Request body
	 */
	public function getBody()
	{
		return $this->request->getContent();
	}


	/**
	 * Returns the client IP address.
	 *
	 * @return string Client IP address
	 */
	public function getClientAddress()
	{
		return $this->request->ip();
	}


	/**
	 * Returns the current page or route name
	 *
	 * @return string|null Current page or route name
	 */
	public function getTarget()
	{
		if( ( $route = $this->request->route() ) !== null ) {
			return $route->getName();
		}
	}


	/**
	 * Creates a normalized file upload data from the given array.
	 *
	 * @param \Traversable|array $files File upload data
	 * @return array Multi-dimensional list of file objects
	 */
	protected function createUploadedFiles( $files )
	{
		$files = array();

		foreach( $files as $key => $value )
		{
			if( $value instanceof UploadedFile )

The class Aimeos\MW\View\Helper\Request\UploadedFile does not exist. Did you forget a USE statement, or did you not list all dependencies?

			{
				$files[$key] = new \Aimeos\MW\View\Helper\Request\File\Standard(
					$value->getRealPath(),
					$value->getClientOriginalName(),
					$value->getSize(),
					$value->getClientMimeType(),
					$value->getError()
				);
			}
			else
			{
				$files[$key] = $this->createUploadedFiles($value);
			}
		}

		return $files;
	}
}