Opened 5 years ago
Closed 5 years ago
#30881 closed Cleanup/optimization (duplicate)
Optimize _tx_resource_for_name() function in django/scripts/manage_translations.py
Reported by: | ankit1219 | Owned by: | nobody |
---|---|---|---|
Component: | Core (Other) | Version: | 3.0 |
Severity: | Normal | Keywords: | Optimize scripts |
Cc: | asrokx917@… | Triage Stage: | Unreviewed |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
The _tx_resource_for_name() function in django/scripts/manage_translations.py uses simple if else statement to return the Transifex resource name.
def _tx_resource_for_name(name):
""" Return the Transifex resource name """
if name == 'core':
return "django.core"
else:
return "django.contrib-%s" % name
You can use Python ternary operator to reduce code size and increase readability of the code.
def _tx_resource_for_name(name):
""" Return the Transifex resource name """
return "django.core" if name == 'core' else "django.contrib-%s" % name
It allows us to replace simple if statements with a single line expression. Increases code readability by reducing number of lines of code.
Duplicate of #30880.