[#18269] If import product has missing categories, make a lookup for existing categories to keep the correct url rewrites

[ulftietze]

…existing categories to keep the correct url rewrites

Description

Fixed Issues (if relevant)

1. Product Import -> Missing categories entry in importfile removes url rewrites #18269: Product Import -> Missing categories entry in importfile removes url rewrites

Manual testing scenarios

1. Updated Unit Test

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[magento-cicd2]

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

---

Ulf Tietze seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.

[magento-engcom-team]

Hi @ulftietze. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento-engcom-team give me test instance - deploy test instance based on PR changes
• @magento-engcom-team give me $VERSION instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Assistant documentation

[slavvka]

Hey @ulftietze. Are you sure that this is a bug? Product may not have categories at all so the current behavior may make sense if a merchant want them to be cleared.

[ulftietze]

Hey @slavvka,

i'm sure, that this is a bug :)
We have to discuss about, because Magento's Importer as far as i know, is declared as delta update importer. So if you update something, i expect that nothing else is deleted here.
If the product are removed from categories, i think that this would be a correct way to go, but they're keeped into these categories.
This is probably more an error into the url-rewrites and not in the importer, because everytime rewrites for a product are generated, the old one's are deleted. And so it is with the product_category url rewrites.

What do you think about?

[slavvka]

Hey @ulftietze. Yes, you are right. Seems that your commit was done with non-github email. Could you please add that email to your GitHub profile or force-push your branch with a new commit with your GitHub email?
Could you also fix the static tests before we can accept your PR? Thank you!

[sidolov]

Hi @ulftietze , I am closing this PR now due to inactivity.
Please reopen and update if you wish to continue.
Thank you for the collaboration!