View Issue Details

IDProjectCategoryView StatusLast Update
0003170OXID eShop (all versions)3.1. Design, GUI, UXpublic2011-08-24 10:44
Reporterakoeplin Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Product Version4.5.1 revision 38045 
Fixed in Version4.6.0_beta2 
Summary0003170: Password is not validated when creating new user account during checkout
DescriptionWhen you create a new user account during checkout, the password(s) for the account are not checked for matching values and an error is not shown if they do not match.

Please note that this only affects the client-side JavaScript validation, it works on the server when posting the form back (which generates the correct error message).
It also only affects the validation for matching passwords, the length check works OK.
Steps To Reproduce1. Go to a 4.5.1 shop (e.g. http://demoshop.oxid-esales.com/community-edition/)
2. Add an article to the basket
3. Proceed to checkout
4. Select "Open personal customer account" in step 2 of the checkout process
5. Try to enter two different (long enough) values in the "password" and "confirm password" boxes

Current result: There's no error message until the user submits the form

Expected result: An error message should tell the user that the passwords do not match, as soon as he changes to focus to another input box
Additional InformationThe check for matching passwords seems to be tailored to the html structure used in the "normal" registration page, as it works just fine there.

Specifically, the problem comes from azure\src\js\widgets\oxinputvalidator.js, line 107:

var oForm = $( oInput ).parent(oOptions.listItem).parent(oOptions.list).parent(oOptions.form);

This code expects a very specific layout, which is not present when creating a user account during checkout, thus oForm is not found.
TagsNo tags attached.
Attached Files
validation_error.png (99,049 bytes)   
validation_error.png (99,049 bytes)   
ThemeAzure
BrowserAll
PHP Versionany
Database Versionany

Activities

Linas Kukulskis

2011-08-24 10:44

reporter   ~0005093

fixed js