Skip to content

Fixed order items list for order printing #11631

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

Merged
merged 1 commit into from
Nov 9, 2017

Conversation

rogyar
Copy link
Contributor

@rogyar rogyar commented Oct 22, 2017

Description

The block class defined in the XML layout for order items on the order print page is definitely not correct. As result, you cannot see the order items information upon the order printing. The Magento\Sales\Block\Order\PrintShipment is not responsible for showing order items. By changing the block class to Magento\Sales\Block\Order\Items we will have the order items upon the order printing

Fixed Issues (if relevant)

  1. Print order error on magento 2.1.8 #10530: Print order error on magento 2.1.8
  2. Null order in Magento\Sales\Block\Order\PrintShipment.php #9830: Null order in Magento\Sales\Block\Order\PrintShipment.php

Manual testing scenarios

  1. Place an order as a registered customer
  2. Go to My Account -> My Orders. Open the order and click on the "Print Order" link
  3. You should see all order's information for printing including the order items

@ricoliv
Copy link

ricoliv commented Oct 22, 2017

We had same problem #10530 on Magento 2.1.9.
We hardcoded fix above in
$OURMAG2HOME/vendor/magento/module-sales/view/frontend/layout/sales_order_print.xml

Now printing order works as expected.

Beeing unfamiliar with Magento2 Upgrades we hope above .xml file will be replaced inxluding official fix automatically by next System Upgrade via Composer (part of magento core components).

@fooman fooman self-assigned this Oct 25, 2017
@fooman fooman added this to the October 2017 milestone Oct 25, 2017
@fooman fooman added Release Line: 2.1 2.2.x Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release Component: Sales labels Oct 25, 2017
@okorshenko okorshenko modified the milestones: October 2017, November 2017 Nov 1, 2017
@magento-engcom-team magento-engcom-team added the Issue: Cannot Reproduce Cannot reproduce the issue on the latest `2.4-develop` branch label Nov 7, 2017
@fooman fooman added Progress: accept and removed Issue: Cannot Reproduce Cannot reproduce the issue on the latest `2.4-develop` branch labels Nov 9, 2017
@fooman
Copy link
Contributor

fooman commented Nov 9, 2017

thanks @rogyar for this pull request - I can confirm this fixes the issue of the missing items:

Before:
screen shot 2017-11-09 at 17 53 40
After:
screen shot 2017-11-09 at 17 53 50

Will add this to the queue of items to be merged.

@fooman
Copy link
Contributor

fooman commented Nov 9, 2017

@rogyar so far this PR will fix this issue in 2.1 branch. Can I ask you to also take a look at creating a separate PR for the 2.2 branch.

One further thing I noticed is that this seems to affect guest orders as well - see sales_guest_print.xml. If you could confirm that be great.

@okorshenko okorshenko merged commit 6b5329b into magento:2.1-develop Nov 9, 2017
@okorshenko
Copy link
Contributor

We need a PR for 2.2-develop branch

@magento-engcom-team magento-engcom-team added the Fixed in 2.1.x The issue has been fixed in 2.1 release line label Nov 9, 2017
@rut4
Copy link

rut4 commented Nov 23, 2017

Also the same action for a guest should be fixed in vendor/magento/module-sales/view/frontend/layout/sales_guest_print.xml:20.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Sales Fixed in 2.1.x The issue has been fixed in 2.1 release line Progress: accept Release Line: 2.1 Reproduced on 2.1.x The issue has been reproduced on latest 2.1 release Reproduced on 2.2.x The issue has been reproduced on latest 2.2 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants