Skip to content

Commit

Permalink
Fix psalm issues
Browse files Browse the repository at this point in the history
  • Loading branch information
Slamdunk committed Oct 4, 2023
1 parent c6100a6 commit fb35644
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/TestAsset/Annotation/InputFilter.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@

use Laminas\InputFilter\InputFilter as Ancestor;

/**
* @extends Ancestor<null>
*/
class InputFilter extends Ancestor

Check failure on line 12 in test/TestAsset/Annotation/InputFilter.php

View workflow job for this annotation

GitHub Actions / ci / QA Checks (Psalm [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-action@v1, ...

TooManyTemplateParams

test/TestAsset/Annotation/InputFilter.php:12:7: TooManyTemplateParams: LaminasTest\Form\TestAsset\Annotation\InputFilter has too many template params when extending Laminas\InputFilter\InputFilter, expecting 0 (see https://psalm.dev/184)

Check failure on line 12 in test/TestAsset/Annotation/InputFilter.php

View workflow job for this annotation

GitHub Actions / ci / QA Checks (Psalm [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-action@v1, ...

TooManyTemplateParams

test/TestAsset/Annotation/InputFilter.php:12:7: TooManyTemplateParams: LaminasTest\Form\TestAsset\Annotation\InputFilter has too many template params when extending Laminas\InputFilter\InputFilter, expecting 0 (see https://psalm.dev/184)
{
}
3 changes: 3 additions & 0 deletions test/TestAsset/InputFilter.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@

use Laminas\InputFilter\InputFilter as InputFilterBase;

/**
* @extends InputFilterBase<null>
*/
class InputFilter extends InputFilterBase

Check failure on line 12 in test/TestAsset/InputFilter.php

View workflow job for this annotation

GitHub Actions / ci / QA Checks (Psalm [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-action@v1, ...

TooManyTemplateParams

test/TestAsset/InputFilter.php:12:7: TooManyTemplateParams: LaminasTest\Form\TestAsset\InputFilter has too many template params when extending Laminas\InputFilter\InputFilter, expecting 0 (see https://psalm.dev/184)

Check failure on line 12 in test/TestAsset/InputFilter.php

View workflow job for this annotation

GitHub Actions / ci / QA Checks (Psalm [8.1, locked], ubuntu-latest, laminas/laminas-continuous-integration-action@v1, ...

TooManyTemplateParams

test/TestAsset/InputFilter.php:12:7: TooManyTemplateParams: LaminasTest\Form\TestAsset\InputFilter has too many template params when extending Laminas\InputFilter\InputFilter, expecting 0 (see https://psalm.dev/184)
{
}
1 change: 1 addition & 0 deletions test/View/Helper/Factory/FormElementErrorsFactoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ public function testFactoryShouldCreateHelperWithTranslateOption(
if ($result['translate_messages'] === true) {
$errorMessage = 'tranlated message';
}
self::assertIsString($result['message_open_format']);
$expected = sprintf(
'%s%s%s',
str_replace('%s', '', $result['message_open_format']),
Expand Down

0 comments on commit fb35644

Please sign in to comment.