View Issue Details

IDProjectCategoryView StatusLast Update
0006366OXID eShop (all versions)4.01. Database handlingpublic2017-06-29 11:32
Reportersmxsm 
PrioritylowSeveritymajorReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.9.5 / 5.2.5 
Target VersionFixed in Version 
Summary0006366: oxarticle::unassignFromShop() always tries to clean complete oxfield2shop table
Descriptionoxarticle::unassignFromShop() always calls
foreach ($aShopIds as $sShopId) {
    $oField2Shop->cleanMultishopFields($sShopId);
}
without specifying a product id, so for *every* article unassignment all oxarticles and oxfield2shop are joined and searched for "orphans".
This is very, very, very ineffective with lots of subshops, articles and oxfield2shop values.
It should really be called with the article id, e.g.
foreach ($aShopIds as $sShopId) {
    $oField2Shop->cleanMultishopFields($sShopId, $this->getId());
}
TagsEE
ThemeNot defined
BrowserNot defined
PHP Version5.4
MySQL Version5.5

Activities

There are no notes attached to this issue.