edited the mapping of event, url generation works now #3043
main.yml
on: push
Coding Guidelines Check
13s
FE Asset Linting
17s
FE asset formatting check
13s
PHPMD
10s
Static Code Analysis
31s
Run frontend type check
28s
Run frontend unit- and functional-tests
50s
Matrix: Run E2E-tests via Cypress
Annotations
18 errors and 7 warnings
PHPMD
Process completed with exit code 2.
|
FE asset formatting check
Process completed with exit code 1.
|
Coding Guidelines Check
Process completed with exit code 8.
|
TooFewArguments:
src/Mealz/MealBundle/Controller/MealGuestController.php#L73
src/Mealz/MealBundle/Controller/MealGuestController.php:73:50: TooFewArguments: Too few arguments for App\Mealz\MealBundle\Repository\GuestInvitationRepositoryInterface::findOrCreateInvitation - expecting eventParticipation to be passed (see https://psalm.dev/025)
|
TooFewArguments:
src/Mealz/MealBundle/Controller/MealGuestController.php#L73
src/Mealz/MealBundle/Controller/MealGuestController.php:73:50: TooFewArguments: Too few arguments for method App\Mealz\MealBundle\Repository\GuestInvitationRepositoryInterface::findorcreateinvitation saw 2 (see https://psalm.dev/025)
|
TooManyArguments:
src/Mealz/MealBundle/Controller/MealGuestController.php#L124
src/Mealz/MealBundle/Controller/MealGuestController.php:124:45: TooManyArguments: Too many arguments for method Doctrine\Persistence\ObjectRepository::find - saw 2 (see https://psalm.dev/026)
|
InvalidReturnType:
src/Mealz/MealBundle/Entity/Day.php#L178
src/Mealz/MealBundle/Entity/Day.php:178:22: InvalidReturnType: The declared return type 'array{dateTime: DateTime, enabled: bool, event: int|null, lockParticipationDateTime: DateTime, meals: array<''|int, non-empty-list<array<array-key, mixed>>>, week: int|null}' for App\Mealz\MealBundle\Entity\Day::jsonSerialize is incorrect, got 'array{dateTime: DateTime, dayId: int|null, enabled: bool, events: array<int, array<array-key, mixed>>, lockParticipationDateTime: DateTime, meals: array<''|int, non-empty-list<array{dateTime: DateTime, day: int|null, dish: null|string, id: int|null, lockTime: DateTime, participationLimit: int}>>, week: int|null}' which is different due to additional array shape fields (events, dayId) (see https://psalm.dev/011)
|
InvalidReturnStatement:
src/Mealz/MealBundle/Entity/Day.php#L202
src/Mealz/MealBundle/Entity/Day.php:202:16: InvalidReturnStatement: The inferred type 'array{dateTime: DateTime, dayId: int|null, enabled: bool, events: array<int, array<array-key, mixed>>, lockParticipationDateTime: DateTime, meals: array<''|int, non-empty-list<array{dateTime: DateTime, day: int|null, dish: null|string, id: int|null, lockTime: DateTime, participationLimit: int}>>, week: int|null}' does not match the declared return type 'array{dateTime: DateTime, enabled: bool, event: int|null, lockParticipationDateTime: DateTime, meals: array<''|int, non-empty-list<array<array-key, mixed>>>, week: int|null}' for App\Mealz\MealBundle\Entity\Day::jsonSerialize due to additional array shape fields (events, dayId) (see https://psalm.dev/128)
|
InvalidNullableReturnType:
src/Mealz/MealBundle/Entity/GuestInvitation.php#L50
src/Mealz/MealBundle/Entity/GuestInvitation.php:50:47: InvalidNullableReturnType: The declared return type 'App\Mealz\MealBundle\Entity\EventParticipation' for App\Mealz\MealBundle\Entity\GuestInvitation::getEventParticipation is not nullable, but 'App\Mealz\MealBundle\Entity\EventParticipation|null' contains null (see https://psalm.dev/144)
|
NullableReturnStatement:
src/Mealz/MealBundle/Entity/GuestInvitation.php#L52
src/Mealz/MealBundle/Entity/GuestInvitation.php:52:16: NullableReturnStatement: The declared return type 'App\Mealz\MealBundle\Entity\EventParticipation' for App\Mealz\MealBundle\Entity\GuestInvitation::getEventParticipation is not nullable, but the function returns 'App\Mealz\MealBundle\Entity\EventParticipation|null' (see https://psalm.dev/139)
|
InvalidNullableReturnType:
src/Mealz/MealBundle/Helper/MealAdminHelper.php#L34
src/Mealz/MealBundle/Helper/MealAdminHelper.php:34:46: InvalidNullableReturnType: The declared return type 'App\Mealz\MealBundle\Entity\Event' for App\Mealz\MealBundle\Helper\MealAdminHelper::findEvent is not nullable, but 'App\Mealz\MealBundle\Entity\Event|null' contains null (see https://psalm.dev/144)
|
NullableReturnStatement:
src/Mealz/MealBundle/Helper/MealAdminHelper.php#L36
src/Mealz/MealBundle/Helper/MealAdminHelper.php:36:16: NullableReturnStatement: The declared return type 'App\Mealz\MealBundle\Entity\Event' for App\Mealz\MealBundle\Helper\MealAdminHelper::findEvent is not nullable, but the function returns 'App\Mealz\MealBundle\Entity\Event|null' (see https://psalm.dev/139)
|
UndefinedDocblockClass:
src/Mealz/MealBundle/Message/WeeklyMenuMessage.php#L94
src/Mealz/MealBundle/Message/WeeklyMenuMessage.php:94:38: UndefinedDocblockClass: Docblock-defined class, interface or enum named App\Mealz\MealBundle\Message\Event does not exist (see https://psalm.dev/200)
|
tests/unit/stores/dashboardStore.spec.ts > Test dashboardStore > should update the participation count of an event:
src/Resources/tests/unit/stores/dashboardStore.spec.ts#L96
TypeError: Cannot read properties of undefined (reading 'participations')
❯ tests/unit/stores/dashboardStore.spec.ts:96:53
|
tests/unit/stores/eventsStore.spec.ts > Test EventsStore > should filter the state after setting a filterString:
src/Resources/tests/unit/stores/eventsStore.spec.ts#L108
AssertionError: expected { id: 49, …(3) } to deeply equal { id: 48, …(3) }
- Expected
+ Received
Object {
- "id": 48,
+ "id": 49,
"public": true,
"slug": "alumni-afterwork",
"title": "Alumni Afterwork",
}
❯ tests/unit/stores/eventsStore.spec.ts:108:41
|
tests/unit/stores/eventsStore.spec.ts > Test EventsStore > should fetch all the users that participate in an event and return a list of their names:
src/Resources/tests/unit/stores/eventsStore.spec.ts#L158
AssertionError: expected [ { id: 47, …(3) }, { id: 49, …(3) } ] to have a length of 3 but got 2
- Expected
+ Received
- 3
+ 2
❯ tests/unit/stores/eventsStore.spec.ts:158:23
|
tests/unit/stores/weeksStore.spec.ts > Test weeksStore > should return the right menuday for a given weekId and dayId:
src/Resources/src/stores/weeksStore.ts#L260
TypeError: Cannot convert undefined or null to object
❯ getMenuDay src/stores/weeksStore.ts:260:47
❯ tests/unit/stores/weeksStore.spec.ts:98:25
|
Run frontend unit- and functional-tests
Process completed with exit code 1.
|
PHPMD:
src/Mealz/MealBundle/Controller/MealAdminController.php#L32
The class MealAdminController has an overall complexity of 60 which is very high. The configured complexity threshold is 55.
|
PHPMD:
src/Mealz/MealBundle/Controller/MealAdminController.php#L34
The method __construct has 10 parameters. Consider reducing the number of parameters to less than 10.
|
PHPMD:
src/Mealz/MealBundle/Controller/MealAdminController.php#L312
Avoid excessively long variable names like $eventExistsForDayAlready. Keep variable name length under 20.
|
PHPMD:
src/Mealz/MealBundle/Controller/MealAdminController.php#L315
Avoid excessively long variable names like $eventParticipationEntity. Keep variable name length under 20.
|
PHPMD:
src/Mealz/MealBundle/Helper/MealAdminHelper.php#L19
Avoid excessively long variable names like $eventParticipationRepository. Keep variable name length under 20.
|
PHPMD:
src/Mealz/MealBundle/Message/WeeklyMenuMessage.php#L72
The method getDishesByDay() has a Cyclomatic Complexity of 10. The configured cyclomatic complexity threshold is 10.
|
PHPMD:
src/Mealz/UserBundle/DataFixtures/ORM/LoadUsers.php#L124
Avoid unused parameters such as '$email'.
|