Aimeos\Shop\Controller\CatalogController

Complexity

Total Complexity

16

Size/Duplication

Total Lines	146
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	countAction()	0	11	2
A	homeAction()	0	10	2
A	listAction()	0	10	2
A	detailAction()	0	10	2
A	treeAction()	0	10	2
A	sessionAction()	0	10	2
A	stockAction()	0	11	2
A	suggestAction()	0	11	2

<?php

/**
 * @license MIT, http://opensource.org/licenses/MIT
 * @copyright Aimeos (aimeos.org), 2015-2016
 * @package laravel
 * @subpackage Controller
 */


namespace Aimeos\Shop\Controller;

use Aimeos\Shop\Facades\Shop;
use Illuminate\Routing\Controller;
use Illuminate\Support\Facades\Response;


/**
 * Aimeos controller for catalog related functionality.
 *
 * @package laravel
 * @subpackage Controller
 */
class CatalogController extends Controller
{
	/**
	 * Returns the view for the XHR response with the counts for the facetted search.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function countAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-count' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.count' ), $params )

The variable $params seems to be defined by a foreach iteration on line 33. Are you sure the iterator is never empty, otherwise this variable is not defined?

The method template() does not exist on Aimeos\Shop\Facades\Shop. Since you implemented __callStatic, consider adding a @method annotation.

			->header( 'Content-Type', 'application/javascript' )
			->header( 'Cache-Control', 'public, max-age=300' );
	}


	/**
	 * Returns the html for the catalog detail page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function detailAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-detail' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.detail' ), $params )

The variable $params seems to be defined by a foreach iteration on line 52. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'private, max-age=10' );
	}


	/**
	 * Returns the html for the catalog home page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function homeAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-home' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.home' ), $params )

The variable $params seems to be defined by a foreach iteration on line 70. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'private, max-age=10' );
	}


	/**
	 * Returns the html for the catalog list page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function listAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-list' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.list' ), $params )

The variable $params seems to be defined by a foreach iteration on line 88. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'private, max-age=10' );
	}


	/**
	 * Returns the html for the catalog session page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function sessionAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-session' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.session' ), $params )

The variable $params seems to be defined by a foreach iteration on line 106. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'no-cache' );
	}


	/**
	 * Returns the html body part for the catalog stock page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function stockAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-stock' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.stock' ), $params )

The variable $params seems to be defined by a foreach iteration on line 124. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Content-Type', 'application/javascript' )
			->header( 'Cache-Control', 'public, max-age=30' );
	}


	/**
	 * Returns the view for the XHR response with the product information for the search suggestion.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function suggestAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-suggest' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.suggest' ), $params )

The variable $params seems to be defined by a foreach iteration on line 143. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'private, max-age=300' )
			->header( 'Content-Type', 'application/json' );
	}


	/**
	 * Returns the html for the catalog tree page.
	 *
	 * @return \Illuminate\Http\Response Response object with output and headers
	 */
	public function treeAction()
	{
		foreach( app( 'config' )->get( 'shop.page.catalog-tree' ) as $name )
		{
			$params['aiheader'][$name] = Shop::get( $name )->getHeader();

The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name.

			$params['aibody'][$name] = Shop::get( $name )->getBody();
		}

		return Response::view( Shop::template( 'catalog.tree' ), $params )

The variable $params seems to be defined by a foreach iteration on line 162. Are you sure the iterator is never empty, otherwise this variable is not defined?

			->header( 'Cache-Control', 'private, max-age=10' );
	}
}