MHL3060 / bentux

Automatically exported from code.google.com/p/bentux
0 stars 0 forks source link

phone field has to match hk phone number standard #72

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
need a regular expression that takes care of only 8 digits phone number.

--->>    [\d{8}]  this doesn't seem to work, any idea?

Already make the phone field mandatory.

Original issue reported on code.google.com by lhkh...@gmail.com on 7 Sep 2010 at 4:36

GoogleCodeExporter commented 9 years ago
Did you restart the app after you made the changes?

Original comment by benli3...@gmail.com on 7 Sep 2010 at 5:09

GoogleCodeExporter commented 9 years ago
No. I worked on the local

Original comment by lhkh...@gmail.com on 8 Sep 2010 at 1:08

GoogleCodeExporter commented 9 years ago
also as for the validation, password and confirm password has to be the same. 
Should prompt user if they have entered mismatched password.

Original comment by lhkh...@gmail.com on 9 Sep 2010 at 9:47

GoogleCodeExporter commented 9 years ago
this should work out of the box.

Original comment by benli3...@gmail.com on 10 Sep 2010 at 3:20

GoogleCodeExporter commented 9 years ago
I think you have removed the password checking before.
you should add it back. 

please check the revision of the validation.xml and add it back

Original comment by benli3...@gmail.com on 10 Sep 2010 at 3:21

GoogleCodeExporter commented 9 years ago
no, is not working for the 8 digits

the source code on live is:
<constant-value>^\(?(\d{3})\)?[-| ]?(\d{3})[-| ]?(\d{4})$</constant-value>

I can create a new login, with different password (confirm password is 
different), and with an emptied phone number (will complaint with any digit). 

I don't think I have removed the password checkin, will double check.

Original comment by lhkh...@gmail.com on 10 Sep 2010 at 3:59

GoogleCodeExporter commented 9 years ago
\d{8} should work

Original comment by benli3...@gmail.com on 10 Sep 2010 at 4:26

GoogleCodeExporter commented 9 years ago
^\(?(\d{4})\)?[-| ]?(\d{4})$

fixed with this.

Original comment by lhkh...@gmail.com on 22 Oct 2010 at 5:54