MartinButt\Laravel\Adsense\AdsenseBuilder

Complexity

Total Complexity

2

Size/Duplication

Total Lines	16
Duplicated Lines	0 %

Test Coverage

Coverage

25%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	ads()	0	8	1
A	javascript()	0	3	1

<?php

Class found in ".php" file; use ".inc" extension instead

There must be no blank lines before the file comment

The PHP open tag does not have a corresponding PHP close tag

Filename "AdsenseBuilder.php" doesn't match the expected filename "adsensebuilder.php"

/**

Namespaced classes, interfaces and traits should not begin with a file doc comment

 * Google Adsense Ads for Laravel.
 *
 * Package for easily including Google Adsense Ad units
 * in Laravel and Lumen.
 *
 * @developer Martin Butt <https://www.martinbutt.com/>

The tag in position 1 should be the @package tag

 *
 * @copyright Copyright (c) 2021 Martin Butt

Expected "xxxx-xxxx Squiz Pty Ltd (ABN 77 084 670 600)" for copyright declaration

The tag in position 2 should be the @subpackage tag

@copyright tag must contain a year and the name of the copyright holder

 * @license   MIT

The tag in position 3 should be the @author tag

@license tag must contain a URL and a license name

Tag value indented incorrectly; expected 1 space but found 3

 *
 * Copyright (c) 2016 Galen Han
 * Copyright (c) 2019 Crypto Technology srl
 * Copyright (c) 2021 Martin Butt
 *
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy of

Line exceeds 80 characters; contains 82 characters

 * this software and associated documentation files (the "Software"), to deal in
 * the Software without restriction, including without limitation the rights to
 * use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of

Line exceeds 80 characters; contains 83 characters

 * the Software, and to permit persons to whom the Software is furnished to do so,

Line exceeds 80 characters; contains 82 characters

 * subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in all

Line exceeds 80 characters; contains 81 characters

 * copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS

Line exceeds 80 characters; contains 83 characters

 * FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR

Line exceeds 80 characters; contains 81 characters

 * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER

Line exceeds 80 characters; contains 81 characters

 * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
 * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 */

PHP version not specified

Missing @package tag in file comment

Missing @subpackage tag in file comment

Missing @author tag in file comment

Missing @category tag in file comment

Missing @link tag in file comment

declare(strict_types=1);

namespace MartinButt\Laravel\Adsense;

class AdsenseBuilder

Missing doc comment for class AdsenseBuilder

Missing class doc comment

{

Opening brace should be on the same line as the declaration for class AdsenseBuilder

Opening brace should be on the same line as the declaration

    public function ads($unit)

Line indented incorrectly; expected 2 spaces, found 4

Expected 1 blank line before function; 0 found

Missing doc comment for function ads()

Missing function doc comment

    {

Opening brace should be on the same line as the declaration

        return view('adsense::ads')->with([

Line indented incorrectly; expected 4 spaces, found 8

Short array syntax is not allowed

            'ad_client' => config('adsense.client_id'),

Array indentation error, expected 10 spaces but found 12

            'ad_style' => config("adsense.ads.$unit.ad_style", 'display:block;'),

Array indentation error, expected 10 spaces but found 12

As per coding-style, please use concatenation or sprintf for the variable $unit instead of interpolation.

            'ad_slot' => config("adsense.ads.$unit.ad_slot"),

Array indentation error, expected 10 spaces but found 12

As per coding-style, please use concatenation or sprintf for the variable $unit instead of interpolation.

            'ad_format' => config("adsense.ads.$unit.ad_format"),

Array indentation error, expected 10 spaces but found 12

As per coding-style, please use concatenation or sprintf for the variable $unit instead of interpolation.

            'ad_full_width_responsive' => config("adsense.ads.$unit.ad_full_width_responsive"),

Array indentation error, expected 10 spaces but found 12

As per coding-style, please use concatenation or sprintf for the variable $unit instead of interpolation.

        ]);
    }

Expected 1 blank line before closing function brace; 0 found

Line indented incorrectly; expected 2 spaces, found 4

Expected //end ads()

    public function javascript()

Line indented incorrectly; expected 2 spaces, found 4

Missing function doc comment

Missing doc comment for function javascript()

    {

Opening brace should be on the same line as the declaration

        return view('adsense::javascript', ['client_id' => config('adsense.client_id')]);

Line indented incorrectly; expected 4 spaces, found 8

Short array syntax is not allowed

    }

Expected //end javascript()

Expected 1 blank line before closing function brace; 0 found

Expected 1 blank line after function; 0 found

Line indented incorrectly; expected 2 spaces, found 4

}

As per coding style, files should not end with a newline character.

Expected //end class