WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#17205 closed defect (bug) (fixed)

wp_templating_constants() PHPDoc is confusing

Reported by: kawauso Owned by:
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.1
Component: Template Keywords: has-patch
Focuses: Cc:

Description

The term 'Web Path' sounds like it ought to be the URL path, not an absolute filesystem path. I can't find this terminology anywhere else.

Let's just make it 'Path' and save people like me having to scratch their heads for a while?

Attachments (2)

17205.diff​ (692 bytes) - added by kawauso 4 years ago.
Web Path -> Path
17205.2.diff​ (714 bytes) - added by kawauso 4 years ago.
Web Path -> Filesystem path

Download all attachments as: .zip

Change History (4)

@kawauso — 4 years ago

Web Path -> Path

comment:1 @nacin — 4 years ago

  • Milestone changed from Awaiting Review to 3.2

For clarity we can/should probably include the word filesystem in there.

@kawauso — 4 years ago

Web Path -> Filesystem path

comment:2 @nacin — 4 years ago

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

(In [17745]) s/Web Path/Filesystem Path/ in inline documentation. props kawauso, fixes #17205.

Note: See TracTickets for help on using tickets.