Check for direct load of searchform.php about wpthemereview

[khacoder]

I have this sniff ready to go and will upload after final testing.

[justintadlock]

Somewhat related but might need its own ticket is that I sometimes see themes directly outputting the search form content within other templates rather than using get_search_form(). Just wanted to make a quick note.

[jrfnl]

From: #82 (comment)

I have run the sniff on the some of the theme of the repo and got some false positives

I don't think limiting the sniff to certain files is the way to go per se.
Let's take a step back and try to get a crystal clear definition of what this sniff should do and catch.
Let's write unit tests to match.
And only once that's done, let's look at the actual sniff again.

Let's continue this discussion here.

[dingo-d]

We agreed in the theme triage that we should check for usage of

require 'searchform.php'; // include and the _once counterparts

and

get_template_part( 'searchform.php')

And if these are found, we should throw a warning that users should use get_search_form() instead.

This falls under use core functionality category.

An ok case can be:

echo do_blocks( '<!-- wp:search {"label":""} /-->' ); 
get_search_form();
get_search_form(
  array(
    'label' => _x( '404 not found', 'Label', 'twentytwenty' ),
  )
);