Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#21149 closed enhancement (fixed)

Misspelled @return statement and wrong function signatures in PHPDocs wp-includes/formatting.php

Reported by: c3mdigital's profile c3mdigital Owned by: markjaquith's profile markjaquith
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.1
Component: Inline Docs Keywords: has-patch commit
Focuses: Cc:

Description


Attachments (3)

21149.patch (4.6 KB) - added by c3mdigital 12 years ago.
21149.2.patch (2.8 KB) - added by SergeyBiryukov 12 years ago.
21149.3.patch (5.2 KB) - added by SergeyBiryukov 12 years ago.

Download all attachments as: .zip

Change History (9)

@c3mdigital
12 years ago

#1 @c3mdigital
12 years ago

  • Cc chris@… added
  • Component changed from General to Inline Docs

#2 follow-up: @SergeyBiryukov
12 years ago

  • Milestone changed from Awaiting Review to 3.5

Reviewed and adjusted the patch. Didn't include some changes because they don't seem valid to me.

#3 in reply to: ↑ 2 ; follow-up: @c3mdigital
12 years ago

Replying to SergeyBiryukov:

Reviewed and adjusted the patch. Didn't include some changes because they don't seem valid to me.

Thanks for reviewing the patch. One thing I wanted to point out was the @usedby tag. This is a virtual tag, it is created by @uses to cross-reference the used element. http://manual.phpdoc.org/HTMLSmartyConverter/HandS/phpDocumentor/DocBlockTags/parserUsedByTag.html

#4 @georgestephanis
12 years ago

+1 to 21149.3.patch

#5 in reply to: ↑ 3 @SergeyBiryukov
12 years ago

  • Keywords commit added

Replying to c3mdigital:

This is a virtual tag, it is created by @uses to cross-reference the used element.

Thanks, got it. I didn't notice at first that wp_iso_descrambler() is actually used by wp-mail.php, not by wp_mail().

21149.3.patch also replaces other instances of @usedby.

#6 @markjaquith
12 years ago

  • Owner set to markjaquith
  • Resolution set to fixed
  • Status changed from new to closed

In [21241]:

Bunch of PHPDoc corrections and cleanups. props c3mdigital. props SergeyBiryukov. fixes #21149

Note: See TracTickets for help on using tickets.