Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#40455 closed defect (bug) (fixed)

Docs for set_site_transient() says max length of transient name is 40, it's actually 167

Reported by: jdgrimes's profile jdgrimes Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.8 Priority: normal
Severity: normal Version: 4.4
Component: Options, Meta APIs Keywords: good-first-bug
Focuses: docs Cc:

Description

In [28735] the docs were added, and in [34030] the length of the option_name column was lengthened. The docs for set_transient() were updated in [34045], but set_site_transient() was missed.

The length of the option_name column was updated to 191, but 24 characters are taken up by prefixing the transient name with _site_transient_timeout_ when the timeout is saved. 191 - 24 = 167.

Attachments (1)

40455.diff (713 bytes) - added by lukecavanagh 8 years ago.
Basic patch

Download all attachments as: .zip

Change History (4)

#1 @jdgrimes
8 years ago

  • Keywords good-first-bug added

@lukecavanagh
8 years ago

Basic patch

#2 @SergeyBiryukov
8 years ago

  • Milestone changed from Awaiting Review to 4.8

#3 @SergeyBiryukov
8 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 40432:

Docs: Update the length limit in set_site_transient() docs after [34030].

Props lukecavanagh, jdgrimes.
Fixes #40455.

Note: See TracTickets for help on using tickets.