Opened 19 months ago
Closed 19 months ago
#62849 closed defect (bug) (fixed)
Rename the "setted_transient" action to "set_transient".
| Reported by: | peterwilsoncc | Owned by: | sukhendu2002 |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.8 |
| Component: | Options, Meta APIs | Version: | |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: |
Description (last modified by )
To quote the Merriam-Webster dictionary's page for the definition for setted:
The word you've entered isn't in the dictionary.
The correct work in the context of the action is "set", so I propose renaming the action to correct the misspelling.
This will also introduce consistency with the action name set_transient_{$transient}.
The incorrectly spelt action is used in some popular plugins, the two highest installations with 200K+ active installs, so that may be a blocker due to the amount of log entries it will create. It will not be a back-compat break as it would be covered by a do_action_deprecated() call.
Edit: setted_site_transient ought to be renamed too as part of this, see comment #3 -- @peterwilsoncc
Change History (7)
This ticket was mentioned in PR #8185 on WordPress/wordpress-develop by @sukhendu2002.
19 months ago
#2
- Keywords has-patch added; needs-patch removed
#3
follow-up:
↓ 4
@
19 months ago
- Milestone Awaiting Review → 6.8
- Owner set to
- Status new → assigned
It was pointed out to me (props @johnbillion) that site transients also contain a miss-spelling, setted_site_transient -- that action should be renamed too.
@sukhendu2002, are you able to push a commit to your PR renaming that too?
I've put this ticket on the 6.8 milestone for consideration.
#4
in reply to: ↑ 3
@
19 months ago
Replying to peterwilsoncc:
It was pointed out to me (props @johnbillion) that site transients also contain a miss-spelling,
setted_site_transient-- that action should be renamed too.
@sukhendu2002, are you able to push a commit to your PR renaming that too?
I've put this ticket on the 6.8 milestone for consideration.
Thanks for pointing that out! I've added a commit to rename setted_site_transient as well.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Trac ticket: https://core.trac.wordpress.org/ticket/62849