Opened 6 years ago
Closed 6 years ago
#50199 closed defect (bug) (fixed)
Theme Editor: Remove unused $has_templates boolean
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 5.5 | Priority: | normal |
| Severity: | trivial | Version: | |
| Component: | Themes | Keywords: | needs-patch good-first-bug |
| Focuses: | administration | Cc: |
Description
In wp-admin/theme-editor.php, the $has_templates boolean is a remnant from when the Theme Editor still had the "Templates" heading and is unused in the current logic.
Some history:
Attachments (1)
Change History (3)
Note: See
TracTickets for help on using
tickets.
Hi,
I am not sure that is right.
On line 77: $has_templates = false is declared.
On line 85: if $allowed_files is not empty, $has_templates will be set to true.
Then down on line 254: if the theme is a child theme, it uses that boolean value to decide if the message, This child theme inherits templates from a parent theme (Parent theme name)
To me it seems like it is still required?
Please let me know if I am wrong.
Thanks