Skip to content
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

Bug in Customer CSV Export column email #15875

Closed
elia-c opened this issue Oct 9, 2019 · 2 comments
Closed

Bug in Customer CSV Export column email #15875

elia-c opened this issue Oct 9, 2019 · 2 comments

Comments

@elia-c
Copy link

@elia-c elia-c commented Oct 9, 2019

Go to Customers > Customers
Click on Export

In the exported CSV, the column email is filled with the customer's firstname instead of email

Source:
src\PrestaShopBundle\Controller\Admin\Sell\Customer\CustomerController.php

line 690:
'email' => $record['firstname'],

@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

@khouloudbelguith khouloudbelguith commented Oct 9, 2019

Hi @elia-c,

Thanks for your report.
I manage to reproduce the issue with ps1.7.6.0 & ps1.7.6.1.
image
Here's the file exported

customer_2019-10-09_103451.csv.txt

It is Ok with PS1.7.5.2. It is a regression.
I’ll add this to the debug roadmap so that it’s fixed. If you have already fixed it on your end or if you think you can do it, please do send us a pull request!
Thanks!

@marionf marionf added this to Backlog in PrestaShop 1.7.6 via automation Oct 9, 2019
@colinegin colinegin added the S label Oct 9, 2019
@colinegin colinegin moved this from Backlog to To do in PrestaShop 1.7.6 Oct 9, 2019
@colinegin colinegin moved this from To do to In progress in PrestaShop 1.7.6 Oct 9, 2019
@khouloudbelguith

This comment has been minimized.

Copy link
Contributor

@khouloudbelguith khouloudbelguith commented Oct 9, 2019

Hello!

The Lastname is incorrect also.
image

It is OK ps1.7.5.2

@eternoendless eternoendless moved this from In progress to To be reviewed in PrestaShop 1.7.6 Oct 9, 2019
@matks matks moved this from To be reviewed to To be tested in PrestaShop 1.7.6 Oct 9, 2019
@khouloudbelguith khouloudbelguith added Fixed and removed To Do labels Oct 9, 2019
@khouloudbelguith khouloudbelguith added this to the 1.7.6.2 milestone Oct 9, 2019
@khouloudbelguith khouloudbelguith moved this from To be tested to To be merged in PrestaShop 1.7.6 Oct 9, 2019
@marionf marionf closed this Oct 10, 2019
PrestaShop 1.7.6 automation moved this from To be merged to Done Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.