Completed
Push — master ( 473ec9...c952f4 )
by Seong
09:31
created
src/Events/NewReply.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -24,8 +24,8 @@
 block discarded – undo
24 24
     /**
25 25
      * Create a new event instance.
26 26
      * ClientAction constructor.
27
-     * @param Client $client
28 27
      * @param $action
28
+     * @param \Illuminate\Database\Eloquent\Model $reply
29 29
      */
30 30
     public function __construct($reply)
31 31
     {
Please login to merge, or discard this patch.
Unused Use Statements   +2 added lines, -5 removed lines patch added patch discarded remove patch
@@ -3,12 +3,9 @@
 block discarded – undo
3 3
 namespace Seongbae\Discuss\Events;
4 4
 
5 5
 use Illuminate\Broadcasting\Channel;
6
-use Illuminate\Queue\SerializesModels;
7
-use Illuminate\Broadcasting\PrivateChannel;
8
-use Illuminate\Broadcasting\PresenceChannel;
9 6
 use Illuminate\Broadcasting\InteractsWithSockets;
10
-use Illuminate\Contracts\Broadcasting\ShouldBroadcast;
11
-use Seongbae\Discuss\Models\Reply;
7
+use Illuminate\Broadcasting\PrivateChannel;
8
+use Illuminate\Queue\SerializesModels;
12 9
 
13 10
 class NewReply
14 11
 {
Please login to merge, or discard this patch.
src/Events/NewThread.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -24,7 +24,6 @@
 block discarded – undo
24 24
     /**
25 25
      * Create a new event instance.
26 26
      * ClientAction constructor.
27
-     * @param Client $client
28 27
      * @param $action
29 28
      */
30 29
     public function __construct($thread)
Please login to merge, or discard this patch.
Unused Use Statements   +2 added lines, -5 removed lines patch added patch discarded remove patch
@@ -3,12 +3,9 @@
 block discarded – undo
3 3
 namespace Seongbae\Discuss\Events;
4 4
 
5 5
 use Illuminate\Broadcasting\Channel;
6
-use Illuminate\Queue\SerializesModels;
7
-use Illuminate\Broadcasting\PrivateChannel;
8
-use Illuminate\Broadcasting\PresenceChannel;
9 6
 use Illuminate\Broadcasting\InteractsWithSockets;
10
-use Illuminate\Contracts\Broadcasting\ShouldBroadcast;
11
-use Seongbae\Discuss\Models\Reply;
7
+use Illuminate\Broadcasting\PrivateChannel;
8
+use Illuminate\Queue\SerializesModels;
12 9
 
13 10
 class NewThread
14 11
 {
Please login to merge, or discard this patch.
src/Http/Controllers/RepliesController.php 2 patches
Doc Comments   +1 added lines, -4 removed lines patch added patch discarded remove patch
@@ -43,8 +43,6 @@  discard block
 block discarded – undo
43 43
     /**
44 44
      * Update the specified resource in storage.
45 45
      *
46
-     * @param  \Illuminate\Http\Request  $request
47
-     * @param  \App\Thread  $thread
48 46
      * @return \Illuminate\Http\Response
49 47
      */
50 48
     public function update(Reply $reply)
@@ -59,8 +57,7 @@  discard block
 block discarded – undo
59 57
     /**
60 58
      * Remove the specified resource from storage.
61 59
      *
62
-     * @param  \App\Thread  $thread
63
-     * @return \Illuminate\Http\Response
60
+     * @return \Illuminate\Http\RedirectResponse
64 61
      */
65 62
     public function destroy(Reply $reply)
66 63
     {
Please login to merge, or discard this patch.
Unused Use Statements   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -3,11 +3,11 @@
 block discarded – undo
3 3
 namespace Seongbae\Discuss\Http\Controllers;
4 4
 
5 5
 
6
+use App\Http\Controllers\Controller;
6 7
 use Illuminate\Http\Request;
7
-use Seongbae\Discuss\Models\Thread;
8 8
 use Illuminate\Support\Facades\Auth;
9 9
 use Seongbae\Discuss\Models\Reply;
10
-use App\Http\Controllers\Controller;
10
+use Seongbae\Discuss\Models\Thread;
11 11
 
12 12
 class RepliesController extends Controller
13 13
 {
Please login to merge, or discard this patch.
src/Http/Controllers/ThreadsController.php 2 patches
Unused Use Statements   +3 added lines, -4 removed lines patch added patch discarded remove patch
@@ -2,12 +2,11 @@
 block discarded – undo
2 2
 
3 3
 namespace Seongbae\Discuss\Http\Controllers;
4 4
 
5
-use Seongbae\Discuss\Models\Thread;
6
-use Illuminate\Http\Request;
7
-use Seongbae\Discuss\Models\Channel;
8 5
 use App\Http\Controllers\Controller;
9 6
 use Auth;
10
-use Illuminate\Support\Facades\Log;
7
+use Illuminate\Http\Request;
8
+use Seongbae\Discuss\Models\Channel;
9
+use Seongbae\Discuss\Models\Thread;
11 10
 
12 11
 class ThreadsController extends Controller
13 12
 {
Please login to merge, or discard this patch.
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
      * Store a newly created resource in storage.
75 75
      *
76 76
      * @param  \Illuminate\Http\Request  $request
77
-     * @return \Illuminate\Http\Response
77
+     * @return \Illuminate\Http\RedirectResponse
78 78
      */
79 79
     public function store(Request $request)
80 80
     {
@@ -135,7 +135,6 @@  discard block
 block discarded – undo
135 135
     /**
136 136
      * Update the specified resource in storage.
137 137
      *
138
-     * @param  \Illuminate\Http\Request  $request
139 138
      * @param  \App\Thread  $thread
140 139
      * @return \Illuminate\Http\Response
141 140
      */
Please login to merge, or discard this patch.
src/Notifications/NewReplyNotification.php 2 patches
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,6 +21,7 @@  discard block
 block discarded – undo
21 21
     /**
22 22
      * Create a new notification instance.
23 23
      *
24
+     * @param string $msg
24 25
      * @return void
25 26
      */
26 27
     public function __construct($reply, $msg=null)
@@ -33,7 +34,7 @@  discard block
 block discarded – undo
33 34
      * Get the notification's delivery channels.
34 35
      *
35 36
      * @param  mixed  $notifiable
36
-     * @return array
37
+     * @return string[]
37 38
      */
38 39
     public function via($notifiable)
39 40
     {
Please login to merge, or discard this patch.
Unused Use Statements   -4 removed lines patch added patch discarded remove patch
@@ -3,12 +3,8 @@
 block discarded – undo
3 3
 namespace Seongbae\Discuss\Notifications;
4 4
 
5 5
 use Illuminate\Bus\Queueable;
6
-use Illuminate\Contracts\Queue\ShouldQueue;
7 6
 use Illuminate\Notifications\Messages\MailMessage;
8 7
 use Illuminate\Notifications\Notification;
9
-use Illuminate\Support\Facades\Log;
10
-use App\Mail\TeamMemberAdded;
11
-use Helper;
12 8
 
13 9
 class NewReplyNotification extends Notification
14 10
 {
Please login to merge, or discard this patch.
src/Notifications/NewThreadNotification.php 2 patches
Doc Comments   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,6 +21,7 @@  discard block
 block discarded – undo
21 21
     /**
22 22
      * Create a new notification instance.
23 23
      *
24
+     * @param string $msg
24 25
      * @return void
25 26
      */
26 27
     public function __construct($thread, $msg=null)
@@ -33,7 +34,7 @@  discard block
 block discarded – undo
33 34
      * Get the notification's delivery channels.
34 35
      *
35 36
      * @param  mixed  $notifiable
36
-     * @return array
37
+     * @return string[]
37 38
      */
38 39
     public function via($notifiable)
39 40
     {
Please login to merge, or discard this patch.
Unused Use Statements   -4 removed lines patch added patch discarded remove patch
@@ -3,12 +3,8 @@
 block discarded – undo
3 3
 namespace Seongbae\Discuss\Notifications;
4 4
 
5 5
 use Illuminate\Bus\Queueable;
6
-use Illuminate\Contracts\Queue\ShouldQueue;
7 6
 use Illuminate\Notifications\Messages\MailMessage;
8 7
 use Illuminate\Notifications\Notification;
9
-use Illuminate\Support\Facades\Log;
10
-use App\Mail\TeamMemberAdded;
11
-use Helper;
12 8
 
13 9
 class NewThreadNotification extends Notification
14 10
 {
Please login to merge, or discard this patch.