Opened 12 years ago
Closed 8 years ago
#24603 closed enhancement (fixed)
Introduce pre_move_uploaded_file filter to allow pre-processing of uploaded files
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.9 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Upload | Keywords: | has-patch |
| Focuses: | Cc: |
Description
This is handy for use in file replication plugins. We've been using this on wordpress.com for awhile for just that purpose.
Attachments (5)
Change History (14)
#2
@
9 years ago
- Keywords dev-feedback added; needs-refresh removed
Ignore the 24603.patch this is a wrong testing patch, the real refreshed patch is 24603.2.patch.
This ticket was mentioned in Slack in #docs by morganestes. View the logs.
9 years ago
#5
follow-up:
↓ 6
@
9 years ago
- Keywords dev-feedback removed
@Mte90 Can you add a docblock for the new filter please?
#6
in reply to:
↑ 5
@
9 years ago
- Keywords needs-docs removed
Replying to johnbillion:
@Mte90 Can you add a docblock for the new filter please?
Done, I don't know if I written in the best way.
This ticket was mentioned in Slack in #core by drew. View the logs.
8 years ago
#8
@
8 years ago
- Milestone changed from Awaiting Review to 4.9
@Mte90 Updated patch looks pretty good. There's just a couple things I wanted to note that I'll go ahead and fix on commit:
- There should be a space between the DocBlock summary and description
- In the case of "real" values like
nullwe typically assign "pseudo-variables" in the DocBlock so as to best illustrate what the value to represents. In this case, we'll probably just reuse$move_new_file
Note: See
TracTickets for help on using
tickets.
refreshed patch