WordPress.org

Ready to get started?Download WordPress

Forums

[resolved] [Plugin: eShop] Multiple Emails on a Paypal Account (5 posts)

  1. csogilvie
    Member
    Posted 4 years ago #

    Hello

    I set up the eshop plugin on a website last night and came around to testing it.

    Paypal offers you the ability to add multiple email addresses to a paypal account, and then you can accept emails to any number of these addresses. This is a facility that both websites that I am in the process of developing with eShop uses.

    However, the Paypal IPN notification verification within eshop does not like this.

    Consider:

    Main Email Address: main@example.com
    Secondary Address: second@example.com

    I would like eshop to take the payments from second@example.com which then shows up in the IPN notification as:

    business: second@example.com
    receiver_email: main@example.com

    However, eshop only compares that the 'receiver_email' matches what is defined in the settings, it doesn't seem to compare the 'business' email which is the one the payment was actually sent to.

    This doesn't look like a large modification to the code, and would make the system work better for people that run different email addresses for their paypal accounts.

    Would suggest something like the following
    paypal.php, line 561:

    if($p->ipn_data['receiver_email']!= $eshopoptions['business']){

    ->

    $paypalRecipient = (isset($p->ipn_data['business'])) ? $p->ipn_data['business'] : $p->ipn_data['receiver_email'];
    if($paypalRecipient != $eshopoptions['business'])

    http://wordpress.org/extend/plugins/eshop/

  2. csogilvie
    Member
    Posted 4 years ago #

    (note the code posted above was unteted, and may actually vary by line number!)

  3. elfin
    Moderator
    Posted 4 years ago #

    yes I might add that in. It does cause people a few issues, thanks!

  4. csogilvie
    Member
    Posted 4 years ago #

    Thanks,

    I've just tested it on one of the sites I needed it for, and it seems to have worked correctly...

    Colin

  5. elfin
    Moderator
    Posted 4 years ago #

    added, but currently untested.

Topic Closed

This topic has been closed to new replies.

About this Topic

Tags