@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme\Logger; |
4 | 4 | |
@@ -10,12 +10,12 @@ discard block |
||
10 | 10 | */ |
11 | 11 | class MultiLogger extends AbstractLogger implements IFlushingAwareLogger { |
12 | 12 | /** @var IFlushingAwareLogger[] */ |
13 | - private $loggers = []; |
|
13 | + private $loggers = [ ]; |
|
14 | 14 | |
15 | 15 | /** |
16 | 16 | * @param IFlushingAwareLogger ...$loggers |
17 | 17 | */ |
18 | - public function __construct( IFlushingAwareLogger ...$loggers ) { |
|
18 | + public function __construct ( IFlushingAwareLogger ...$loggers ) { |
|
19 | 19 | $this->loggers = $loggers; |
20 | 20 | } |
21 | 21 | |
@@ -23,7 +23,7 @@ discard block |
||
23 | 23 | * @inheritDoc |
24 | 24 | * @suppress PhanUnusedPublicMethodParameter |
25 | 25 | */ |
26 | - public function log( $level, $message, array $context = [] ) { |
|
26 | + public function log ( $level, $message, array $context = [ ] ) { |
|
27 | 27 | foreach ( $this->loggers as $logger ) { |
28 | 28 | $logger->log( $level, $message ); |
29 | 29 | } |
@@ -32,7 +32,7 @@ discard block |
||
32 | 32 | /** |
33 | 33 | * @inheritDoc |
34 | 34 | */ |
35 | - public function flush() : void { |
|
35 | + public function flush () : void { |
|
36 | 36 | foreach ( $this->loggers as $logger ) { |
37 | 37 | $logger->flush(); |
38 | 38 | } |
@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme\Logger; |
4 | 4 | |
@@ -17,7 +17,7 @@ discard block |
||
17 | 17 | /** |
18 | 18 | * @param string $minlevel |
19 | 19 | */ |
20 | - public function __construct( $minlevel = LogLevel::INFO ) { |
|
20 | + public function __construct ( $minlevel = LogLevel::INFO ) { |
|
21 | 21 | $this->minLevel = $this->levelToInt( $minlevel ); |
22 | 22 | } |
23 | 23 | |
@@ -25,7 +25,7 @@ discard block |
||
25 | 25 | * @inheritDoc |
26 | 26 | * @suppress PhanUnusedPublicMethodParameter |
27 | 27 | */ |
28 | - public function log( $level, $message, array $context = [] ) { |
|
28 | + public function log ( $level, $message, array $context = [ ] ) { |
|
29 | 29 | if ( $this->levelToInt( $level ) >= $this->minLevel ) { |
30 | 30 | echo $this->getFormattedMessage( $level, $message ) . "\n"; |
31 | 31 | } |
@@ -34,7 +34,7 @@ discard block |
||
34 | 34 | /** |
35 | 35 | * @inheritDoc |
36 | 36 | */ |
37 | - public function flush() : void { |
|
37 | + public function flush () : void { |
|
38 | 38 | // Everything else is printed immediately |
39 | 39 | echo "\n" . str_repeat( '-', 80 ) . "\n\n"; |
40 | 40 | } |
@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme\Logger; |
4 | 4 | |
@@ -26,7 +26,7 @@ discard block |
||
26 | 26 | * @param Page $logPage |
27 | 27 | * @param string $minlevel |
28 | 28 | */ |
29 | - public function __construct( Page $logPage, $minlevel = LogLevel::INFO ) { |
|
29 | + public function __construct ( Page $logPage, $minlevel = LogLevel::INFO ) { |
|
30 | 30 | $this->minLevel = $this->levelToInt( $minlevel ); |
31 | 31 | $this->logPage = $logPage; |
32 | 32 | } |
@@ -35,16 +35,16 @@ discard block |
||
35 | 35 | * @inheritDoc |
36 | 36 | * @suppress PhanUnusedPublicMethodParameter |
37 | 37 | */ |
38 | - public function log( $level, $message, array $context = [] ) { |
|
38 | + public function log ( $level, $message, array $context = [ ] ) { |
|
39 | 39 | if ( $this->levelToInt( $level ) >= $this->minLevel ) { |
40 | - $this->buffer[] = $this->getFormattedMessage( $level, $message ); |
|
40 | + $this->buffer[ ] = $this->getFormattedMessage( $level, $message ); |
|
41 | 41 | } |
42 | 42 | } |
43 | 43 | |
44 | 44 | /** |
45 | 45 | * @return string |
46 | 46 | */ |
47 | - protected function getOutput() : string { |
|
47 | + protected function getOutput () : string { |
|
48 | 48 | $line = str_repeat( '-', 80 ); |
49 | 49 | return implode( "\n", $this->buffer ) . "\n$line\n\n"; |
50 | 50 | } |
@@ -52,7 +52,7 @@ discard block |
||
52 | 52 | /** |
53 | 53 | * @inheritDoc |
54 | 54 | */ |
55 | - public function flush() : void { |
|
55 | + public function flush () : void { |
|
56 | 56 | if ( $this->buffer ) { |
57 | 57 | $this->logPage->edit( [ |
58 | 58 | 'appendtext' => $this->getOutput(), |
@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme\Task; |
4 | 4 | |
@@ -13,7 +13,7 @@ discard block |
||
13 | 13 | */ |
14 | 14 | abstract class TaskBase extends ContextSource { |
15 | 15 | /** @var string[] */ |
16 | - protected $errors = []; |
|
16 | + protected $errors = [ ]; |
|
17 | 17 | /** @var TaskDataProvider */ |
18 | 18 | protected $dataProvider; |
19 | 19 | |
@@ -24,7 +24,7 @@ discard block |
||
24 | 24 | * @param Wiki $wiki |
25 | 25 | * @param TaskDataProvider $dataProvider |
26 | 26 | */ |
27 | - final public function __construct( |
|
27 | + final public function __construct ( |
|
28 | 28 | LoggerInterface $logger, |
29 | 29 | Wiki $wiki, |
30 | 30 | TaskDataProvider $dataProvider |
@@ -38,7 +38,7 @@ discard block |
||
38 | 38 | * |
39 | 39 | * @return TaskResult |
40 | 40 | */ |
41 | - final public function run() : TaskResult { |
|
41 | + final public function run () : TaskResult { |
|
42 | 42 | $class = ( new \ReflectionClass( $this ) )->getShortName(); |
43 | 43 | $opName = $this->getOperationName(); |
44 | 44 | $this->getLogger()->info( "Starting $opName $class" ); |
@@ -69,19 +69,19 @@ discard block |
||
69 | 69 | * |
70 | 70 | * @return int One of the STATUS_* constants |
71 | 71 | */ |
72 | - abstract protected function runInternal() : int; |
|
72 | + abstract protected function runInternal () : int; |
|
73 | 73 | |
74 | 74 | /** |
75 | 75 | * How this operation should be called in logs |
76 | 76 | * |
77 | 77 | * @return string |
78 | 78 | */ |
79 | - abstract public function getOperationName() : string; |
|
79 | + abstract public function getOperationName () : string; |
|
80 | 80 | |
81 | 81 | /** |
82 | 82 | * @return TaskDataProvider |
83 | 83 | */ |
84 | - protected function getDataProvider() : TaskDataProvider { |
|
84 | + protected function getDataProvider () : TaskDataProvider { |
|
85 | 85 | return $this->dataProvider; |
86 | 86 | } |
87 | 87 | } |
@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme\Logger; |
4 | 4 | |
@@ -8,5 +8,5 @@ discard block |
||
8 | 8 | * Logger aware of flushing. Can be declared empty if the buffer is flushed immediately. |
9 | 9 | */ |
10 | 10 | interface IFlushingAwareLogger extends LoggerInterface { |
11 | - public function flush() : void; |
|
11 | + public function flush () : void; |
|
12 | 12 | } |