Opened 5 years ago
Closed 5 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