Fix some issues

- Fix `has_contract_fallback_function`: it always returned `True`
- Fix ERC721 sorting. Q2 could be `NULL` due to the `LEFT JOIN`
- Fix a comment

Author: Uxio0

safe_transaction_service/contracts/tx_decoder.py

@@ -575,11 +575,7 @@ def has_contract_fallback_function(self, address: ChecksumAddress) -> bool:
         abi = self.get_contract_abi(address)
         if not abi:
             return False
-        return bool(
-            dict(fn_abi, name="fallback")
-            for fn_abi in abi
-            if fn_abi.get("type") == "fallback"
-        )
+        return any(fn_abi.get("type") == "fallback" for fn_abi in abi)
 
     @cachedmethod(
         cache=operator.attrgetter(

safe_transaction_service/history/models.py

@@ -750,7 +750,7 @@ def erc721_owned_by(
             owned_by_query += " AND Q1.address NOT IN (SELECT address FROM tokens_token WHERE spam = TRUE)"
 
         # Sort by token `address`, then by `token_id` to be stable
-        owned_by_query += " ORDER BY Q1.address, Q2.token_id"
+        owned_by_query += " ORDER BY Q1.address, Q1.token_id"
 
         with connection.cursor() as cursor:
             hex_address = HexBytes(address)

safe_transaction_service/history/services/balance_service.py

@@ -83,7 +83,7 @@ def __init__(self, ethereum_client: EthereumClient, redis: Redis):
         self.redis = redis
         self.cache_token_info = TTLCache(
             maxsize=4096, ttl=60 * 30
-        )  # 2 hours of caching
+        )  # 30 minutes of caching
 
     def _filter_tokens(
         self,