Skip to:
Content

bbPress.org

Opened 5 months ago

Closed 2 weeks ago

Last modified 2 weeks ago

#3623 closed enhancement (fixed)

Document global $pagenow in src/includes/common/template.php file.

Reported by: upadalavipul's profile upadalavipul Owned by: johnjamesjacoby's profile johnjamesjacoby
Milestone: 2.6.14 Priority: normal
Severity: normal Version: 2.0
Component: General Keywords: commit
Cc:

Description

I have reviewed the below file code and found that we are using a global $pagenowe variable. but they do not mention it in the function comment. So I think we need to improve the function comment code.

Files:

src/includes/common/template.php

Attachments (1)

3623.patch (2.5 KB) - added by upadalavipul 5 months ago.
patch added

Download all attachments as: .zip

Change History (4)

@upadalavipul
5 months ago

patch added

#1 @johnjamesjacoby
2 weeks ago

  • Component changed from Site - bbPress.org to General
  • Keywords commit added
  • Milestone changed from Awaiting Review to 2.6.14
  • Owner set to johnjamesjacoby
  • Status changed from new to assigned
  • Type changed from defect (bug) to enhancement
  • Version changed from trunk to 2.0

#2 @johnjamesjacoby
2 weeks ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 7312:

Common: document a few global variable usages.

Props upadalavipul.

In trunk, for 2.7.

Fixes #3623.

#3 @johnjamesjacoby
2 weeks ago

In 7313:

Common: document a few global variable usages.

Props upadalavipul.

In branches/2.6, for 2.6.14.

Fixes #3623.

Note: See TracTickets for help on using tickets.