View Issue Details

IDProjectCategoryView StatusLast Update
0006304OXID eShop (all versions)4.09. SEO, SEO URLpublic2018-08-13 07:29
ReporterAdrian.Kirchner 
PrioritynormalSeveritymajorReproducibilityalways
Status acknowledgedResolutionopen 
Product Version4.9.6 / 5.2.6 
Target VersionFixed in Version 
Summary0006304: oxseohistory is left untouched when deleting entities like oxArticle, oxCategory, oxContent, oxManufacturer, oxVendor
DescriptionThe deletion of the entities oxArticle, oxCategory, oxContent, oxManufacturer and oxVendor triggers a call to the corresponding onDelete* method in the associated oxSeoEncoder* class. For all listed entity types the related tuples in oxseo and oxobject2seodata are deleted but the oxseohistory table is left untouched.
Steps To Reproduce- https://github.com/OXID-eSales/oxideshop_ce/blob/5dfebddcb24ed4c444a735d48258bb381b7c4e0d/source/application/models/oxseoencoderarticle.php#L642
- https://github.com/OXID-eSales/oxideshop_ce/blob/5dfebddcb24ed4c444a735d48258bb381b7c4e0d/source/application/models/oxseoencodercategory.php#L239
- https://github.com/OXID-eSales/oxideshop_ce/blob/5dfebddcb24ed4c444a735d48258bb381b7c4e0d/source/application/models/oxseoencodercontent.php#L109
- https://github.com/OXID-eSales/oxideshop_ce/blob/5dfebddcb24ed4c444a735d48258bb381b7c4e0d/source/application/models/oxseoencodermanufacturer.php#L140
- https://github.com/OXID-eSales/oxideshop_ce/blob/5dfebddcb24ed4c444a735d48258bb381b7c4e0d/source/application/models/oxseoencodervendor.php#L140
Additional InformationRecommlists and tags may need further treatment.
A pull request will follow
TagsSolution Provided
ThemeNot defined
BrowserNot defined
PHP VersionNot defined
MySQL VersionNot defined

Relationships

related to 0006880 acknowledgedbenjamin.joerger oxseo and oxseohistory entries will not deleted by deletetion of oxvendor and oxmanufacturer 

Activities

Adrian.Kirchner

2016-01-02 23:57

reporter   ~0011400

PR: https://github.com/OXID-eSales/oxideshop_ce/pull/310

Adrian.Kirchner

2018-08-13 02:26

reporter   ~0012570

It seems like the referenced pr got already merged so this bug can be closed