Opened 14 years ago
Closed 13 years ago
#23674 closed defect (bug) (fixed)
Twenty Eleven: twentyeleven_url_grabber() should not be pluggable
| Reported by: | nacin | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.6 |
| Component: | Bundled Theme | Version: | 3.6 |
| Severity: | normal | Keywords: | |
| Cc: | Focuses: |
Description
Looking at [23474]. In the past, we've had two types of functions for default themes:
- It's a function that gets used in a theme. Should be pluggable.
- It's a function that gets used as a hook callback. No need to be pluggable.
However, twentyeleven_url_grabber() is a utility function. It wasn't an oversight, rather, I don't think there is a valid reason for it to be pluggable. Any change in the regular expression could expose a security vulnerability. A function to extract items from content could certainly make for a good core function, and there are tickets for that. But allowing this function to be pluggable doesn't help much.
Change History (2)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Agreed on all points. Once #23618 is closed we can avoid the issue entirely by deleting the entire function from the theme.