Add or improve logs

[netojoaobatista]

This add or improve existing logs

[AleBark]

@netojoaobatista thanks for your PR, I noticed some points that I believe we could improve for better future maintenance, here they are:

• In most validators, there's a list of available currencies in each scope, how about creating a centralization mechanism, and then only invoking it in these validators? I believe that an Enum would make it much easier in case of adding new currencies to these validators in the future.

• Also, some validation methods have a try-catch and others use simple if logic in a variable called $isValid, is there any reason for this? I think it would be nice, if possible, to also standardize the validation methods.

• In case the last scenario is not possible, I think it is possible to simplify the validation code, instead of testing the $isValid variable more than once, something like this could be done:

Taken from CountryValidator.php

public function validate(array $validationSubject)
    {
        $available_currencies = ['COP','USD','EUR']; //This could be in a Enum class

        $isValid =
            $validationSubject['country'] == 'CO'
            && in_array($this->storeManager->getStore()->getBaseCurrencyCode(), $available_currencies);

        if(!$isValid){
            $this->_logger->info('Country is not CO or the store base currency is not in the available currencies');
        }

        return $this->createResult($isValid);
    }

..or even better, separate isValid logic to a separate function, should work fine as well

• This one is not related to the PR itself, but I noticed it when I was reading the updates on PaymentDataBuilder.php

if(isset($additionalData[DataAssignObserver::USE_SAVED_CC]) && $additionalData[DataAssignObserver::USE_SAVED_CC] && $this->_session->getQuote()->getCustomerId()){}

could be replaced with:

 if(!empty($additionalData[DataAssignObserver::USE_SAVED_CC]) && $this->_session->getQuote()->getCustomerId())

There are some files in this PR that could be a little more succinct with this change

• There are some files that are "coming up" with commented lines, could you remove those lines, please?
  Example: // $this->_logger->info('AuthorizationHandler :: response', [$response['payment_result']]);

[AleBark]

The suggestion are commented above

[netojoaobatista]

The suggestions are good, but out of the scope of this PR. A new PR for the validators will be made.