@@ -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 | } |