Opened 10 years ago

Closed 10 years ago

#24409 closed Cleanup/optimization (fixed)

Combine filesystem and app directories template loader implementation

Reported by: Preston Timmons Owned by: nobody
Component: Template system Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

These two loaders are functionally equivalent. The difference is the list of directories they look in. If the loader implementation is combined, there won't be a need to duplicate tests between the two.

Change History (3)

comment:1 by Preston Timmons, 10 years ago

Has patch: set

comment:2 by Tim Graham, 10 years ago

Triage Stage: Unreviewed → Accepted

comment:3 by Tim Graham <timograham@…>, 10 years ago

Resolution: → fixed
Status: new → closed

In 80d6b6b863109cfb60a7fcc5936136e347ecd57a:

Fixed #24409 -- Combined the app_directories and filesystem loader implementation.

Besides the directories they look in, these two loaders are functionally
the same. This removes unnecessary code duplication between the two.

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