PayPal vs Multiple Items

I agree ... it's disconcerting not to see a description in Paypal of the shopping cart items.

From my limited understanding of php, it looks like the paypal.module's function "paypal_goto" (starting at line 108) is what needs to change, but I don't know how to add the item description to the existing text.

Anyone able to help out?

Thanks,
Anna Marie

Fixed in v4

Where can I get a copy of the version 4 paypal module with Gordons fix? I'd like to look at the code and see if I can alter it to work with v3.3. I'm trying to alter v3.3 paypal.module already, but I cannot seem to get the per item shipping (using flexipay) from the $txn variable to pass to the paypal cart.

I assume your fix is for ecommerce v4 not v3.4? If so where can I track it's progress.

TIA
Drew Reece

This feature is for e-Commerce v4, which is not using the payment module, but the new ec_receipt module. The Payment system for e-Commerce has been completely rewritten along with the paypal module.

You can follow the development in CVS or at http://git.drupalecommerce.org where it is easy to track all the changes for any branch.

Well this annoyed me, so I made a patch for the current ECommerce (5.x-3.4).

Hopefully someone can review & commit it. It will pass the item name if it's the only item in the cart, otherwise it passes 'Cart containing X items' to PayPal.

It looks a little prettier than the current approach, it should be altered to pass the individual items for carts with more than one item, I just can't get my head around the PayPal documentation right now.

HTH,
Drew

I'm attaching my patch that sends the entire cart contents to PayPal. The PayPal payment URL has to change in order to pass the entire contents.

WARNING! VERY LITTLE TESTING has been performed on this patch. I've gotten it to work reliably on a test system with the PayPal developer site, however I haven't had time to test it on a production system yet.

The patch is against 5.x-3.x-dev from about 200710 so it may not patch cleanly to the current 3.x-dev code but you should get the idea and be able to patch by hand if it doesn't work.

ASIDE: I've been slowly adding functionality to my own eCommerce source tree based on 3.x-dev because it's stable and does almost everything I need. I'd love for some of these patches to be committed to the 3.x branch so that it can continue to grow and provide some small updated functionality.

Thanks again to the EC developers for all their hard work

Hi,

I'm looking at the paypal-sendallitems-ec5.x-3.x-dev patch, it seems ok so far.

My IPN debug returns with no transaction ID, it isn't passed correctly to PayPal when there are multiple items (the $itemname needs to be fixed for multiple items).

As far as I can see it doesn't seem to pass the postage to PayPal. I seem to remember reading that when using a Paypal '_cart' you need to pass a shipping charge for each item eg 'shipping_x'

I have an elaborately hacked flexicharge and custom.inc that will need to pass the per item postal charges, so I'm trying to figure out how to add it to this.

FYI I have 3 postage rates, one product type, each product gets assigned one postage rate in the node/edit form.
The rate can increment for every item in that postage class. I'm trying to clean it up for contributing back incase anyone is interested … how do you submit the changes to CVS?

I'll post back when I have a patch.

Drew

Someone with commit access needs to approve the patch and then apply it to CVS.

FYI, most of my patch is backported code from ec4. I just looked at the way they were submitting cart contents and modified the 3.x PayPal code to somewhat match it.

The ec4 code does not have a shipping_item1, etc, although that doesn't mean one can't use those variables. That functionality may exist in the PayPal API, but neither my code, nor the ec4 PayPal code uses it.

It seems kinda silly that Paypal allows per item shipping but EC4 doesn't. I was hoping these issues would be getting fixed in EC4 since it seems EC3 is now not adding features.

FWIW I found that passing shipping (via shipping variable) to paypal doesn't work for a '_cart'. It needs to use the cart_handling variable.
Sadly the Paypal dev docs don't mention this and all recommend using the shipping and shipping2 variable (for shipping cost per cart) that just doesn't work. The per item shipping works though.

I think my modifications mean that I have veered away from a basic EC store. The changes require flexicharge for example, so I'm uncertain about posting the code since it's specific to my site.

Drew