[wp-trac] Re: [WordPress Trac] #5418: remove unused variables from
wp-admin/*.php
WordPress Trac
wp-trac at lists.automattic.com
Sun Dec 9 00:07:50 GMT 2007
#5418: remove unused variables from wp-admin/*.php
-----------------------+----------------------------------------------------
Reporter: DD32 | Owner: anonymous
Type: defect | Status: reopened
Priority: normal | Milestone: 2.4
Component: General | Version: 2.4
Severity: normal | Resolution:
Keywords: has-patch |
-----------------------+----------------------------------------------------
Comment (by JeremyVisser):
Replying to [comment:13 DD32]:
> However, that filter is used elsewhere, and in those cases, the output
''may'' be modified.
But in this instance, the value is being ignored, so ''semantically'', I
believe it's better to do {{{do_action()}}}. (Although, I think
{{{do_action()}}} uses {{{apply_filters()}}} internally, so it's
definitely not for optimisation reasons.)
--
Ticket URL: <http://trac.wordpress.org/ticket/5418#comment:14>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list