Opened 12 years ago
Closed 12 years ago
#21303 closed enhancement (fixed)
class-wp-customize-*.php files inline documentation
Reported by: | bananastalktome | Owned by: | |
---|---|---|---|
Milestone: | 3.5 | Priority: | normal |
Severity: | trivial | Version: | |
Component: | Inline Docs | Keywords: | has-patch |
Focuses: | Cc: |
Description
I took a first pass at the inline documentation for the WP_Customize_Control class. I am not sure whether this sort of patch is helpful or not, so I will wait for feedback before I potentially take on additional classes.
Thanks!
Attachments (3)
Change History (12)
#4
@
12 years ago
- Summary changed from First pass at WP_Customize_Control class inline documentation to WP_Customize_* class inline documentation
#5
@
12 years ago
- Summary changed from WP_Customize_* class inline documentation to class-wp-customize-*.php files inline documentation
#7
follow-up:
↓ 8
@
12 years ago
Are @access keywords really necessary when there's an explicit keyword present in the code?
#8
in reply to:
↑ 7
@
12 years ago
Replying to scribu:
Are @access keywords really necessary when there's an explicit keyword present in the code?
Good question. I don't see them documented anywhere on the phpdoc 2 site. I do know that @access private prevents documentation from being generated (unless a flag is set on generation), which is very helpful for hiding our many internal functions from phpdoc output. But it's definitely possible that it was removed on the basis of PHP5 visibility keywords.
That said, we should still use them especially in our pre-3.2 classes, where we might have a "private" variable or function that, for compatibility reasons, we can't mark with a visibility keyword. In these cases, I have avoided marking these functions as explicitly "public," and I've left should-be variables as "var".
Seems like you are using whitespaces to indent? Core does use tabs to indent lines.
Also inline docs are always good.