postcode-nl / PostcodeNl_Api_MagentoPlugin

A Magento plugin 1.9 that implements the Postcode.nl API for Dutch addresses
BSD 2-Clause "Simplified" License
21 stars 12 forks source link

Housenumber additions are re-checked on page load #18

Closed DaanBroekhof closed 8 years ago

DaanBroekhof commented 9 years ago

@paales wrote: https://github.com/postcode-nl/PostcodeNl_Api_MagentoPlugin/issues/14#issuecomment-65399242

@DaanBroekhof Still some errors, http://cl.ly/2Q0A3H2n3H3L

  • It should select the XI suffix directly on page reload, not suggesting that the suffix is wrong.
DaanBroekhof commented 9 years ago

This is an issue where the browser inserts the remembered form values, similar to having an address already present in the form field. Our extension has no good way of determining if the suffix was just typed in by the customer or if it was an existing value that was 'approved' in the past. So it runs the validation again, resulting in the warning. This behaviour is also present when the third-street-field-for-suffix option is not used. Not sure if I can fix this behaviour - we don't store if an 'unknown' suffix is used anyway, that state is only known during the request itself.