View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004224 | OXID eShop (all versions) | 3.1. Design, GUI, UX | public | 2012-07-06 15:13 | 2012-11-15 17:11 |
| Reporter | design3000 | Assigned To | |||
| Priority | low | Severity | tweak | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.6.2 revision 46646 | ||||
| Fixed in Version | 4.8.0_5.1.0_beta1 | ||||
| Summary | 0004224: Changing E-Mail Address fails without error message if user enters wrong password | ||||
| Description | When a user wants to change his e-mail-address in "my account", the password field appears. If the user then enters a wrong password, the form is submitted but the e-mail-address is not changed. There is no feedback that somathing went wrong until the user clicks on "change" again. When he does so, a wrong error message appears ('Passwords don't match!'). IMHO there is no need to ask the users password here, because he is already logged in. Maybe a confirmation e-mail should be sent to the new e-mail-address. | ||||
| Steps To Reproduce | 1.) In demo shop, create a new user 2.) In "My Account > Billing and Shipping Settings > Billing Address" click "change" 3.) Change E-Mail-Address > The password field appears 4.) Type in wrong password 5.) Click "save" | ||||
| Tags | No tags attached. | ||||
| Theme | Azure | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
Also the label should read "Please enter your current password to confirm email address change" or something like this, because the label "Password" suggests that you could enter a new Password, just like you can enter changed data in all the other fields. Imho "Billing Address" is the wrong place for e-mail address change, a better place could be "Account Settings", where customer could choose email change, password change and newsletter settings. |
|
|
Thank you for your suggestions on changing the interface, we will consider it. As for the password, it was checked, but the form with error was not showing up after submit. This will be fixed in the 5.1 patch. If you need to fix it in earlier versions, just update tpl/form/user.tpl 19 line to: <ul class="form clear" [{if empty($aErrors)}]style="display: none;"[{/if}] id="addressForm"> [{ include file="form/fieldset/user_email.tpl" }] [{ include file="form/fieldset/user_billing.tpl" noFormSubmit=true }] |
