What is the purpose of this line of code

Hi everyone,

I am just doing some debugging on my clients OpenCart (1.5.6) website. Basically, the client said that when they complete an order the cart doesn't always clear. They are using the box-standard PayPal Standard payment method.

Whilst debugging I saw that the following file has this line of code: I am confused to why this if condition is required as this is in the success.php page. Surely, when the success.php page is called, this would be an indicator that the order ID does exist as the order was successfully created, thus, this condition isn't needed (however the code within the condition is).

Just wondering what everyone elses thoughts are on this?

it is often prudent to have checks for this sort of thing in case someone refreshes a page or clicks back on their browser etc. it can prevent duplicate orders or someone happening on the page by accident

You can manually load the success page at anytime.. or you could refresh the success page which at that point would no longer have the order id since they are unset.

Qphoria wrote:You can manually load the success page at anytime.. or you could refresh the success page which at that point would no longer have the order id since they are unset.

I see! Thank you both for getting back.

I knew I was probably overlooking something!