diff --git a/lib/Segments/DynamicSegments/Filters/EmailAction.php b/lib/Segments/DynamicSegments/Filters/EmailAction.php index 7f4dd7303a..c229491ff7 100644 --- a/lib/Segments/DynamicSegments/Filters/EmailAction.php +++ b/lib/Segments/DynamicSegments/Filters/EmailAction.php @@ -103,7 +103,7 @@ class EmailAction implements Filter { )->setParameter('newsletter' . $parameterSuffix, $newsletterId); } if ($action === EmailAction::ACTION_OPENED) { - $queryBuilder->andWhere('(stats.user_agent_type = :userAgentType) OR (stats.user_agent_type IS NULL)') + $queryBuilder->andWhere('stats.user_agent_type = :userAgentType') ->setParameter('userAgentType', UserAgentEntity::USER_AGENT_TYPE_HUMAN); } if ($action === EmailAction::ACTION_MACHINE_OPENED) { diff --git a/lib/Segments/DynamicSegments/Filters/EmailOpensAbsoluteCountAction.php b/lib/Segments/DynamicSegments/Filters/EmailOpensAbsoluteCountAction.php index 3984b5f584..82267e2f7e 100644 --- a/lib/Segments/DynamicSegments/Filters/EmailOpensAbsoluteCountAction.php +++ b/lib/Segments/DynamicSegments/Filters/EmailOpensAbsoluteCountAction.php @@ -45,10 +45,10 @@ class EmailOpensAbsoluteCountAction implements Filter { } $queryBuilder->setParameter('opens' . $parameterSuffix, $filterData->getParam('opens')); if ($action === EmailOpensAbsoluteCountAction::TYPE) { - $queryBuilder->andWhere('(opens.user_agent_type = :userAgentType) OR (opens.user_agent_type IS NULL)') + $queryBuilder->andWhere('opens.user_agent_type = :userAgentType') ->setParameter('userAgentType', UserAgentEntity::USER_AGENT_TYPE_HUMAN); } else { - $queryBuilder->andWhere('(opens.user_agent_type = :userAgentType)') + $queryBuilder->andWhere('opens.user_agent_type = :userAgentType') ->setParameter('userAgentType', UserAgentEntity::USER_AGENT_TYPE_MACHINE); } return $queryBuilder;