You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With WP 6.2, the %i placeholder has been introduced, that can be used for table or field names in queries. Until now, WP developers had to phpcs:ignore those instances because there was no way of dealing with this.
We have a lot of queries where we do that and now we're able to improve our handling of those with the new placeholder. This task is the first step of this.
Run vendor/bin/phpcs --sniffs=WordPress.DB.PreparedSQL,WordPress.DB.PreparedSQLPlaceholders --ignore-annotations in all Yoast plugins
Manually review the results and single out the ones that are about unprepared table/field names. An example is this one.
Create a Google sheet with all those cases
That sheet will later used in order to improve our handling of those unprepared table/field data, as per this.
The content you are editing has changed. Please copy your edits and refresh the page.
With WP 6.2, the
%i
placeholder has been introduced, that can be used for table or field names in queries. Until now, WP developers had tophpcs:ignore
those instances because there was no way of dealing with this.We have a lot of queries where we do that and now we're able to improve our handling of those with the new placeholder. This task is the first step of this.
vendor/bin/phpcs --sniffs=WordPress.DB.PreparedSQL,WordPress.DB.PreparedSQLPlaceholders --ignore-annotations
in all Yoast pluginsThat sheet will later used in order to improve our handling of those unprepared table/field data, as per this.
Tasks
The text was updated successfully, but these errors were encountered: