Context Navigation

← Previous Ticket
Next Ticket →

#64084 closed defect (bug) (fixed)

Fix incorrect @global variable name in wp-admin/revision.php

Reported by:	hanimbarek	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:	6.9
Component:	Revisions	Version:	3.7
Severity:	trivial	Keywords:	has-patch
Cc:		Focuses:	docs, administration

Description

The @global docblock in wp-admin/revision.php incorrectly uses $revision instead of $revision_id.

This: @global int $revision (src/wp-admin/revision.php:18)
Should be: @global int $revision_id

Change History (5)

This ticket was mentioned in PR #10178 on WordPress/wordpress-develop by @hanimbarek.

10 months ago #1

Keywords has-patch added

This PR fixes a minor documentation issue in wp-admin/revision.php.

The @global docblock incorrectly referenced $revision instead of $revision_id.
This change updates the variable name to match the actual code used below.

#2 @SergeyBiryukov
10 months ago

Component Administration → Revisions
Focuses administration added
Milestone Awaiting Review → 6.9
Version 6.8.3 → 3.7

#3 @SergeyBiryukov
10 months ago

Owner set to SergeyBiryukov
Resolution → fixed
Status new → closed

In 60912:

Docs: Correct $revision_id global reference in wp-admin/revision.php.

Follow-up to [25419], [42795].

Props hanimbarek.
Fixes #64084.

#4 follow-up: ↓ 5 @SergeyBiryukov
10 months ago

Hi there, welcome to WordPress Trac! Thanks for the ticket and the PR :)

#5 in reply to: ↑ 4 @hanimbarek
10 months ago

Thanks! Happy to contribute.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#64084 closed defect (bug) (fixed)

Fix incorrect @global variable name in wp-admin/revision.php

Description

Change History (5)

This ticket was mentioned in ​PR #10178 on ​WordPress/wordpress-develop by ​@hanimbarek. 10 months ago #1

#2 @SergeyBiryukov 10 months ago

#3 @SergeyBiryukov 10 months ago

#4 follow-up: ↓ 5 @SergeyBiryukov 10 months ago

#5 in reply to: ↑ 4 @hanimbarek 10 months ago