Merge branch 'master' of github.com:chamilo/chamilo-lms

pull/5235/head
Yannick Warnier 2 years ago
commit e2187b5ca4
  1. 2
      src/CoreBundle/Controller/CourseController.php
  2. 4
      src/CoreBundle/Controller/SecurityController.php
  3. 2
      src/CoreBundle/Entity/ExtraField.php
  4. 2
      src/CoreBundle/Entity/User.php
  5. 6
      src/CoreBundle/Repository/Node/CourseRepository.php

@ -7,7 +7,6 @@ declare(strict_types=1);
namespace Chamilo\CoreBundle\Controller; namespace Chamilo\CoreBundle\Controller;
use Chamilo\CoreBundle\Entity\Course; use Chamilo\CoreBundle\Entity\Course;
use Chamilo\CoreBundle\Entity\CourseCategory;
use Chamilo\CoreBundle\Entity\CourseRelUser; use Chamilo\CoreBundle\Entity\CourseRelUser;
use Chamilo\CoreBundle\Entity\ExtraField; use Chamilo\CoreBundle\Entity\ExtraField;
use Chamilo\CoreBundle\Entity\Session; use Chamilo\CoreBundle\Entity\Session;
@ -711,7 +710,6 @@ class CourseController extends ToolBaseController
AccessUrlHelper $accessUrlHelper, AccessUrlHelper $accessUrlHelper,
CourseRepository $courseRepository CourseRepository $courseRepository
): JsonResponse { ): JsonResponse {
$searchTerm = $request->query->get('search', ''); $searchTerm = $request->query->get('search', '');
$accessUrl = $accessUrlHelper->getCurrent(); $accessUrl = $accessUrlHelper->getCurrent();

@ -49,8 +49,8 @@ class SecurityController extends AbstractController
/** @var User $user */ /** @var User $user */
$user = $this->getUser(); $user = $this->getUser();
if ($user->getActive() !== 1) { if (1 !== $user->getActive()) {
if ($user->getActive() === 0) { if (0 === $user->getActive()) {
$message = $translator->trans('Account not activated.'); $message = $translator->trans('Account not activated.');
} else { } else {
$message = $translator->trans('Invalid credentials. Please try again or contact support if you continue to experience issues.'); $message = $translator->trans('Invalid credentials. Please try again or contact support if you continue to experience issues.');

@ -127,7 +127,7 @@ class ExtraField
protected DateTime $createdAt; protected DateTime $createdAt;
#[Groups(['extra_field:read'])] #[Groups(['extra_field:read'])]
#[ORM\Column(name: "auto_remove", type: "boolean", options: ["default" => false])] #[ORM\Column(name: 'auto_remove', type: 'boolean', options: ['default' => false])]
protected bool $autoRemove = false; protected bool $autoRemove = false;
public function __construct() public function __construct()

@ -943,7 +943,7 @@ class User implements UserInterface, EquatableInterface, ResourceInterface, Reso
*/ */
public function getIsActive(): bool public function getIsActive(): bool
{ {
return $this->active === 1; return 1 === $this->active;
} }
public function isEnabled(): bool public function isEnabled(): bool

@ -104,14 +104,16 @@ class CourseRepository extends ResourceRepository
'user' => $user, 'user' => $user,
'url' => $url, 'url' => $url,
'status' => $status, 'status' => $status,
]); ])
;
if (!empty($keyword)) { if (!empty($keyword)) {
$qb->andWhere($qb->expr()->orX( $qb->andWhere($qb->expr()->orX(
$qb->expr()->like('c.title', ':keyword'), $qb->expr()->like('c.title', ':keyword'),
$qb->expr()->like('c.code', ':keyword') $qb->expr()->like('c.code', ':keyword')
)) ))
->setParameter('keyword', '%'.$keyword.'%'); ->setParameter('keyword', '%'.$keyword.'%')
;
} }
$query = $qb->getQuery(); $query = $qb->getQuery();

Loading…
Cancel
Save