[ 
https://issues.apache.org/jira/browse/OFBIZ-4905?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14354974#comment-14354974
 ] 

Oleg Andreyev commented on OFBIZ-4905:
--------------------------------------

A few issues I found when tried this code.

1. UtilValidate.isNotEmpty(emailAddress.getString("infoString")) throw NPE if 
user has no email.
2. CardType.valueOf(cardType.toUpperCase()) does not work well for MasterCard 
because they use notation with underscore MASTER_CARD. So, this patch does not 
work with MasterCard, AmericanExpress and DinersClub but work with Visa.
3. Perhaps author developed this for ecommerce component. If I try to edit 
credit card from a party profile in Party Manager (other than mine) it can not 
find CIM profile because use userLogin.partyId. Why don't use partyId from 
parameters?
4. The patch does not support a few cards per party. Error E00039 - A duplicate 
record with ID 0000000 already exists. More over it display correct card ending 
for first card only. I believe card operations works for first card too.
5. Perhaps this problem might not exist in API version that author used to 
develop the feature but cardCode is mandatory. Since we have no CVV field this 
rises the problem.

Not sure this is useful but decide to share my findings.

> Authorize.net Customer Information Manager
> ------------------------------------------
>
>                 Key: OFBIZ-4905
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-4905
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: accounting
>            Reporter: Sam Hamilton
>         Attachments: Readme.txt, authorize_CIM_V8.patch
>
>
> This is a patch to allow OFBiz to store credit card information inside the 
> Authorize.net Customer Information Manager platform. 
> The required SDK can be downloaded from 
> http://developer.authorize.net/downloads/
> As this a major patch we would love some developer feedback and reviews
> Thanks
> Sam 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to