Issues (145)

src/Controller/AccountController.php (2 issues)

Severity
1
<?php
2
3
/**
4
 * @license MIT, http://opensource.org/licenses/MIT
5
 * @copyright Aimeos (aimeos.org), 2015-2023
6
 */
7
8
9
namespace Aimeos\Shop\Controller;
10
11
use Aimeos\Shop\Facades\Shop;
12
use Illuminate\Routing\Controller;
13
use Illuminate\Support\Facades\Response;
14
15
16
/**
17
 * Aimeos controller for account related functionality.
18
 */
19
class AccountController extends Controller
20
{
21
	/**
22
	 * Returns the html for the "My account" page.
23
	 *
24
	 * @return \Illuminate\Http\Response Response object with output and headers
25
	 */
26
	public function indexAction()
27
	{
28
		$params = ['page' => 'page-account-index'];
29
30
		foreach( app( 'config' )->get( 'shop.page.account-index' ) as $name )
31
		{
32
			$params['aiheader'][$name] = Shop::get( $name )->header();
0 ignored issues
show
The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with $name. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

32
			$params['aiheader'][$name] = Shop::/** @scrutinizer ignore-call */ get( $name )->header();

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
33
			$params['aibody'][$name] = Shop::get( $name )->body();
34
		}
35
36
		return Response::view( Shop::template( 'account.index' ), $params )
37
			->header( 'Cache-Control', 'no-store, max-age=0' );
38
	}
39
40
41
	/**
42
	 * Returns the html for the "My account" download page.
43
	 *
44
	 * @return \Illuminate\Contracts\View\View View for rendering the output
45
	 */
46
	public function downloadAction()
47
	{
48
		$response = Shop::get( 'account/download' )->response();
0 ignored issues
show
The call to Aimeos\Shop\Facades\Shop::get() has too many arguments starting with 'account/download'. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

48
		$response = Shop::/** @scrutinizer ignore-call */ get( 'account/download' )->response();

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
49
		return Response::make( (string) $response->getBody(), $response->getStatusCode(), $response->getHeaders() );
50
	}
51
}