From 5cf4013a997deccfaf2290b3c7f86d00ad8c21ce Mon Sep 17 00:00:00 2001 From: Julie Lenaerts Date: Fri, 25 Feb 2022 13:12:25 +0100 Subject: [PATCH] cs fixes --- .../Controller/AccompanyingCourseApiController.php | 1 - .../Security/Authorization/AccompanyingPeriodVoter.php | 5 ++--- .../AccompanyingPeriod/AccompanyingPeriodValidity.php | 1 - 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/Bundle/ChillPersonBundle/Controller/AccompanyingCourseApiController.php b/src/Bundle/ChillPersonBundle/Controller/AccompanyingCourseApiController.php index 6bbdb64e2..f1306640b 100644 --- a/src/Bundle/ChillPersonBundle/Controller/AccompanyingCourseApiController.php +++ b/src/Bundle/ChillPersonBundle/Controller/AccompanyingCourseApiController.php @@ -344,7 +344,6 @@ final class AccompanyingCourseApiController extends ApiController */ public function toggleIntensityApi(AccompanyingPeriod $accompanyingCourse, Request $request) { - if ($request->getMethod() === 'POST') { $this->denyAccessUnlessGranted(AccompanyingPeriodVoter::TOGGLE_INTENSITY, $accompanyingCourse); diff --git a/src/Bundle/ChillPersonBundle/Security/Authorization/AccompanyingPeriodVoter.php b/src/Bundle/ChillPersonBundle/Security/Authorization/AccompanyingPeriodVoter.php index 08a116225..bf6e6c287 100644 --- a/src/Bundle/ChillPersonBundle/Security/Authorization/AccompanyingPeriodVoter.php +++ b/src/Bundle/ChillPersonBundle/Security/Authorization/AccompanyingPeriodVoter.php @@ -131,17 +131,16 @@ class AccompanyingPeriodVoter extends AbstractChillVoter implements ProvideRoleH } if (self::TOGGLE_CONFIDENTIAL === $attribute) { - if (null != $subject->getUser() && ($subject->getUser() === $token->getUser())) { + if (null !== $subject->getUser() && ($subject->getUser() === $token->getUser())) { return true; } return false; - // return $this->voterHelper->voteOnAttribute(self::TOGGLE_CONFIDENTIAL_ALL, $subject, $token); } if (self::TOGGLE_INTENSITY === $attribute) { - if (null != $subject->getUser() && ($subject->getUser() === $token->getUser())) { + if (null !== $subject->getUser() && ($subject->getUser() === $token->getUser())) { return true; } diff --git a/src/Bundle/ChillPersonBundle/Validator/Constraints/AccompanyingPeriod/AccompanyingPeriodValidity.php b/src/Bundle/ChillPersonBundle/Validator/Constraints/AccompanyingPeriod/AccompanyingPeriodValidity.php index a6eb92935..c4bd30ed3 100644 --- a/src/Bundle/ChillPersonBundle/Validator/Constraints/AccompanyingPeriod/AccompanyingPeriodValidity.php +++ b/src/Bundle/ChillPersonBundle/Validator/Constraints/AccompanyingPeriod/AccompanyingPeriodValidity.php @@ -18,7 +18,6 @@ use Symfony\Component\Validator\Constraint; */ class AccompanyingPeriodValidity extends Constraint { - public $messageSocialIssueCannotBeDeleted = 'The social %name% issue cannot be deleted because it is associated with an activity or an action'; public function getTargets()