Opened 11 years ago
Closed 11 years ago
#34234 closed task (blessed) (fixed)
Rename `id` param in get_the_permalink, get_permalink, and the_permalink
| Reported by: | chriscct7 | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.4 |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | template |
Description
The id param in get_the_permalink, get_permalink, and the_permalink isn't very intuitive. id is not a really good parameter name, partially because it doesn't actually describe what it is. Is it a post id? A user id? A random number called id? A serial number for a bikeshed? A MySQL column that autoincrements value?
If the id param was renamed to post it'd make the functions more self-documenting, and would require a minimal effort patch, making the functions more easily understood for newer developers by saving them the time of figuring out what exactly id is.
Attachments (1)
Change History (5)
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
In 35001: