WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#33753 closed defect (bug) (fixed)

Fix documentation of wp_check_browser_version()

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

Description

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

Attachments (1)

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

Download all attachments as: .zip

Change History (3)

@extendwings
10 years ago

#1 @swissspidy
10 years ago

  • Focuses administration removed
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to 4.4
  • Version changed from trunk to 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
10 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to 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.