[Done v1.3.9e/f] 139c DoExpressCheckoutPayment variables missing

[stagebrace]

I noticed the PayPal emails returning sales info were missing the products details. I checked some data and found these variables were not being sent via DoExpressCheckoutPayment when using 139c versus 138.

L_NUMBER(x)
L_QTY(x)
L_NAME(x)
L_AMT(x)
L_TAXAMT(x)

[DrByte]

It should only skip those kinds of items when there are discounts or calculation problems which would throw off PayPal's validation.

I'd need to see your detailed PayPalXXXXXXXXXX.log debug log files to know more why it's doing that.

[stagebrace]

OK, see attachment.
for password see PM

[stagebrace]

I know this was not marked as fixed for 139d but I though I would see what it does. I installed the boilerplate store with no template changes. Installed PayPal express. Changed the price of an item to one dollar. And made a purchase.

Still missing. See attachment
L_NUMBER(x)
L_QTY(x)
L_NAME(x)
L_AMT(x)
L_TAXAMT(x)

This information is useful as a backup to the Zen Cart data. And PayPal can use it for "items not as described" complaints.

[stagebrace]

The following modification fixed the issue for me. I warn it may not fix it for others.

replace a line 1262 in file paypalwpp.php
if ($sumofLineItems == 0) {

with
if ($sumofLineItems === 0) {


There must be a difference between being Equal and Identical. Or the change may just shut down the equation altogether and bypass the needed logic. I don't know, not smart enough. I need the line items and am willing to take the chance at the moment.

Dennis

[stagebrace]

I may have found the bug.
Is
$sumofLineItems
and
$sumOfLineItems

two different varaiables?

[DrByte]

Yes, that would be a problem. The uppercase "O" should be used.
We'll roll that fix into v1.3.9e.

[stagebrace]

Dennis does the happy fat man dance around the house. Happy to make the find.

[TANSTAAFL]

Thank You Stagebrace ! Recently upgraded from 1.3.8a to 1.3.9d and found that order details were now missing from PayPal packing slips. Correcting the misspelling of the variable $sumOfLineItems solved the problem immediately. My PayPal packing slips are important to the business and now they work correctly.

[dbltoe]

Would that mean the $sumoLineTax should be changed as well?

Payphone line 606 and others. Seems like same case mistake everywhere sumofLineItems appears.