Skip to content

Commit ac92d37

Browse files
author
Eldario
committed
Some fixes
1 parent 20285df commit ac92d37

File tree

4 files changed

+10
-25
lines changed

4 files changed

+10
-25
lines changed

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ FROM php:8.3-alpine
22

33
ENV COMPOSER_HOME="/tmp/composer"
44

5-
COPY --from=composer:2.8.9 /usr/bin/composer /usr/bin/composer
5+
COPY --from=composer:2.8 /usr/bin/composer /usr/bin/composer
66

77
RUN set -x \
88
&& apk add --no-cache binutils git \

docker-compose.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
version: '3.2'
2-
31
volumes:
42
composer-data:
53

src/Extensions/BodyCodeValidatorExtension.php

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -42,16 +42,11 @@ public function passes(string $attribute, $value): bool
4242

4343
$length = \mb_strlen($value, 'UTF-8');
4444

45-
if ($length >= self::LENGTH_MIN && $length <= self::LENGTH_MAX &&
46-
Strings::hasOnlyUppercaseLetters($value) &&
47-
Strings::hasOnlyLettersAndDigits($value) &&
48-
Strings::hasAtLeastOneNotZeroDigit($value) &&
49-
!$this->hasBothAlphabets($value)
50-
) {
51-
return true;
52-
}
53-
54-
return false;
45+
return $length >= self::LENGTH_MIN && $length <= self::LENGTH_MAX
46+
&& Strings::hasOnlyUppercaseLetters($value)
47+
&& Strings::hasOnlyLettersAndDigits($value)
48+
&& Strings::hasAtLeastOneNotZeroDigit($value)
49+
&& !$this->hasBothAlphabets($value);
5550
}
5651

5752
/**

src/Extensions/VinCodeValidatorExtension.php

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -34,18 +34,10 @@ public function name(): string
3434
*/
3535
public function passes(string $attribute, $value): bool
3636
{
37-
if (!\is_string($value)) {
38-
return false;
39-
}
40-
41-
if ($this->hasVinCodeFormat($value) &&
42-
Strings::hasAtLeastOneLatinLetter($value) &&
43-
Strings::hasAtLeastOneNotZeroDigit($value)
44-
) {
45-
return true;
46-
}
47-
48-
return false;
37+
return \is_string($value)
38+
&& $this->hasVinCodeFormat($value)
39+
&& Strings::hasAtLeastOneLatinLetter($value)
40+
&& Strings::hasAtLeastOneNotZeroDigit($value);
4941
}
5042

5143
/**

0 commit comments

Comments
 (0)