upgrade method delete by ids to inject array skus

[sarron93]

Description (*)

when I call the deleteByIds method in the Magento \ Catalog \ Model \ CategoryLinkRepository class, I expect the sku array to be the second parameter to quickly remove products from the category, but I get the deletion only one at a time. Method name - does not match its functionality

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 are green)

[m2-assistant]

Hi @sarron93. 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 give me test instance - deploy test instance based on PR changes
• @magento give me 2.3-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

[VladimirZaets]

Hi @sarron93. Thanks for collaboration.
Please take a look at review comments.

[VladimirZaets]

@sarron93 your changes are also backward incompatible. In this case, you should create an interface with just one method and implement is in this class

[sarron93]

@sarron93 your changes are also backward incompatible. In this case, you should create an interface with just one method and implement is in this class

@VladimirZaets, Hello, all changes are ready

[VladimirZaets]

@sarron93 Hi, can you please rename your interface to "CategoryListDeleteBySkuInterface.php

[sarron93]

@sarron93 Hi, can you please rename your interface to "CategoryListDeleteBySkuInterface.php
@VladimirZaets Hi, ready :)

[VladimirZaets]

@sarron93 thank you

[magento-engcom-team]

Hi @VladimirZaets, thank you for the review.
ENGCOM-6379 has been created to process this Pull Request
✳️ @VladimirZaets, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[magento-engcom-team]

@sarron93 thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.

[ihor-sviziev]

Hi @sarron93,
Could you add preference for new interface to di.xml file?
All tests were passing, so that should be it.
Nazar tried to do that, but didn’t succeed with it.

[engcom-Golf]

@sarron93 thank you a lot 👍

[magento-engcom-team]

Hi @ihor-sviziev, thank you for the review.
ENGCOM-6829 has been created to process this Pull Request

[engcom-Alfa]

✔️ QA Passed

[m2-assistant]

Hi @sarron93, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.