Skip to content

Commit

Permalink
[TASK] Address PHPStan remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
ohader committed Jan 24, 2024
1 parent cf926c5 commit de54fa4
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/Controller/Admin/RequirementCrudController.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ public static function getEntityFqcn(): string
return Requirement::class;
}

public function createEntity(string $entityFqcn)
public function createEntity(string $entityFqcn): Requirement
{
return Requirement::create();
}
Expand Down
2 changes: 1 addition & 1 deletion src/Entity/Requirement.php
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ public function getId(): ?int
return $this->id;
}

public function setVersion(MajorVersion $version): void
public function setVersion(?MajorVersion $version): void
{
$this->version = $version;
}
Expand Down
15 changes: 12 additions & 3 deletions src/EventListener/RequirementListener.php
Original file line number Diff line number Diff line change
Expand Up @@ -37,16 +37,25 @@ public function __construct(

public function postUpdate(Requirement $requirement, PostUpdateEventArgs $eventArgs): void
{
$this->cacheService->purgeMajorVersion((string)$requirement->getVersion()->getVersion());
$majorVersion = $requirement->getVersion();
if ($majorVersion !== null) {
$this->cacheService->purgeMajorVersion((string)$majorVersion->getVersion());
}
}

public function postRemove(Requirement $requirement, PostRemoveEventArgs $eventArgs): void
{
$this->cacheService->purgeMajorVersion((string)$requirement->getVersion()->getVersion());
$majorVersion = $requirement->getVersion();
if ($majorVersion !== null) {
$this->cacheService->purgeMajorVersion((string)$majorVersion->getVersion());
}
}

public function postPersist(Requirement $requirement, PostPersistEventArgs $eventArgs): void
{
$this->cacheService->purgeMajorVersion((string)$requirement->getVersion()->getVersion());
$majorVersion = $requirement->getVersion();
if ($majorVersion !== null) {
$this->cacheService->purgeMajorVersion((string)$majorVersion->getVersion());
}
}
}

0 comments on commit de54fa4

Please sign in to comment.