Opened 43 minutes ago

#36763 assigned Cleanup/optimization

Small code refactoring in TemplateDoesNotExist.__init__

Reported by: Pravin Owned by: Pravin
Component: Template system Version: 5.2
Severity: Normal Keywords: TemplateDoesNotExist
Cc: Pravin Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

I noticed TemplateDoesNotExist.__init__ could be made more readable and more clear.
Instead of this

    def __init__(self, msg, tried=None, backend=None, chain=None):
        self.backend = backend
        if tried is None:
            tried = []
        self.tried = tried
        if chain is None:
            chain = []
        self.chain = chain
        super().__init__(msg)

This could have been more simple, clear and readable like this

    def __init__(self, msg, tried=None, backend=None, chain=None):
        self.backend = backend
        self.tried   = tried if tried else []
        self.chain   = chain if chain else []
        super().__init__(msg)

Change History (0)

Note: See TracTickets for help on using tickets.
Back to Top