magento / magento/inventory

Resolve Dependency of InventoryShipping on InventoryBundleProduct modules

Open
#1,259 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Modularity Progress: ready for dev
Dominant language
PHP
Stars
357
Forks
262
PR merge metrics
No merged PRs in 30d

Description

Resolve Dependency of InventoryShipping on InventoryBundleProduct modules

There is no such dependency in composer.json

But there are tests and fixtures which should be moved from InventoryShipping

Also, there is dependency in \Magento\InventoryShipping\Model\GetItemsToDeduct class

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting composer.json and the InventoryShipping tests and fixtures mentioned in the issue, then review Magento\InventoryShipping\Model\GetItemsToDeduct. Trace references to InventoryBundleProduct and identify which tests and fixtures belong elsewhere. Done means the dependency is removed from InventoryShipping while the affected tests and fixtures continue to work.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.