@@ -1,4 +1,4 @@ discard block |
||
1 | -<?php declare( strict_types=1 ); |
|
1 | +<?php declare(strict_types=1); |
|
2 | 2 | |
3 | 3 | namespace BotRiconferme; |
4 | 4 | |
@@ -65,14 +65,14 @@ discard block |
||
65 | 65 | /** |
66 | 66 | * @return bool |
67 | 67 | */ |
68 | - public static function isCLI() : bool { |
|
68 | + public static function isCLI () : bool { |
|
69 | 69 | return PHP_SAPI === 'cli'; |
70 | 70 | } |
71 | 71 | |
72 | 72 | /** |
73 | 73 | * Populate options and check for required ones |
74 | 74 | */ |
75 | - public function __construct() { |
|
75 | + public function __construct () { |
|
76 | 76 | $opts = getopt( self::SHORT_OPTS, self::LONG_OPTS ); |
77 | 77 | $this->checkRequiredOpts( $opts ); |
78 | 78 | $this->checkConflictingOpts( $opts ); |
@@ -83,7 +83,7 @@ discard block |
||
83 | 83 | /** |
84 | 84 | * @param array $opts |
85 | 85 | */ |
86 | - private function checkRequiredOpts( array $opts ) : void { |
|
86 | + private function checkRequiredOpts ( array $opts ) : void { |
|
87 | 87 | $missingOpts = array_diff( self::REQUIRED_OPTS, array_keys( $opts ) ); |
88 | 88 | if ( $missingOpts ) { |
89 | 89 | exit( 'Required options missing: ' . implode( ', ', $missingOpts ) ); |
@@ -105,7 +105,7 @@ discard block |
||
105 | 105 | /** |
106 | 106 | * @param array $opts |
107 | 107 | */ |
108 | - private function checkConflictingOpts( array $opts ) : void { |
|
108 | + private function checkConflictingOpts ( array $opts ) : void { |
|
109 | 109 | $this->checkNotBothSet( $opts, 'password', 'use-password-file' ); |
110 | 110 | if ( array_key_exists( 'use-password-file', $opts ) && !file_exists( self::PASSWORD_FILE ) ) { |
111 | 111 | exit( 'Please create the password file (' . self::PASSWORD_FILE . ')' ); |
@@ -126,7 +126,7 @@ discard block |
||
126 | 126 | * @param string $first |
127 | 127 | * @param string $second |
128 | 128 | */ |
129 | - private function checkNotBothSet( array $opts, string $first, string $second ) : void { |
|
129 | + private function checkNotBothSet ( array $opts, string $first, string $second ) : void { |
|
130 | 130 | if ( array_key_exists( $first, $opts ) && array_key_exists( $second, $opts ) ) { |
131 | 131 | exit( "Can only use one of '$first' and '$second'" ); |
132 | 132 | } |
@@ -135,16 +135,16 @@ discard block |
||
135 | 135 | /** |
136 | 136 | * @param array &$opts |
137 | 137 | */ |
138 | - private function canonicalize( array &$opts ) : void { |
|
138 | + private function canonicalize ( array &$opts ) : void { |
|
139 | 139 | if ( array_key_exists( 'use-password-file', $opts ) ) { |
140 | 140 | $pw = trim( file_get_contents( self::PASSWORD_FILE ) ); |
141 | - $opts['password'] = $pw; |
|
142 | - unset( $opts['use-password-file'] ); |
|
141 | + $opts[ 'password' ] = $pw; |
|
142 | + unset( $opts[ 'use-password-file' ] ); |
|
143 | 143 | } |
144 | 144 | if ( array_key_exists( 'use-private-password-file', $opts ) ) { |
145 | 145 | $pw = trim( file_get_contents( self::PRIVATE_PASSWORD_FILE ) ); |
146 | - $opts['private-password'] = $pw; |
|
147 | - unset( $opts['use-private-password-file'] ); |
|
146 | + $opts[ 'private-password' ] = $pw; |
|
147 | + unset( $opts[ 'use-private-password-file' ] ); |
|
148 | 148 | } |
149 | 149 | } |
150 | 150 | |
@@ -153,14 +153,14 @@ discard block |
||
153 | 153 | * @param mixed|null $default |
154 | 154 | * @return mixed |
155 | 155 | */ |
156 | - public function getOpt( string $opt, $default = null ) { |
|
157 | - return $this->opts[$opt] ?? $default; |
|
156 | + public function getOpt ( string $opt, $default = null ) { |
|
157 | + return $this->opts[ $opt ] ?? $default; |
|
158 | 158 | } |
159 | 159 | |
160 | 160 | /** |
161 | 161 | * @return array Either [ 'task' => taskname ] or [ 'subtask' => subtaskname ] |
162 | 162 | */ |
163 | - public function getTaskOpt() : array { |
|
163 | + public function getTaskOpt () : array { |
|
164 | 164 | return array_intersect_key( |
165 | 165 | $this->opts, |
166 | 166 | [ 'task' => true, 'subtask' => true ] |
@@ -170,7 +170,7 @@ discard block |
||
170 | 170 | /** |
171 | 171 | * @return string|null |
172 | 172 | */ |
173 | - public function getURL() : ?string { |
|
173 | + public function getURL () : ?string { |
|
174 | 174 | return $this->getOpt( 'force-url' ); |
175 | 175 | } |
176 | 176 | } |