WordPress.org
Get WordPress

Make WordPress Core

Opened 4 years ago

Closed 4 weeks ago

#53580 closed defect (bug) (reported-upstream)

Add support for Tested up to version to themes

Reported by: utz119's profile utz119 Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Themes Keywords: has-patch
Focuses: Cc:

Description (last modified by SergeyBiryukov)

Hello. In addition to Requires at least and Requires PHP in 5.4 #44592, Tested up to was added as a theme requirement since then.

The scope of this ticket is to add support for Tested up to to WP_theme so that it can be parsed correctly and implemented in the theme-check plugin ​#341.

Attachments (2)

class-wp-theme.php​ (50.2 KB) - added by utz119 4 years ago.
Added support for Tested up to
53580.diff​ (514 bytes) - added by utz119 4 years ago.

Download all attachments as: .zip

Change History (6)

@utz119
4 years ago

Added support for Tested up to

@utz119
4 years ago

#1 @utz119
4 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
4 years ago

  • Description modified (diff)

Hi there, welcome to WordPress Trac! Thanks for the ticket.

Just linking to some previous tickets here: #13780, #16868, #28574, #37704.

It seems that the new header would not actually be used by core yet, only by the Theme Check plugin? Would it be possible for the plugin to check the header without any changes to core, e.g. using the ​get_file_data() function?

#3 @utz119
4 years ago

Hi @SergeyBiryukov 

Thanks so much for the info. get_file_data() function worked great!

#4 @poena
4 weeks ago

  • Milestone Awaiting Review deleted
  • Resolution set to reported-upstream
  • Status changed from new to closed

I consider this solved upstreams, lets close this ticket.

Note: See TracTickets for help on using tickets.