Skip to content

Update ShipmentFactory.php #8727

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

cloudjibao
Copy link

@cloudjibao cloudjibao commented Mar 1, 2017

Changed the call to getId() to getProductId(). getId() returns the order ID instead of the product Id.

Changed the call to getId() to getProductId(). getId() returns the order ID instead of the product Id
@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Mar 1, 2017

CLA assistant check
All committers have signed the CLA.

@maghamed maghamed self-requested a review March 1, 2017 12:18
@maghamed maghamed self-assigned this Mar 1, 2017
@maghamed maghamed added this to the March 2017 milestone Mar 1, 2017
@cloudjibao cloudjibao closed this Mar 27, 2017
magento-devops-reposync-svc pushed a commit that referenced this pull request Feb 14, 2024
…ta3_30Jan23

Hammer platform health 247-beta3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants