Hello,

This module has been working good for us for a year now.

Recently, however, and I'm not sure why this has just come up, but we've had two or three customers figure out that they can type "box" instead of PO Pox or P.O. Box and get through the system.

example - doesn't work:
P.0. Box 1234
PO Box 1234

example - works!!:
box 1234

This is proving to be a liability for us because we can't send our products with a tracking number to PO Boxes.

I looked at the code in the module, and it appears that it is just looking for the string pobox in the address fields.

How would we make this a bit more complex and make sure that "box" wasn't at the beginning of the address in addition to the existing checks?

Thanks.

CommentFileSizeAuthor
#2 uc_po_condition-1003746-1.patch488 bytesAri Gold

Comments

jasonabc’s picture

jasonabc commented

hmmmm tricky one because 'box' could quite legitimately appear in the street name (123 Boxtel Street for example). You'd have to look at the first 3 characters entered and match against Box. I'll take a look at this when I have a moment (or if you want to have a stab yourself and post a patch go for it ;-)

Ari Gold’s picture

Ari Gold commented
Status: Active » Needs review
StatusFileSize
new uc_po_condition-1003746-1.patch488 bytes

Patch attached that looks for 'box' in the first 3 characters...