_check_theme_deprecated_files() wording is incorrect

[greenshady]

I'd like to open this up for more discussion on what constitutes an "incomplete" theme. My themes don't use a sidebar.php file and are by no means incomplete. This message is incorrect and will be confusing to my users:

The current theme is incomplete as it is missing sidebar.php. Please update your theme to include these files as you are currently relying on deprecated behaviour.

This is regarding changeset 14369:
​http://core.trac.wordpress.org/changeset/14369

Simply because a file doesn't exist doesn't mean a theme is incomplete nor does it mean that a theme is relying on deprecated behavior. I might be able to understand header.php and footer.php a little more (though they aren't required), but sidebar.php shouldn't ever be thought of as a requirement. Ever used a single-column theme?

I either propose removing the message or updating the wording to something that is correct.

[scribu]

Simply because a file doesn't exist doesn't mean a theme is incomplete nor does it mean that a theme is relying on deprecated behavior.

I agree. The correct way to check for deprecated behaviour (i.e. relying on files from the old default theme) would involve seeing if any of those files are actually used anywhere in the theme.

[westi]

If someone wants to come up with a better way of checking a patch would be very welcome.

For now I will disable the checks.

[westi]

(In [14411]) Remove the deprecation message for now as it shows too many false positives. See #13230.

[hakre]

Related: #14999

[nacin]

Closing as fixed on 3.0.