Comment variable is wrong for shortcode tags

[uttam007]

Filters the list of shortcode tags to remove from the content comment variable is not correct. It should be $tags_to_remove instead of $tag_array.

[uttam007]

Updated the comment parameter name in the comment.

[netweb]

Hi @uttam007, welcome to Trac and thanks for creating his ticket :)

Could you refresh the patch please so that the @param's descriptions are aligned:

src/wp-includes/shortcodes.php

@@ -609 +609 @@
          *
          * @since 4.7.0
          *
-         * @param array  $tag_array Array of shortcode tags to remove.
-         * @param string $content   Content shortcodes are being removed from.
+         * @param array  $tags_to_remove Array of shortcode tags to remove.
+         * @param string $content        Content shortcodes are being removed from.
          */
         $tags_to_remove = apply_filters( 'strip_shortcodes_tagnames', $tags_to_remove, $content );}}}

[subrataemfluence]

Please mention your version of WordPress where you found the issue. And curious to know about the name of your diff file. It has 7361 in it. Usually the diffs are created with the ticket number like 44691.diff. Any specific reason?

[uttam007]

Replying to subrataemfluence:

I have uploaded another attachment with the ticket number and the Wordpress version is 4.9.7.

[SergeyBiryukov]

In 43549:

Docs: Correct parameter name in strip_shortcodes_tagnames filter DocBlock.

Props uttam007.
Fixes #44691.

[SergeyBiryukov]

Switching milestone due to the focus on the new editor (Gutenberg) for WordPress 5.0.