Opened 6 months ago
Closed 6 months ago
#22544 closed defect (bug) (fixed)
Insert Media: Non-image attachments aren't inserted with a link
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.5 |
| Component: | Media | Version: | 3.5 |
| Severity: | normal | Keywords: | has-patch commit dev-reviewed |
| Cc: |
Description
- Upload a PDF file
- Select Linkt to and use one of the options, for example Media File
- Click Insert into post
Only the title will be inserted, the link around the text is missing.
Attachments (1)
Change History (8)
comment:1
koopersmith — 6 months ago
koopersmith — 6 months ago
comment:2
koopersmith — 6 months ago
- Keywords has-patch added; needs-patch removed
Define $_POST['url'] for non-image attachments in wp.media.editor.send.attachment().
Looking at this, I think we should go back to the old field names used by media_upload_form_handler() — align, image_alt, image-size, post_excerpt, post_title. Here's a patch that does that while fixing both this ticket and #22553.
Sidenote: $_POST['attachment']['url'], not $_POST['url'].
An empty line is inserted with 22553.diff. 22544.diff works. (Chrome, Mac OS)
Note: See
TracTickets for help on using
tickets.

Confirmed.