Opened 9 months ago
Closed 9 months ago
#21893 closed enhancement (fixed)
Code formatting and Inline Doc improvements in wp-includes/general-template.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Inline Docs | Version: | 1.5 |
| Severity: | normal | Keywords: | has-patch commit |
| Cc: |
Description
Patch is a coding standards cleanup (whitespace) for wp-includes/general-template.php. Also adds some missing @return statements in PHPDocs.
Attachments (3)
Change History (9)
c3mdigital
— 9 months ago
comment:2
follow-up:
↓ 3
SergeyBiryukov
— 9 months ago
comment:3
in reply to:
↑ 2
c3mdigital
— 9 months ago
Replying to SergeyBiryukov:
http://wpdevel.wordpress.com/2011/03/23/code-refactoring/
There was a number of code cleanup tickets like #17997, and the consensus was that coding style fixes should be a result of more substantial changes than just improving readability.
Let's focus on PHPDocs here.
Thanks for the link. Makes a lot of sense. I updated the patch focusing only on the PHPDocs.
SergeyBiryukov
— 9 months ago
comment:4
SergeyBiryukov
— 9 months ago
21893.3.patch brings some consistency by reusing the pattern from *_title() functions:
http://core.trac.wordpress.org/browser/tags/3.4.2/wp-includes/general-template.php#L633
http://wpdevel.wordpress.com/2011/03/23/code-refactoring/
There was a number of code cleanup tickets like #17997, and the consensus was that coding style fixes should be a result of more substantial changes than just improving readability.
Let's focus on PHPDocs here.