skobkin/point-tools
1
1
Fork 0
Code Issues 4 Pull requests 1 Releases Activity

Telegram bot /link case sensitive user search fix.

Browse source
This commit is contained in:
Alexey Skobkin 2017-01-06 00:00:37 +03:00
parent 4ebc547ab2
commit d9082a3a71
2 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/Skobkin/Bundle/PointToolsBundle/Repository/UserRepository.php
View file

@ -15,7 +15,7 @@ class UserRepository extends EntityRepository
* @return User|null * @return User|null
* @throws \Doctrine\ORM\NonUniqueResultException * @throws \Doctrine\ORM\NonUniqueResultException
*/ */
public function findUserByLogin($login) public function findUserByLogin(string $login)
{ {
$qb = $this->createQueryBuilder('u'); $qb = $this->createQueryBuilder('u');
@ -36,7 +36,7 @@ class UserRepository extends EntityRepository
* *
* @return User[] * @return User[]
*/ */
public function findUsersLikeLogin($login, $limit = 10) public function findUsersLikeLogin(string $login, int $limit = 10)
{ {
if (empty($login)) { if (empty($login)) {
return []; return [];
@ -68,7 +68,7 @@ class UserRepository extends EntityRepository
* @param integer $id * @param integer $id
* @return User[] * @return User[]
*/ */
public function findUserSubscribersById($id) public function findUserSubscribersById(int $id)
{ {
if (!is_int($id)) { if (!is_int($id)) {
throw new \InvalidArgumentException('$id must be an integer'); throw new \InvalidArgumentException('$id must be an integer');
@ -93,7 +93,7 @@ class UserRepository extends EntityRepository
* *
* @return TopUserDTO[] * @return TopUserDTO[]
*/ */
public function getTopUsers($limit = 30) public function getTopUsers(int $limit = 30)
{ {
if (!is_int($limit)) { if (!is_int($limit)) {
throw new \InvalidArgumentException('$limit must be an integer'); throw new \InvalidArgumentException('$limit must be an integer');

2
src/Skobkin/Bundle/PointToolsBundle/Service/Telegram/PrivateMessageProcessor.php
View file

@ -209,7 +209,7 @@ class PrivateMessageProcessor
{ {
if ($this->userApi->isAuthDataValid($login, $password)) { if ($this->userApi->isAuthDataValid($login, $password)) {
/** @var User $user */ /** @var User $user */
if (null === $user = $this->userRepo->findOneBy(['login' => $login])) { if (null === $user = $this->userRepo->findUserByLogin($login)) {
throw new CommandProcessingException('User not found in Point Tools database. Please try again later.'); throw new CommandProcessingException('User not found in Point Tools database. Please try again later.');
} }