ReviewOS

also looking at this

ci-on/laravel-log-reader

Apply fixes from StyleCI

#1
Merged chris wants to merge analysis-qvaDZr into master
5 files +11 -12
src/Reader/LineReader.phpmodified+6-6
Changes to src/Reader/LineReader.php
@@ -17,7 +17,7 @@ class LineReader
1717 'ERROR',
1818 'WARNING',
1919 'NOTICE',
20 'INFO'
20 'INFO',
2121 ];
2222
2323 public $type;
formatting only · 8 hidden lines
Changes to src/Reader/LineReader.php
@@ -32,7 +32,7 @@ class LineReader formatting only
3232 {
3333 $this->line = $line;
3434
35 if(request()->filled('logreader_type') && request('logreader_type') !== 'all') {
35 if (request()->filled('logreader_type') && request('logreader_type') !== 'all') {
3636 $this->types = [strtoupper(request()->logreader_type)];
3737 }
3838 }
Changes to src/Reader/LineReader.php
@@ -51,9 +51,9 @@ class LineReader
5151 public function getArray()
5252 {
5353 return [
54 "date" => $this->date,
55 "type" => $this->type,
56 "message" => $this->message
54 'date' => $this->date,
55 'type' => $this->type,
56 'message' => $this->message,
5757 ];
5858 }
5959
@@ -88,7 +88,7 @@ class LineReader
8888
8989 public function retrieveMessage()
9090 {
91 $this->message = Str::replaceFirst(config('logreader.env').'.'.$this->type.": ", '', $this->LineWithoutDate());
91 $this->message = Str::replaceFirst(config('logreader.env').'.'.$this->type.': ', '', $this->LineWithoutDate());
9292 }
9393
9494 public function hasLoggerType($type)