Issues (257)

src/AdsenseBuilder.php (7 issues)

1
<?php
2
3
/**
4
 * Google Adsense Ads for Laravel.
5
 *
6
 * Package for easily including Google Adsense Ad units
7
 * in Laravel and Lumen.
8
 *
9
 * @developer Martin Butt <https://www.martinbutt.com/>
0 ignored issues
show
The tag in position 1 should be the @package tag
Loading history...
10
 *
11
 * @copyright Copyright (c) 2021 Martin Butt
0 ignored issues
show
Coding Style Documentation introduced by
Expected "xxxx-xxxx Squiz Pty Ltd (ABN 77 084 670 600)" for copyright declaration
Loading history...
The tag in position 2 should be the @subpackage tag
Loading history...
@copyright tag must contain a year and the name of the copyright holder
Loading history...
12
 * @license   MIT
13
 *
14
 * Copyright (c) 2016 Galen Han
15
 * Copyright (c) 2019 Crypto Technology srl
16
 * Copyright (c) 2021 Martin Butt
17
 *
18
 *
19
 * Permission is hereby granted, free of charge, to any person obtaining a copy of
20
 * this software and associated documentation files (the "Software"), to deal in
21
 * the Software without restriction, including without limitation the rights to
22
 * use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
23
 * the Software, and to permit persons to whom the Software is furnished to do so,
24
 * subject to the following conditions:
25
 *
26
 * The above copyright notice and this permission notice shall be included in all
27
 * copies or substantial portions of the Software.
28
 *
29
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
30
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
31
 * FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
32
 * COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
33
 * IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
34
 * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
35
 */
36
37
declare(strict_types=1);
38
39
namespace MartinButt\Laravel\Adsense;
40
41
class AdsenseBuilder
42
{
43
    public function ads($unit)
44
    {
45
        return view('adsense::ads')->with([
46
            'ad_client' => config('adsense.client_id'),
47
            'ad_style' => config("adsense.ads.$unit.ad_style", 'display:block;'),
48
            'ad_slot' => config("adsense.ads.$unit.ad_slot"),
49
            'ad_format' => config("adsense.ads.$unit.ad_format"),
50
            'ad_full_width_responsive' => config("adsense.ads.$unit.ad_full_width_responsive"),
0 ignored issues
show
Array indentation error, expected 10 spaces but found 12
Loading history...
Coding Style Best Practice introduced by
As per coding-style, please use concatenation or sprintf for the variable $unit instead of interpolation.

It is generally a best practice as it is often more readable to use concatenation instead of interpolation for variables inside strings.

// Instead of
$x = "foo $bar $baz";

// Better use either
$x = "foo " . $bar . " " . $baz;
$x = sprintf("foo %s %s", $bar, $baz);
Loading history...
51
        ]);
52
    }
53 2
54
    public function javascript()
55 2
    {
56
        return view('adsense::javascript', ['client_id' => config('adsense.client_id')]);
0 ignored issues
show
Short array syntax is not allowed
Loading history...
57
    }
58
}
59