* @param string $url
* @param int $userId
*
- * @return Entry|bool
+ * @return Entry|false
*/
public function findByUrlAndUserId($url, $userId)
{
return $this->findByHashedUrlAndUserId(
UrlHasher::hashUrl($url),
- $userId);
+ $userId
+ );
}
/**
* @param string $hashedUrl Url hashed using sha1
* @param int $userId
*
- * @return Entry|bool
+ * @return Entry|false
*/
public function findByHashedUrlAndUserId($hashedUrl, $userId)
{
+ // try first using hashed_url (to use the database index)
$res = $this->createQueryBuilder('e')
->where('e.hashedUrl = :hashed_url')->setParameter('hashed_url', $hashedUrl)
->andWhere('e.user = :user_id')->setParameter('user_id', $userId)
return current($res);
}
+ // then try using hashed_given_url (to use the database index)
+ $res = $this->createQueryBuilder('e')
+ ->where('e.hashedGivenUrl = :hashed_given_url')->setParameter('hashed_given_url', $hashedUrl)
+ ->andWhere('e.user = :user_id')->setParameter('user_id', $userId)
+ ->getQuery()
+ ->getResult();
+
+ if (\count($res)) {
+ return current($res);
+ }
+
return false;
}
return $this->find($randomId);
}
- /**
- * Inject a UrlHasher.
- *
- * @param UrlHasher $hasher
- */
- public function setUrlHasher(UrlHasher $hasher)
- {
- $this->urlHasher = $hasher;
- }
-
- /**
- * Get the UrlHasher, or create a default one if not injected.
- *
- * XXX: the default uses the default hash algorithm
- *
- * @return UrlHasher
- */
- protected function getUrlHasher()
- {
- if (!isset($this->urlHasher)) {
- $this->setUrlHasher(new UrlHasher());
- }
-
- return $this->urlHasher;
- }
-
/**
* Return a query builder to be used by other getBuilderFor* method.
*