#23619 closed defect (bug) (fixed)
Twenty Thirteen: use new URL extraction function for link post format
Reported by: | lancewillett | Owned by: | lancewillett |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Bundled Theme | Keywords: | |
Focuses: | Cc: |
Description
Replace twentythirteen_get_first_url()
URL grabber function in Twenty Thirteen with the new core version.
See #23570 for the suggested new function(s).
Note: Twenty Thirteen does not need back compat with pre 3.6 versions
Attachments (1)
Change History (12)
#7
@
12 years ago
- Keywords needs-testing added; has-patch removed
Lets keep this ticket open to test all the scenarios listed by Frank Klein above.
#8
@
12 years ago
Looks good. I wonder if the function name is still the best since the function no longer looks for the first url. Couldn't come up with anything better than twentythirteen_get_link_url()
, though.
#9
@
12 years ago
- Keywords needs-testing removed
twentythirteen_get_link_url()
is a better name, thanks.
Also noting, we should not allow this function to be pluggable, per same idea of #23674.
Note: See
TracTickets for help on using
tickets.
Here are the different scenarios with the output they produce: