Opened 14 years ago
Closed 14 years ago
#20311 closed enhancement (invalid)
Docblock error for wp_unique_filename
| Reported by: | transom | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Inline Docs | Version: | 3.4 |
| Severity: | trivial | Keywords: | has-patch |
| Cc: | Focuses: |
Description
The current docblock for wp_unique_filename inaccurately describes the third parameter passed to this function
Attachments (2)
Change History (8)
#1
@
14 years ago
- Component General → Inline Docs
- Keywords needs-refresh added
A note: patches should be made from root.
#2
@
14 years ago
- Cc added
- Keywords needs-refresh removed
- Severity normal → trivial
Amended transom's initial patch with the right format :)
#6
@
14 years ago
- Keywords commit removed
- Milestone 3.5
- Resolution → invalid
- Status new → closed
The documentation itself appears wrong.
* The callback is passed three parameters, the first one is the directory, the * second is the filename, and the third is an optional supplied callback * to generate a unique filename.
This paragraph is not talking about the function, whose third argument is indeed the optional supplied callback. This is talking about how the supplied callback gets called. As you can see, the third argument is indeed the extension:
call_user_func( $unique_filename_callback, $dir, $name, $ext );
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Update docblock for wp_unique_filename