Test Failed
Pull Request — master (#70)
by Rafael
04:52
created

AppsPushNotifications   A

Complexity

Total Complexity 2

Size/Duplication

Total Lines 12
Duplicated Lines 0 %

Test Coverage

Coverage 0%

Importance

Changes 0
Metric Value
eloc 5
dl 0
loc 12
ccs 0
cts 9
cp 0
rs 10
c 0
b 0
f 0
wmc 2

2 Methods

Rating   Name   Duplication   Size   Complexity  
A __construct() 0 5 1
A assemble() 0 3 1
1
<?php
2
3
namespace Gewaer\Notifications\PushNotifications;
4
5
use Namshi\Notificator\Notification;
6
use Gewaer\Contracts\PushNotificationsContract;
7
use Gewaer\Notifications\PushNotifications\PushNotifications as PushNotifications;
8
9
class AppsPushNotifications extends PushNotifications implements PushNotificationsContract
10
{
11
    public function __construct( Users $user,string $content, int $system_module_id = 0)
0 ignored issues
show
Coding Style introduced by
Expected 0 spaces after opening parenthesis; 1 found
Loading history...
Coding Style introduced by
Expected 1 space between comma and type hint "string"; 0 found
Loading history...
Bug introduced by
The type Gewaer\Notifications\PushNotifications\Users was not found. Maybe you did not declare it correctly or list all dependencies?

The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. excluded_paths: ["lib/*"], you can move it to the dependency path list as follows:

filter:
    dependency_paths: ["lib/*"]

For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths

Loading history...
12
    {
13
        $this->user = $user;
14
        $this->content  = $content;
15
        $this->system_module_id = $system_module_id;
0 ignored issues
show
Bug Best Practice introduced by
The property system_module_id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.
Loading history...
16
    }
17
18
    public function assemble()
19
    {
20
        return $this->content . " From Apps";
21
    }
22
}
23