Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#47399 closed enhancement (wontfix)

Docs: Need to correct the inline documentation for _encode()

Reported by: priyankkpatel Owned by:
Priority: normal Milestone:
Component: External Libraries Version: 5.3
Severity: normal Keywords: has-patch
Cc: Focuses: coding-standards

Description

According to the comments for _encode() it can return JSON format of input variable or error. While it can also return boolean, NULL, int, double, float, string, JSON, array.

Attachments (1)

47399.diff (1.1 KB ) - added by priyankkpatel 7 years ago.
Patch for wp-includes/classjson

Download all attachments as: .zip

Change History (2)

@priyankkpatel
7 years ago

Patch for wp-includes/classjson

#1 @SergeyBiryukov
7 years ago

  • Component CommentsExternal Libraries
  • Milestone Awaiting Review
  • Resolutionwontfix
  • Status newclosed

Hi @priyankkpatel, welcome to WordPress Trac!

Thanks for the patch, but the current documentation is correct, all of these cases are a JSON representation of the provided input. As noted in #47387, the file is a third-party library which we usually don’t modify.

Note: See TracTickets for help on using tickets.