Two loans instead of one

[BadrAly]

When a patron requests then checkouts out an item, the system creates two loans instead of one

Steps to reproduce the behavior:

1. A patron places a request on an item to pickup at an external library, system creates a loan (pid 1 for example) with a state PENDING
2. A librarian sends the item to the pickup library, the loan state become IN_TRANSIT_FOR_PICKUP
3. A librarian receives the item at the destination library, the loan state becomes ITEM_AT_DESK
4. A librarian checks out the item to a patron, the loan state becomes CANCELLED and a new loan (pid=2) is created with state ON_LOAN

Expected behavior
System should continue to work on the first loan and never creates a new loan.