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

Customer with unique attribute can't be saved #7844 #9712

Merged
merged 2 commits into from
Jun 1, 2017

Conversation

storbahn
Copy link
Contributor

@storbahn storbahn commented May 20, 2017

Description

Added the missing entity id of customer to validation process of customer (unique attribute) before saving it.

Fixed Issues (if relevant)

  1. Customer with unique attribute can't be saved #7844

Manual testing scenarios

See "Steps to reproduce" of #7844

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented May 20, 2017

CLA assistant check
All committers have signed the CLA.

@ishakhsuvarov ishakhsuvarov self-assigned this May 20, 2017
@ishakhsuvarov ishakhsuvarov added this to the May 2017 milestone May 20, 2017
@okorshenko okorshenko modified the milestones: May 2017, June 2017 Jun 1, 2017
@magento-team magento-team merged commit 9e25d8c into magento:develop Jun 1, 2017
magento-team pushed a commit that referenced this pull request Jun 1, 2017
[EngCom] Public Pull Requests
 - MAGETWO-69573: Adding logo in media folder #9797
 - MAGETWO-69555: Allow for referenceBlock to include template argument #9772
 - MAGETWO-69540: Fix for #5897: getIdentities relies on uninitialized collection #9777
 - MAGETWO-69533: [BUGFIX][6244] Fix Issue with code label display in cart checkout. #9721
 - MAGETWO-69499: Update select.js #9475
 - MAGETWO-69451: Replace Zend_Json in the configurable product block test #9753
 - MAGETWO-69373: Customer with unique attribute can't be saved #7844 #9712
 - MAGETWO-69369: Replace the direct usage of Zend_Json with a call to the Json Help class #9344
 - MAGETWO-69085: Do not hardcode product link types #9600
 - MAGETWO-69554: Patch to allow multiple filter_url_params to function #9723
@magento-team
Copy link
Contributor

@storbahn thank you for your contribution. Your Pull Request has been successfully merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants