Refreshing payment_complete creates a new payment with approving_amount etc set to 0

[gonzalovilaseca]

If the user accidentally refreshes the browser at payment_complete, a new payment is created with all values set to zero, and state STATE_APPROVING.
The cause is this piece of code:

$instruction = $order->getPaymentInstruction();
if (null === $pendingTransaction = $instruction->getPendingTransaction()) {
$payment = $this->ppc->createPayment($instruction->getId(), $instruction->getAmount() - $instruction->getDepositedAmount());
} else {
$payment = $pendingTransaction->getPayment();
}

Shouldn't a check be placed before?
Something like:
if (( $instruction->getAmount() - $instruction->getDepositedAmount() ) > 0) {
if (null === $pendingTransaction = $instruction->getPendingTransaction()) {
$payment = $this->ppc->createPayment($instruction->getId(), $instruction->getAmount() - $instruction->getDepositedAmount());
} else {
$payment = $pendingTransaction->getPayment();
}
..etc
}