Opened 10 years ago
Closed 10 years ago
#32173 closed enhancement (fixed)
Add support for `i` parameter in Press This
Reported by: | stephdau | Owned by: | azaozz |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | 4.2 |
Component: | Press This | Keywords: | has-patch |
Focuses: | Cc: |
Description
I was browsing the Codex page for Press This (https://codex.wordpress.org/Press_This#Technical_Note), and I noticed that there was a documented data param that wasn't replicated in the new version: i
, to pass an image URL.
The attached patch enables that parameter (GET
and POST
), and offers the image to the user for selection (no auto-embed).
Attachments (3)
Change History (11)
#2
@
10 years ago
It's definitely something that can be leveraged by 3rd-parties to integrate with PT at a basic level, and supposedly was there.
#3
@
10 years ago
Chatter about this feature and to include it or not: https://wordpress.slack.com/archives/core-pressthis/p1430255033001395
#5
@
10 years ago
- Milestone changed from Awaiting Review to 4.3
Perhaps we can add the extra image src at the bottom in merge_or_fetch_data()
, prepending it to the existing array so it is shown as first choice. Seems a bit simpler, see 32173.2.diff.
#6
follow-up:
↓ 7
@
10 years ago
It should be $_REQUEST['i']
rather than $_REQUEST('i')
.
Let's get this moving quickly, beta is only two weeks away.
Is this necessary?