Skip to content

Commit 090d399

Browse files
committed
feat: binary file size validation support
1 parent bac0903 commit 090d399

File tree

2 files changed

+681
-35
lines changed

2 files changed

+681
-35
lines changed

src/Illuminate/Validation/Rules/File.php

Lines changed: 109 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,16 @@ class File implements Rule, DataAwareRule, ValidatorAwareRule
1717
{
1818
use Conditionable, Macroable;
1919

20+
/**
21+
* Binary units flag used for size validation.
22+
*/
23+
public const BINARY = 'binary';
24+
25+
/**
26+
* International units flag used for size validation.
27+
*/
28+
public const INTERNATIONAL = 'international';
29+
2030
/**
2131
* The MIME types that the given file should match. This array may also contain file extensions.
2232
*
@@ -45,6 +55,11 @@ class File implements Rule, DataAwareRule, ValidatorAwareRule
4555
*/
4656
protected $maximumFileSize = null;
4757

58+
/**
59+
* The units used for size validation.
60+
*/
61+
protected string $units = self::INTERNATIONAL;
62+
4863
/**
4964
* An array of custom rules that will be merged into the validation rules.
5065
*
@@ -150,83 +165,138 @@ public function extensions($extensions)
150165
return $this;
151166
}
152167

168+
/**
169+
* Set the units for size validation to binary (instance method).
170+
*/
171+
public function binary(): static
172+
{
173+
$this->units = self::BINARY;
174+
return $this;
175+
}
176+
177+
/**
178+
* Set the units for size validation to international (instance method).
179+
*/
180+
public function international(): static
181+
{
182+
$this->units = self::INTERNATIONAL;
183+
return $this;
184+
}
185+
186+
187+
153188
/**
154189
* Indicate that the uploaded file should be exactly a certain size in kilobytes.
155-
*
156-
* @param string|int $size
157-
* @return $this
158190
*/
159-
public function size($size)
191+
public function size(string|int $size, ?string $units = null): static
160192
{
161-
$this->minimumFileSize = $this->toKilobytes($size);
193+
$this->minimumFileSize = $this->toKilobytes($size, $this->units($units));
162194
$this->maximumFileSize = $this->minimumFileSize;
163195

164196
return $this;
165197
}
166198

167199
/**
168200
* Indicate that the uploaded file should be between a minimum and maximum size in kilobytes.
169-
*
170-
* @param string|int $minSize
171-
* @param string|int $maxSize
172-
* @return $this
173201
*/
174-
public function between($minSize, $maxSize)
202+
public function between(string|int $minSize, string|int $maxSize, ?string $units = null): static
175203
{
176-
$this->minimumFileSize = $this->toKilobytes($minSize);
177-
$this->maximumFileSize = $this->toKilobytes($maxSize);
204+
$this->minimumFileSize = $this->toKilobytes($minSize, $this->units($units));
205+
$this->maximumFileSize = $this->toKilobytes($maxSize, $this->units($units));
178206

179207
return $this;
180208
}
181209

182210
/**
183211
* Indicate that the uploaded file should be no less than the given number of kilobytes.
184-
*
185-
* @param string|int $size
186-
* @return $this
187212
*/
188-
public function min($size)
213+
public function min(string|int $size, ?string $units = null): static
189214
{
190-
$this->minimumFileSize = $this->toKilobytes($size);
215+
$this->minimumFileSize = $this->toKilobytes($size, $this->units($units));
191216

192217
return $this;
193218
}
194219

195220
/**
196221
* Indicate that the uploaded file should be no more than the given number of kilobytes.
197-
*
198-
* @param string|int $size
199-
* @return $this
200222
*/
201-
public function max($size)
223+
public function max(string|int $size, ?string $units = null): static
202224
{
203-
$this->maximumFileSize = $this->toKilobytes($size);
225+
$this->maximumFileSize = $this->toKilobytes($size, $this->units($units));
204226

205227
return $this;
206228
}
207229

230+
protected function units(?string $units = null): string
231+
{
232+
return $units ?? $this->units;
233+
}
234+
235+
/**
236+
* Converts whole numbers to integers for exact arithmetic while keeping
237+
* fractional numbers as floats; also provides overflow protection by
238+
* falling back to float arithmetic for values too large for integer range.
239+
*/
240+
protected function prepareValueForPrecision(float $value): float|int
241+
{
242+
return $value > PHP_INT_MAX || $value < PHP_INT_MIN || ((float) (int) $value) !== $value
243+
? $value
244+
: (int) $value;
245+
}
246+
208247
/**
209248
* Convert a potentially human-friendly file size to kilobytes.
210-
*
211-
* @param string|int $size
212-
* @return mixed
213249
*/
214-
protected function toKilobytes($size)
250+
protected function toKilobytes(string|int $size, string $units): float|int
215251
{
216252
if (! is_string($size)) {
217253
return $size;
218254
}
219255

220256
$size = strtolower(trim($size));
221257

222-
$value = floatval($size);
258+
$value = (float) $size;
259+
260+
return $units === self::BINARY
261+
? $this->toBinaryKilobytes($size, $value)
262+
: $this->toInternationalKilobytes($size, $value);
263+
}
264+
265+
/**
266+
* Convert a human-friendly file size to kilobytes using the International System.
267+
*/
268+
protected function toInternationalKilobytes(string $size, float $value): float|int
269+
{
270+
$value = $this->prepareValueForPrecision($value);
223271

224272
return round(match (true) {
225-
Str::endsWith($size, 'kb') => $value * 1,
273+
is_numeric($size) => $value,
274+
Str::endsWith($size, 'kb') => $value,
226275
Str::endsWith($size, 'mb') => $value * 1_000,
227276
Str::endsWith($size, 'gb') => $value * 1_000_000,
228277
Str::endsWith($size, 'tb') => $value * 1_000_000_000,
229-
default => throw new InvalidArgumentException('Invalid file size suffix.'),
278+
default => throw new InvalidArgumentException(
279+
'Invalid file size suffix. Valid suffixes are: KB, MB, GB, TB (case insensitive).'
280+
),
281+
});
282+
}
283+
284+
/**
285+
* Convert a human-friendly file size to kilobytes using the Binary System.
286+
*/
287+
protected function toBinaryKilobytes(string $size, float $value): float|int
288+
{
289+
$value = $this->prepareValueForPrecision($value);
290+
291+
return round(match (true) {
292+
is_numeric($size) => $value,
293+
Str::endsWith($size, 'kb') => $value,
294+
Str::endsWith($size, 'mb') => $value * 1_024,
295+
Str::endsWith($size, 'gb') => $value * 1_048_576,
296+
Str::endsWith($size, 'tb') => $value * 1_073_741_824,
297+
default => throw new InvalidArgumentException(
298+
'Invalid file size suffix. Valid suffixes are: KB, MB, GB, TB (case insensitive).'
299+
),
230300
});
231301
}
232302

@@ -283,14 +353,18 @@ protected function buildValidationRules()
283353
$rules[] = 'extensions:'.implode(',', array_map(strtolower(...), $this->allowedExtensions));
284354
}
285355

286-
$rules[] = match (true) {
287-
is_null($this->minimumFileSize) && is_null($this->maximumFileSize) => null,
288-
is_null($this->maximumFileSize) => "min:{$this->minimumFileSize}",
289-
is_null($this->minimumFileSize) => "max:{$this->maximumFileSize}",
290-
$this->minimumFileSize !== $this->maximumFileSize => "between:{$this->minimumFileSize},{$this->maximumFileSize}",
291-
default => "size:{$this->minimumFileSize}",
356+
$rule = match (true) {
357+
$this->minimumFileSize === null && $this->maximumFileSize === null => null,
358+
$this->maximumFileSize === null => "min:{$this->minimumFileSize}",
359+
$this->minimumFileSize === null => "max:{$this->maximumFileSize}",
360+
$this->minimumFileSize === $this->maximumFileSize => "size:{$this->minimumFileSize}",
361+
default => "between:{$this->minimumFileSize},{$this->maximumFileSize}",
292362
};
293363

364+
if ($rule) {
365+
$rules[] = $rule;
366+
}
367+
294368
return array_merge(array_filter($rules), $this->customRules);
295369
}
296370

0 commit comments

Comments
 (0)