magento / magento2

Prior to making any Submission(s), you must sign an Adobe Contributor License Agreement, available here at: https://opensource.adobe.com/cla.html. All Submissions you make to Adobe Inc. and its affiliates, assigns and subsidiaries (collectively “Adobe”) are subject to the terms of the Adobe Contributor License Agreement.
http://www.magento.com
Open Software License 3.0
11.56k stars 9.32k forks source link

Magento2 Report export empty row #17913

Closed penglongyun closed 6 years ago

penglongyun commented 6 years ago

Preconditions

1.Admimhtml reports module,when 'filter' params contains "report_period",and values 'day',must value "day","month" or "year" not error. this param must in the last.

  1. then when Refresh,the url will show param 'filter' and value ,the value is urlencode. ok, the value is end by "==", but when export, the url will be encode again, so the report action will not correctly resolve the param "report_period",ERROR SHOW.

3.the template file : Magento_Backend\templates\widget\grid\export.phtml .I monified bellow: image

image

diffrent between with:

image

Steps to reproduce

1.Magento2 Version:2.2.2 2.click adminhtml menu [REPORTS]->[PRODUCTS]->[Ordered] 2.input special date range and show by.click [Refresh], will show correct data row,but when click[Export],empty row in CSV file.

Expected result

  1. [Screenshots, logs or description] 2.CSV file show the same rows with Grid.

Actual result

  1. [Screenshots, logs or description]
  2. CSV file empty row.
magento-engcom-team commented 6 years ago

Hi @penglongyun. Thank you for your report. To help us process this issue please make sure that you provided the following information:

Please make sure that the issue is reproducible on the vanilla Magento instance following Steps to reproduce. To deploy vanilla Magento instance on our environment, please, add a comment to the issue:

@magento-engcom-team give me {$VERSION} instance

where {$VERSION} is version tags (starting from 2.2.0+) or develop branches (2.2-develop +). For more details, please, review the Magento Contributor Assistant documentation.

@penglongyun do you confirm that you was able to reproduce the issue on vanilla Magento instance following steps to reproduce?

ghost commented 6 years ago

Hi @penglongyun thank you for you report, the fix for this issue already available in 2.2develop branch