This repository has been archived by the owner on Jan 8, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
JCB credit card validation could be 15 digits long #6278
Comments
Closed
Ocramius
pushed a commit
that referenced
this issue
Nov 27, 2014
Ocramius
pushed a commit
that referenced
this issue
Nov 27, 2014
Ocramius
added a commit
that referenced
this issue
Nov 27, 2014
Ocramius
added a commit
that referenced
this issue
Nov 27, 2014
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
…dding missing `@group` docblock referencing the pull request that introduced the test
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
…dding missing docblock
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
…dding missing docblock parameters, minor cs fixes
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
…endframework#6278-jcb-credit-card-validator-allowed-ranges-update' Close zendframework/zendframework#6927 Close zendframework/zendframework#6278
gianarb
pushed a commit
to zendframework/zend-validator
that referenced
this issue
May 15, 2015
…endframework#6278-jcb-credit-card-validator-allowed-ranges-update' into develop Close zendframework/zendframework#6927 Close zendframework/zendframework#6278 Forward port zendframework/zendframework#6927 Forward port zendframework/zendframework#6278
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The JCB credit card validation is not right. JCB could be 15, 16 digits long. So the file
Zend/Validator/CreditCard.php should be modified on line 87:
and be replaced with the following:
Also the prefixes 2131, 1800 corresponding to the 15 digits credit cards must be added. Just update line 111:
Adding the new prefixes:
The text was updated successfully, but these errors were encountered: