Refactor/check effects interaction

[tcrosado]

Description

Change code order in some of the AppCoinsTimeLock functions and AppCoinsCreditBalance functions to implement check effect interaction pattern.

Motivation and Context

It is a best practice in Solidity to implement Check effect interaction pattern to prevent re-entry attacks.

Although the use of external contracts in this project is restricted to contracts produced within the project, it is a best practice to implement Check effect interaction pattern.

How Has This Been Tested?

Using the current unit tests

Screenshots (if appropriate):

Checklist:

• You have followed our contributing guidelines
• double-check your branch is based on dev and targets dev
• Pull request has tests (at least 80% coverage)
• Code is well-commented, linted and follows project conventions
• Documentation is updated (if necessary)
• Description explains the issue/use-case resolved and auto-closes related issues