View Issue Details

IDProjectCategoryView StatusLast Update
0003433OXID eShop (all versions)2. ----- eShop backend (admin) -----public2011-12-14 10:39
Reporterarvydas_vapsva 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.5.6 revision 40808 
Target VersionFixed in Version4.5.6 revision 40808 
Summary0003433: Inefficient code in oxNewsLetter::_assignProducts(), which slows down email sending
DescriptionIn oxNewsLetter::_assignProducts() there is a query:

$sSelect = "select $sArticleTable.* from oxorder left join oxorderarticles on oxorderarticles.oxorderid = oxorder.oxid";
$sSelect .= " left join $sArticleTable on oxorderarticles.oxartid = $sArticleTable.oxid";
$sSelect .= " where ".$oArticle->getSqlActiveSnippet();
$sSelect .= " and oxorder.oxuserid = '".$this->_oUser->getId()."' order by oxorder.oxorderdate desc "

and "limit 1" is missing here. Without it query may take long time on live shops with big amounts of data
TagsNo tags attached.
ThemeBoth
BrowserAll
PHP Versionany
Database Versionany

Activities

There are no notes attached to this issue.