ungleich-public/dynamicweb
6
0
Fork 0
Code Issues 1 Pull requests 2 Projects Releases Wiki Activity

Remove f-strings to make things compatible

Browse source
This commit is contained in:
PCoder 2025-05-21 11:17:15 +05:30
parent 60ddfa78b0
commit b6239159b2
1 changed files with 13 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
hosting/models.py
View file

@ -719,8 +719,10 @@ class UserCardDetail(AssignPermissionsMixin, models.Model):
if user_card_details.count() > 1: if user_card_details.count() > 1:
# Log a warning about the duplicate entries # Log a warning about the duplicate entries
logger.warning( logger.warning(
f"Multiple UserCardDetail objects found for card_id={card_id}. " "Multiple UserCardDetail objects found for card_id={}. "
f"Found {user_card_details.count()} objects. Using the latest one." "Found {} objects. Using the latest one.".format(
card_id, user_card_details.count()
)
) )
# Use the first object found # Use the first object found
user_card_detail = user_card_details.first() user_card_detail = user_card_details.first()
@ -732,14 +734,16 @@ class UserCardDetail(AssignPermissionsMixin, models.Model):
# Depending on expected behavior, you might want to raise an error or handle this case. # Depending on expected behavior, you might want to raise an error or handle this case.
# If the original get() call happened here, it would raise DoesNotExist. # If the original get() call happened here, it would raise DoesNotExist.
logger.error( logger.error(
f"No UserCardDetail found for card_id={card_id}." "No UserCardDetail found for card_id={}.".format(card_id)
) )
raise UserCardDetail.DoesNotExist(f"No UserCardDetail found for card {card_id}") raise UserCardDetail.DoesNotExist("No UserCardDetail found for card {}".format(card_id))
if user_card_details.count() > 1: if user_card_details.count() > 1:
# Log a warning about the duplicate entries # Log a warning about the duplicate entries
logger.warning( logger.warning(
f"Multiple UserCardDetail objects found for card_id={card_id}. " "Multiple UserCardDetail objects found for card_id={}. "
f"Found {user_card_details.count()} objects. Using the first one." "Found {} objects. Using the first one.".format(
card_id, user_card_details.count()
)
) )
# Use the first object found # Use the first object found
user_card_detail = user_card_details.first() user_card_detail = user_card_details.first()
@ -751,16 +755,15 @@ class UserCardDetail(AssignPermissionsMixin, models.Model):
# Depending on expected behavior, you might want to raise an error or handle this case. # Depending on expected behavior, you might want to raise an error or handle this case.
# If the original get() call happened here, it would raise DoesNotExist. # If the original get() call happened here, it would raise DoesNotExist.
logger.error( logger.error(
f"No UserCardDetail found for card_id={card_id}." "No UserCardDetail found for card_id={}.".format(card_id)
) )
raise UserCardDetail.DoesNotExist(f"No UserCardDetail found for card {card_id}") raise UserCardDetail.DoesNotExist("No UserCardDetail found for card {}".format(card_id))
except Exception as e: except Exception as e:
# Catch other potential exceptions during the filter/get process if necessary # Catch other potential exceptions during the filter/get process if necessary
logger.error(f"An unexpected error occurred while fetching UserCardDetail: {e}") logger.error("An unexpected error occurred while fetching UserCardDetail: {}".format(e))
raise raise
return user_card_detail return user_card_detail
@staticmethod @staticmethod
def get_ucd_from_stripe_cust_n_card_id(stripe_cust, card_id): def get_ucd_from_stripe_cust_n_card_id(stripe_cust, card_id):
try: try: