PayPal Pro UK Express Checkout Button Placement

When using the wizard to implement a PayPal Pro checkout the Express Checkout Button is placed on the checkout.php page.

This means that potentially an order could be submitted with no information collected from the checkout form as you would validate the checkout form on confirm.php normally as that is the target of the checkout form itself, yes I know I could use client side validation but I don't trust it as it is to easy to bypass and I always validate server side.

Surely it would make sense to collect billing and delivery information before taking the client to the payment page ?

Interested to hear thoughts on this issue.

In Express Checkout, the customer sets the billing and shipping address on the Paypal server, It is not necessary to collect this information before transferring, because if is set as part of the customers Paypal Account.

Jason,

ok I can see that the shipping address should be set by the post from PayPal (in my test this did not happen for some reason which I need to debug) but the billing info on pp_confirm.php seems to rely on it being set elsewhere
e.g.

<?php
if (!session_id()) session_start();
if(($_SERVER["REQUEST_METHOD"] == "POST") && (isset($_SERVER["HTTP_REFERER"]) && strpos(urldecode($_SERVER["HTTP_REFERER"]), urldecode($_SERVER["SERVER_NAME"].$_SERVER["PHP_SELF"])) > 0) && isset($_POST))     {
  $_SESSION["eCartCheckoutForm_firstname"] = "".((isset($_POST["firstname"]))?$_POST["firstname"]:"")  ."";
}
?>

FWIW I still think people would be potentially confused by not having the full amount inc shipping provided to them before sending them off to a payment processor (I know it sends you back to the site but not everyone else out there would)

I have created a support ticket so we can look into this further. Please log into your support history to view and edit the ticket:
supporthistory.php

I have attached the checkout and paypal files to the support ticket

This was a problem with using the wrong variable in the get payer profile server bahavior.

We changed it to use the $i8_merchant variable instead.

Thanks for your help today Jason.

No worries.