refactor: use legacy approach first, fallback onto new parsing approach

relayer/chains/cosmos/tx.go

@@ -507,10 +507,8 @@ func parseEventsFromTxResponse(resp *sdk.TxResponse) []provider.RelayerEvent {
 		return events
 	}
 
-	// After SDK v0.50, indexed events are no longer provided in the logs on
-	// transaction execution, the response events can be directly used
-	if len(resp.Events) != 0 {
-		for _, event := range resp.Events {
+	for _, logs := range resp.Logs {
+		for _, event := range logs.Events {
 			attributes := make(map[string]string)
 			for _, attribute := range event.Attributes {
 				attributes[attribute.Key] = attribute.Value
@@ -520,12 +518,12 @@ func parseEventsFromTxResponse(resp *sdk.TxResponse) []provider.RelayerEvent {
 				Attributes: attributes,
 			})
 		}
-		return events
 	}
 
-	// fallback on legacy behaviour
-	for _, logs := range resp.Logs {
-		for _, event := range logs.Events {
+	// After SDK v0.50, indexed events are no longer provided in the logs on
+	// transaction execution, the response events can be directly used
+	if len(events) == 0 {
+		for _, event := range resp.Events {
 			attributes := make(map[string]string)
 			for _, attribute := range event.Attributes {
 				attributes[attribute.Key] = attribute.Value
@@ -536,6 +534,7 @@ func parseEventsFromTxResponse(resp *sdk.TxResponse) []provider.RelayerEvent {
 			})
 		}
 	}
+
 	return events
 }