🐛 Fix banking interaction zone

[NewtyMan]

Description

This PR introduces a fix for the following issue #144.

This problem appears/happens when the resource is configured to not use qb-target, but instead uses the zones to determine whether the player can open the bank or not.

This bug occurs, because currently the while loop inside the onPlayerInOut callback conditions the local variable instead of a global one. This means that when the player leaves, a new event is fired (new callback function executed) which has the isPointInside set to false, but the previous callback function which is still running the while loop is not aware of it, so it continues to run until the player presses E for the first time.

I also removed the break statement on line 210, since it was causing another issue/bug. This bug is that if you close the banking window, you needed to walk out of the banking zone and back in, if you want to open the window again. If you do not walk out, you still have the Open Bank notification, but pressing E does nothing. This happens, because the break statement breaks the while loop (which ends the listener for E key) until you walk out of the zone and then back in.

Questions:

• Have you personally loaded this code into an updated qbcore project and checked all it's functionality?
• Does your code fit the style guidelines?
• Does your PR fit the contribution guidelines?