Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Use better alias for queries

Browse Source 
[MAILPOET-2439]
This commit is contained in:
Pavel Dohnal
2019-10-30 12:55:05 +01:00
committed by Jack Kitterhing
parent 7a0eb82aad
commit 37a97d966f
2 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Cron/Workers/StatsNotifications/NewsletterLinkRepository.php
View File

@@ -16,12 +16,12 @@ class NewsletterLinkRepository extends Repository {
*/ */
public function findTopLinkForNewsletter($newsletter_id) { public function findTopLinkForNewsletter($newsletter_id) {
return $this->doctrine_repository return $this->doctrine_repository
->createQueryBuilder('nlr') ->createQueryBuilder('nl')
->join('nlr.clicks', 'c') ->join('nl.clicks', 'c')
->addSelect('COUNT(c.id) AS HIDDEN counter') ->addSelect('COUNT(c.id) AS HIDDEN counter')
->where('nlr.newsletter_id = :newsletterId') ->where('nl.newsletter_id = :newsletterId')
->setParameter('newsletterId', $newsletter_id) ->setParameter('newsletterId', $newsletter_id)
->groupBy('nlr.id') ->groupBy('nl.id')
->orderBy('counter', 'desc') ->orderBy('counter', 'desc')
->setMaxResults(1) ->setMaxResults(1)
->getQuery() ->getQuery()

10
lib/Cron/Workers/StatsNotifications/StatsNotificationsRepository.php
View File

@@ -19,8 +19,8 @@ class StatsNotificationsRepository extends Repository {
*/ */
public function findByNewsletterId($newsletter_id) { public function findByNewsletterId($newsletter_id) {
return $this->doctrine_repository return $this->doctrine_repository
->createQueryBuilder('stn') ->createQueryBuilder('sn')
->andWhere('stn.newsletter = :newsletterId') ->andWhere('sn.newsletter = :newsletterId')
->setParameter('newsletterId', $newsletter_id) ->setParameter('newsletterId', $newsletter_id)
->setMaxResults(1) ->setMaxResults(1)
->getQuery() ->getQuery()
@@ -34,9 +34,9 @@ class StatsNotificationsRepository extends Repository {
public function findScheduled($limit = null) { public function findScheduled($limit = null) {
$date = new Carbon(); $date = new Carbon();
$query = $this->doctrine_repository $query = $this->doctrine_repository
->createQueryBuilder('stn') ->createQueryBuilder('sn')
->join('stn.task', 'tasks') ->join('sn.task', 'tasks')
->join('stn.newsletter', 'n') ->join('sn.newsletter', 'n')
->addSelect('tasks') ->addSelect('tasks')
->addSelect('n') ->addSelect('n')
->addOrderBy('tasks.priority') ->addOrderBy('tasks.priority')