WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#33753 closed defect (bug) (fixed)

Fix documentation of wp_check_browser_version()

Reported by: extendwings Owned by: SergeyBiryukov
Priority: normal Milestone: 4.4
Component: General Version: 3.2
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

It seems the documentation of wp_check_browser_version()'s return value is wrong.

Attachments (1)

33753.diff​ (837 bytes ) - added by extendwings 11 years ago.

Download all attachments as: .zip

Change History (3)

@extendwings
11 years ago

#1 @swissspidy
11 years ago

  • Focuses administration removed
  • Keywords has-patch added
  • Milestone Awaiting Review → 4.4
  • Version trunk → 3.2

Just tested this with some API requests and can confirm this.

Surprisingly the API is kinda broken/outdated, returning 9 as the current_version of Internet Explorer for example. However, this value is never used in core.

#2 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 33932:

Correct description for version and current_version parameters in wp_check_browser_version() response.

Props extendwings.
Fixes #33753.

Note: See TracTickets for help on using tickets.