Failed Conditions
Pull Request — master (#298)
by Maximo
02:56
created
src/Cli/jobs/Pusher.php 1 patch
Doc Comments   -3 removed lines patch added patch discarded remove patch
@@ -33,9 +33,6 @@
 block discarded – undo
33 33
     /**
34 34
      * Constructor setup info for Pusher.
35 35
      *
36
-     * @param string $channel
37
-     * @param string $event
38
-     * @param array $params
39 36
      */
40 37
     public function __construct(PusherNotification $pusherNotification)
41 38
     {
Please login to merge, or discard this patch.
src/Cli/jobs/PushNotifications.php 2 patches
Doc Comments   +1 added lines, -4 removed lines patch added patch discarded remove patch
@@ -40,9 +40,6 @@  discard block
 block discarded – undo
40 40
     /**
41 41
      * Constructor setup info for Pusher.
42 42
      *
43
-     * @param string $channel
44
-     * @param string $event
45
-     * @param array $params
46 43
      */
47 44
     public function __construct(PushNotification $pushNotification)
48 45
     {
@@ -54,7 +51,7 @@  discard block
 block discarded – undo
54 51
     /**
55 52
      * Handle the pusher request.
56 53
      *
57
-     * @return void
54
+     * @return boolean
58 55
      */
59 56
     public function handle()
60 57
     {
Please login to merge, or discard this patch.
Unused Use Statements   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -4,16 +4,16 @@
 block discarded – undo
4 4
 
5 5
 use Canvas\Contracts\Queue\QueueableJobInterfase;
6 6
 use Canvas\Jobs\Job;
7
-use Phalcon\Di;
7
+use Canvas\Models\UserLinkedSources;
8
+use Canvas\Notifications\PushNotification;
9
+use Exception;
8 10
 use GuzzleHttp\Client as GuzzleClient;
9 11
 use Http\Adapter\Guzzle6\Client as GuzzleAdapter;
10 12
 use Http\Client\Common\HttpMethodsClient as HttpClient;
11 13
 use Http\Message\MessageFactory\GuzzleMessageFactory;
12 14
 use OneSignal\Config;
13 15
 use OneSignal\OneSignal;
14
-use Canvas\Models\UserLinkedSources;
15
-use Canvas\Notifications\PushNotification;
16
-use Exception;
16
+use Phalcon\Di;
17 17
 
18 18
 class PushNotifications extends Job implements QueueableJobInterfase
19 19
 {
Please login to merge, or discard this patch.
src/Cli/tasks/QueueTask.php 1 patch
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3,9 +3,9 @@
 block discarded – undo
3 3
 namespace Canvas\Cli\Tasks;
4 4
 
5 5
 use Canvas\Contracts\Queue\QueueableJobInterfase;
6
-use Phalcon\Cli\Task as PhTask;
7 6
 use Canvas\Models\Users;
8 7
 use Canvas\Queue\Queue;
8
+use Phalcon\Cli\Task as PhTask;
9 9
 use Phalcon\Mvc\Model;
10 10
 use Throwable;
11 11
 
Please login to merge, or discard this patch.
src/Contracts/Controllers/ProcessOutputMapperTrait.php 1 patch
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,8 +4,8 @@
 block discarded – undo
4 4
 
5 5
 namespace Canvas\Contracts\Controllers;
6 6
 
7
-use Canvas\Exception\ServerErrorHttpException;
8 7
 use AutoMapperPlus\DataType;
8
+use Canvas\Exception\ServerErrorHttpException;
9 9
 use StdClass;
10 10
 
11 11
 trait ProcessOutputMapperTrait
Please login to merge, or discard this patch.
src/Contracts/Notifications/NotificationInterfase.php 1 patch
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -10,7 +10,7 @@  discard block
 block discarded – undo
10 10
     /**
11 11
      * Undocumented function
12 12
      *
13
-     * @return void
13
+     * @return string
14 14
      */
15 15
     public function message(): string;
16 16
 
@@ -18,8 +18,14 @@  discard block
 block discarded – undo
18 18
 
19 19
     public function trigger(): bool;
20 20
     
21
+    /**
22
+     * @return void
23
+     */
21 24
     public function setTo(Users $user): void;
22 25
 
26
+    /**
27
+     * @return void
28
+     */
23 29
     public function setFrom(Users $user): void;
24 30
 
25 31
 }
26 32
\ No newline at end of file
Please login to merge, or discard this patch.
src/Core/functions.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
      * @param string     $variable
38 38
      * @param mixed|null $default
39 39
      *
40
-     * @return mixed
40
+     * @return string|null
41 41
      */
42 42
     function envValue(string $variable, $default = null)
43 43
     {
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      * @param string $resource
65 65
      * @param int    $recordId
66 66
      *
67
-     * @return array|false|mixed|string
67
+     * @return string
68 68
      */
69 69
     function appUrl(string $resource, int $recordId)
70 70
     {
Please login to merge, or discard this patch.
src/CustomFields/AbstractCustomFieldsModel.php 1 patch
Unused Use Statements   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,9 +3,9 @@
 block discarded – undo
3 3
 
4 4
 namespace Canvas\CustomFields;
5 5
 
6
-use Canvas\Models\CustomFieldsModules;
7
-use Canvas\Models\AbstractModel;
8 6
 use Baka\Database\Contracts\CustomFields\CustomFieldsTrait;
7
+use Canvas\Models\AbstractModel;
8
+use Canvas\Models\CustomFieldsModules;
9 9
 use PDO;
10 10
 
11 11
 /**
Please login to merge, or discard this patch.
src/EventsManager.php 1 patch
Unused Use Statements   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,8 +4,8 @@
 block discarded – undo
4 4
 
5 5
 namespace Canvas;
6 6
 
7
-use Canvas\Traits\EventManagerAwareTrait;
8 7
 use Canvas\Queue\Queue;
8
+use Canvas\Traits\EventManagerAwareTrait;
9 9
 use Phalcon\Di;
10 10
 
11 11
 /**
Please login to merge, or discard this patch.
src/Exception/HttpException.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
     /**
16 16
      * Get the http status code of the exception.
17 17
      *
18
-     * @return string
18
+     * @return integer
19 19
      */
20 20
     public function getHttpCode(): int
21 21
     {
Please login to merge, or discard this patch.