Fix issue on processing Safe internal txs

- Only one internal tx was processed at a time, as internal tx `iterator` was consumed
- Cache does not need to be emptied before, can be only emptied after processing
- `return` position is not relevant, but it's more clear inside of the `try` block

Author: Uxio0

safe_transaction_service/history/indexers/tx_processor.py

@@ -384,28 +384,24 @@ def process_decoded_transactions(
         :return:
         """
         internal_tx_ids = []
-        results = []
+        results: list[bool] = []
         contract_addresses = set()
 
-        # Clear cache for the involved Safes
-        for internal_tx_decoded in internal_txs_decoded:
-            contract_address = internal_tx_decoded.internal_tx._from
-            contract_addresses.add(contract_address)
-            self.clear_cache(safe_address=contract_address)
-
         try:
             for internal_tx_decoded in internal_txs_decoded:
+                contract_address = internal_tx_decoded.internal_tx._from
+                contract_addresses.add(contract_address)
                 internal_tx_ids.append(internal_tx_decoded.internal_tx_id)
                 results.append(self.__process_decoded_transaction(internal_tx_decoded))
 
             # Set all as decoded in the same batch
             InternalTxDecoded.objects.filter(internal_tx__in=internal_tx_ids).update(
                 processed=True
             )
+            return results
         finally:
             for contract_address in contract_addresses:
                 self.clear_cache(safe_address=contract_address)
-        return results
 
     def __process_decoded_transaction(
         self, internal_tx_decoded: InternalTxDecoded