Some issues with _doing_it_wrong()

[pavelevap]

$message in _doing_it_wrong() function calls should always end with point, because it is part of the whole paragraph, see attached screenshot.

$version in _doing_it_wrong() function calls should always be only WordPress version, because it is part of other sentence This message was added in version %s.

For missing callable function it would be helpfull refering to add_shortcode() instead of do_shortcode_tag(), I guess.

We are also missing some translators comments (not part of current patch). For example %s must set a database connection for use with escaping. is hard for translators when they do not know what is %s.

[swissspidy]

+1 for adding fullstops and removing spaces.

+1 for changing WPAPI-2.0 to 4.4.0. Perhaps in a separate ticket though. (ping @rmccue)

-1 to changing __FUNCTION__ to 'add_shortcode'

Note: The _doing_it_wrong for the shortcode was added in [32867] (see #32445).

+1 for adding a translator comment to that message you mentioned. In that message, %s refers to the database access abstraction class (usually wpdb or a class extending wpdb).

[SergeyBiryukov]

Replying to swissspidy:

+1 for changing WPAPI-2.0 to 4.4.0. Perhaps in a separate ticket though. (ping @rmccue)

See #34490.

[DrewAPicture]

In 35542:

I18N: Add translator comments to three _doing_it_wrong() messages added for 4.3 and 4.4.

Also adds a full-stop to an invalid taxonomy message introduced in [34745] and later modified in [35428] for wp_insert_post().

Props pavelevap.
Fixes #34249.

[SergeyBiryukov]

In 35543:

Use correct placeholders in translator comments added in [35542].

See #34249.

[SergeyBiryukov]

In 35544:

I18N: Add a translator comment for the string introduced in [24758] and made translatable in [29840].

Props swissspidy.
See #34249.