Home Forums Plugins Offline Credit Card Processing Support [Resolved] Extra XXXX

Welcome to our community support forums! We're here to help - but if you have an urgent request for a Pro Plugin, you will get a prioritised response through our Premium Support page.

Viewing 6 posts - 1 through 6 (of 6 total)
  • Author
    Posts
  • #6930
    brianfbb
    Participant

    Just bought and installed the latest version 1.3.1 of offline credit card processing, using WP 3.0.1 and E Commerce 3.7.7. I notice that sometimes the one-click credit card detail action popup will not display the rest of the number. For example, if the number submitted was:
    1111-2222-3333-4444

    then the email containing the last four digits of the code will be 4444

    However, the popup will display XXXX-XXXX-3333-XXXX
    Or sometimes: XXXX-XXXX-XXXX-XXXX

    Is this a bug or just an issue with submitting invalid credit cards?

    #6931
    epschmidt
    Participant

    I noticed a similar issue. For me, the additional numbers being blocked out were the same as the last 4 digits. So 1111222233332222 would be 1111xxxx3333xxxx. This is because the code was doing a search/replace on the string for anything that matched the last 4 digits. I believe this will be addressed in the next version of the plugin.

    #6932
    saqus
    Participant

    I have the same issue. Please advise.

    #6933
    epschmidt
    Participant

    I made a change to the offline-credit-card.php file in Merchants. I am still testing this change, but it seems to be working just fine for me. As always, use at your own risk. I’m using version 1.3 and changed line 182 from this…

    $stored[] = array( 'credit_card','Credit Card #',str_replace( substr( $data['ACCT'],-4,4 ),'XXXX',$data['ACCT'] ) );

    To this…

    $stored[] = array( 'credit_card','Credit Card #',substr_replace( $data['ACCT'], 'XXXX', -4, 4 ) );

    #6934
    Michael Visser
    Keymaster

    Hi epschmidt, your change works a treat; preparing a minor Plugin update to resolve this.

    #6935
    saqus
    Participant

    Seems to be working after the updates. Thanks!

Viewing 6 posts - 1 through 6 (of 6 total)
  • The topic ‘[Resolved] Extra XXXX’ is closed to new replies.