[wp-trac] [WordPress Trac] #31093: Make $tag argument optional for has_shortcode()
WordPress Trac
noreply at wordpress.org
Thu Jan 22 10:27:59 UTC 2015
#31093: Make $tag argument optional for has_shortcode()
------------------------------------------------+--------------------------
Reporter: danielbachhuber | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Future
Component: Shortcodes | Release
Severity: normal | Version:
Keywords: has-patch dev-feedback 2nd-opinion | Resolution:
| Focuses:
------------------------------------------------+--------------------------
Changes (by tyxla):
* keywords: needs-patch => has-patch dev-feedback 2nd-opinion
Comment:
Not sure if that makes sense, or if it is necessary at all.
My point is, if a function would check for any shortcodes within the
content, it would make more sense if a new function
`contains_shortcodes()` is created for that purpose. Mainly because
`has_shortcode()` would do totally different things in the two cases (with
and without a parameter). And the function name would make more sense
IMHO.
Also, I believe a function for checking whether there are any shortcodes
is not necessary. It is quick enough to check the original string against
the result of `strip_shortcodes()` over the string. Something like this:
{{{
<?php
$original_content = 'some content that might have [shortcodes]';
$stripped_content = strip_shortcodes($original_content);
if ( $original_content === $stripped_content ) {
echo 'There are no shortcodes within the content.';
} else {
echo 'There are one or more shortcodes within the content.';
}
?>
}}}
Of course, a built-in function for this can turn out handy for developers.
Attaching a patch here (
https://core.trac.wordpress.org/attachment/ticket/31093/31093.patch ) in
case the core committer team decide to include it. Tests included.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/31093#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list