Opened 13 years ago
Closed 9 years ago
#24603 closed enhancement (fixed)
Introduce pre_move_uploaded_file filter to allow pre-processing of uploaded files
| Reported by: | ryan | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.9 |
| Component: | Upload | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
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
@
10 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.
10 years ago
#5
follow-up:
↓ 6
@
10 years ago
- Keywords dev-feedback removed
@Mte90 Can you add a docblock for the new filter please?
#6
in reply to: ↑ 5
@
10 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.
9 years ago
#8
@
9 years ago
- Milestone Awaiting Review → 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
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
refreshed patch