@@ -83,27 +83,27 @@ discard block |
||
83 | 83 | * |
84 | 84 | * @throws InvalidArgumentException |
85 | 85 | */ |
86 | - public function __construct( $dumpId, |
|
86 | + public function __construct($dumpId, |
|
87 | 87 | ItemId $sourceItemId, |
88 | 88 | array $identifierPropertyIds, |
89 | 89 | $importDate, |
90 | 90 | $languageCode, |
91 | 91 | $sourceUrl, |
92 | 92 | $size, |
93 | - ItemId $licenseItemId ) { |
|
93 | + ItemId $licenseItemId) { |
|
94 | 94 | Assert::parameterElementType( |
95 | 95 | PropertyId::class, |
96 | 96 | $identifierPropertyIds, |
97 | 97 | '$identifierPropertyIds' |
98 | 98 | ); |
99 | 99 | |
100 | - $this->setDumpId( $dumpId ); |
|
100 | + $this->setDumpId($dumpId); |
|
101 | 101 | $this->sourceItemId = $sourceItemId; |
102 | 102 | $this->identifierPropertyIds = $identifierPropertyIds; |
103 | - $this->setImportDate( $importDate ); |
|
104 | - $this->setLanguageCode( $languageCode ); |
|
105 | - $this->setSourceUrl( $sourceUrl ); |
|
106 | - $this->setSize( $size ); |
|
103 | + $this->setImportDate($importDate); |
|
104 | + $this->setLanguageCode($languageCode); |
|
105 | + $this->setSourceUrl($sourceUrl); |
|
106 | + $this->setSize($size); |
|
107 | 107 | $this->licenseItemId = $licenseItemId; |
108 | 108 | } |
109 | 109 | |
@@ -168,10 +168,10 @@ discard block |
||
168 | 168 | * |
169 | 169 | * @throws InvalidArgumentException |
170 | 170 | */ |
171 | - private function setDumpId( $dumpId ) { |
|
172 | - Assert::parameterType( 'string', $dumpId, '$dumpId' ); |
|
173 | - $length = strlen( $dumpId ); |
|
174 | - if ( $length < 1 || $length > 25 ) { |
|
171 | + private function setDumpId($dumpId) { |
|
172 | + Assert::parameterType('string', $dumpId, '$dumpId'); |
|
173 | + $length = strlen($dumpId); |
|
174 | + if ($length < 1 || $length > 25) { |
|
175 | 175 | throw new InvalidArgumentException('$dumpId must be between 1 and 25 characters.'); |
176 | 176 | } |
177 | 177 | |
@@ -183,10 +183,10 @@ discard block |
||
183 | 183 | * |
184 | 184 | * @throws InvalidArgumentException |
185 | 185 | */ |
186 | - private function setLanguageCode( $languageCode ) { |
|
187 | - Assert::parameterType( 'string', $languageCode, '$languageCode' ); |
|
188 | - if( !Language::isValidCode( $languageCode ) ) { |
|
189 | - throw new InvalidArgumentException( '$languageCode is not valid.' ); |
|
186 | + private function setLanguageCode($languageCode) { |
|
187 | + Assert::parameterType('string', $languageCode, '$languageCode'); |
|
188 | + if (!Language::isValidCode($languageCode)) { |
|
189 | + throw new InvalidArgumentException('$languageCode is not valid.'); |
|
190 | 190 | } |
191 | 191 | |
192 | 192 | $this->languageCode = $languageCode; |
@@ -197,12 +197,12 @@ discard block |
||
197 | 197 | * |
198 | 198 | * @throws InvalidArgumentException |
199 | 199 | */ |
200 | - private function setImportDate( $importDate ) { |
|
201 | - Assert::parameterType( 'string', $importDate, '$importDate' ); |
|
200 | + private function setImportDate($importDate) { |
|
201 | + Assert::parameterType('string', $importDate, '$importDate'); |
|
202 | 202 | |
203 | - $timestamp = wfTimestamp( TS_MW, $importDate ); |
|
204 | - if( !$timestamp ) { |
|
205 | - throw new InvalidArgumentException( '$updatedAt has invalid timestamp format.' ); |
|
203 | + $timestamp = wfTimestamp(TS_MW, $importDate); |
|
204 | + if (!$timestamp) { |
|
205 | + throw new InvalidArgumentException('$updatedAt has invalid timestamp format.'); |
|
206 | 206 | } |
207 | 207 | |
208 | 208 | $this->importDate = $importDate; |
@@ -213,13 +213,13 @@ discard block |
||
213 | 213 | * |
214 | 214 | * @throws InvalidArgumentException |
215 | 215 | */ |
216 | - private function setSourceUrl( $sourceUrl ) { |
|
217 | - Assert::parameterType( 'string', $sourceUrl, '$sourceUrl' ); |
|
218 | - if( strlen( $sourceUrl ) > 300 ) { |
|
219 | - throw new InvalidArgumentException( '$sourceUrl must not be longer than 300 characters.' ); |
|
216 | + private function setSourceUrl($sourceUrl) { |
|
217 | + Assert::parameterType('string', $sourceUrl, '$sourceUrl'); |
|
218 | + if (strlen($sourceUrl) > 300) { |
|
219 | + throw new InvalidArgumentException('$sourceUrl must not be longer than 300 characters.'); |
|
220 | 220 | } |
221 | - if( !filter_var($sourceUrl, FILTER_VALIDATE_URL) ) { |
|
222 | - throw new InvalidArgumentException( '$sourceUrl is not a valid url.' ); |
|
221 | + if (!filter_var($sourceUrl, FILTER_VALIDATE_URL)) { |
|
222 | + throw new InvalidArgumentException('$sourceUrl is not a valid url.'); |
|
223 | 223 | } |
224 | 224 | |
225 | 225 | $this->sourceUrl = $sourceUrl; |
@@ -230,10 +230,10 @@ discard block |
||
230 | 230 | * |
231 | 231 | * @throws InvalidArgumentException |
232 | 232 | */ |
233 | - private function setSize( $size ){ |
|
234 | - Assert::parameterType( 'integer', $size, '$size' ); |
|
235 | - if( $size <= 0 ) { |
|
236 | - throw new InvalidArgumentException( '$size must be positive integer.' ); |
|
233 | + private function setSize($size) { |
|
234 | + Assert::parameterType('integer', $size, '$size'); |
|
235 | + if ($size <= 0) { |
|
236 | + throw new InvalidArgumentException('$size must be positive integer.'); |
|
237 | 237 | } |
238 | 238 | |
239 | 239 | $this->size = $size; |