View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006304 | OXID eShop (all versions) | 4.09. SEO, SEO URL | public | 2016-01-02 23:56 | 2022-03-15 12:14 |
| Reporter | Adrian.Kirchner | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.9.6 / 5.2.6 | ||||
| Fixed in Version | 6.0.0 | ||||
| Summary | 0006304: oxseohistory is left untouched when deleting entities like oxArticle, oxCategory, oxContent, oxManufacturer, oxVendor | ||||
| Description | The 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 Information | Recommlists and tags may need further treatment. A pull request will follow | ||||
| Tags | Solution Provided | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
| related to | 0006880 | resolved | MarcelOxid | oxseo and oxseohistory entries will not deleted by deletetion of oxvendor and oxmanufacturer |
|
|
PR: https://github.com/OXID-eSales/oxideshop_ce/pull/310 |
|
|
It seems like the referenced pr got already merged so this bug can be closed |
|
|
Hello guys. Yes, looks like this one is already fixed long ago with a merged pull request. Thanks! |
