Platine\Framework\Demo\Action\User\EditAction::handle()

Complexity

Conditions	12
Paths	55

Size

Total Lines	130
Code Lines	82

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(ServerRequestInterface $request): ResponseInterface
    {
        $id = (int) $request->getAttribute('id');

        /** @var User $user */
        $user = $this->userRepository
                                    ->with('roles')
                                    ->find($id);
        if (!$user) {

$user is of type Platine\Framework\Auth\Entity\User, thus it always evaluated to true.

            $this->session->setFlash('error', 'Can not find the user');
            $this->logger->warning('Can not find user with id {id}', ['id' => $id]);

            return new RedirectResponse(
                $this->routeHelper->generateUrl('user_list')
            );
        }

        $statusList = [
            '0' => 'Deactive',
            '1' => 'Active',
        ];

        $roles = $this->roleRepository->all();

        $currentRolesId = [];
        foreach ($user->roles as $entity) {

The property roles does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __get, consider adding a @property annotation.

            $currentRolesId[] = $entity->id;
        }

        $entityToFormParam = (new UserParam())->fromEntity($user);
        $entityToFormParam->setRoles($currentRolesId);

        if ($request->getMethod() === 'GET') {
            return new TemplateResponse(
                $this->template,
                'user/edit',
                [
                    'param' => $entityToFormParam,
                    'status' => $statusList,
                    'roles' => $roles
                ]
            );
        }

        $param = new RequestData($request);
        $formParam = new UserParam($param->posts());
        $validator = new UserValidator($formParam);

        if (!$validator->validate()) {
            return new TemplateResponse(
                $this->template,
                'user/edit',
                [
                    'errors' => $validator->getErrors(),
                    'param' => $formParam,
                    'status' => $statusList,
                    'roles' => $roles
                ]
            );
        }

        $username = $param->post('username');
        $userExist = $this->userRepository->findBy(['username' => $username]);

        if ($userExist && $userExist->id != $id) {
            $this->session->setFlash('error', 'This user already exists');
            $this->logger->error('User with username {username} already exists', ['username' => $username]);
            return new TemplateResponse(
                $this->template,
                'user/edit',
                [
                   'param' => $formParam,
                   'status' => $statusList,
                   'roles' => $roles
                ]
            );
        }

        $password = $param->post('password');

        $user->username = $formParam->getUsername();

The property username does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->firstname = Str::ucfirst($formParam->getFirstname());

The property firstname does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->lastname = Str::upper($formParam->getLastname());

The property lastname does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->role = $formParam->getRole();

The property role does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->email = $formParam->getEmail();

The property email does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->status = (bool) $formParam->getStatus();

The property status does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        $user->updated_at = date('Y-m-d H::s');

The property updated_at does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        if (!empty($password)) {
            $passwordHash = $this->hash->hash($password);
            $user->password = $passwordHash;

The property password does not exist on Platine\Framework\Auth\Entity\User. Since you implemented __set, consider adding a @property annotation.

        }

        //Handle roles
        $rolesId = $param->post('roles', []);
        $rolesIdToDelete = array_diff($currentRolesId, $rolesId);
        if (!empty($rolesIdToDelete)) {
            $deletedRoles = $this->roleRepository->findAll(...$rolesIdToDelete);
            $user->removeRoles($deletedRoles);
        }
        $newRolesId = array_diff($rolesId, $currentRolesId);
        if (!empty($newRolesId)) {
            $newRoles = $this->roleRepository->findAll(...$newRolesId);
            $user->setRoles($newRoles);
        }
        /////////////////

        $result = $this->userRepository->save($user);

        if (!$result) {
            $this->session->setFlash('error', 'Error when saved the user');
            $this->logger->error('Error when saved the user');
            return new TemplateResponse(
                $this->template,
                'user/edit',
                [
                   'param' => $formParam,
                   'status' => $statusList,
                   'roles' => $roles
                ]
            );
        }

        $this->session->setFlash('success', 'User saved successfully');

        $returnUrl = $this->routeHelper->generateUrl('user_list');
        if ($param->get('from_detail')) {
            $returnUrl = $this->routeHelper->generateUrl('user_detail', ['id' => $id]);
        }
        return new RedirectResponse($returnUrl);
    }