* Returns a random entry, filtering by status.
*
* @param $userId
- * @param string $status can be unread, archive or starred
+ * @param string $type can be unread, archive, starred, etc
*
* @throws \Doctrine\ORM\NoResultException
* @throws \Doctrine\ORM\NonUniqueResultException
*
+ * @see https://github.com/doctrine/doctrine2/issues/5479#issuecomment-403862934
+ *
* @return Entry
*/
- public function getRandomEntry($userId, $status = '')
+ public function getRandomEntry($userId, $type = '')
{
- $max = $this->getEntityManager()
- ->createQuery('SELECT MAX(e.id) FROM Wallabag\CoreBundle\Entity\Entry e WHERE e.user = :userId')
- ->setParameter('userId', $userId)
- ->getSingleScalarResult();
-
- $qb = $this->createQueryBuilder('e')
- ->where('e.user = :user_id')->setParameter('user_id', $userId);
-
- if ('unread' === $status) {
- $qb->andWhere('e.isArchived = false');
+ $qb = $this->getQueryBuilderByUser($userId)
+ ->select('MIN(e.id)', 'MAX(e.id)');
+
+ switch ($type) {
+ case 'unread':
+ $qb->andWhere('e.isArchived = false');
+ break;
+ case 'archive':
+ $qb->andWhere('e.isArchived = true');
+ break;
+ case 'starred':
+ $qb->andWhere('e.isStarred = true');
+ break;
+ case 'untagged':
+ $qb->leftJoin('e.tags', 't');
+ $qb->andWhere('t.id is null');
+ break;
}
- if ('archive' === $status) {
- $qb->andWhere('e.isArchived = true');
- }
-
- if ('starred' === $status) {
- $qb->andWhere('e.isStarred = true');
- }
+ $idLimits = $qb
+ ->getQuery()
+ ->getOneOrNullResult();
+ $randomPossibleIds = rand($idLimits[1], $idLimits[2]);
- return $qb->andWhere('e.id >= :rand')
- ->setParameter('rand', rand(0, $max))
+ return $qb
+ ->select('e')
+ ->andWhere('e.id >= :random_id')
+ ->setParameter('random_id', $randomPossibleIds)
->setMaxResults(1)
->getQuery()
->getSingleResult();