WordPress.org

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#4607 closed enhancement (fixed)

ABSPATH already has a trailing slash

Reported by: Nazgul Owned by: Nazgul
Milestone: 2.3 Priority: low
Severity: normal Version: 2.3
Component: Optimization Keywords: has-patch commit
Focuses: Cc:
PR Number:

Description

The following (pseudo)code occurs a couple of times in the codebase.

include(ABSPATH . '/something.php')

But because ABSPATH by default already has a trailing slash, the extra slash is not needed.

I've made a patch which removes those extra slashes where needed.

Attachments (1)

4607.diff (5.2 KB) - added by Nazgul 12 years ago.

Download all attachments as: .zip

Change History (3)

@Nazgul
12 years ago

#1 @Nazgul
12 years ago

  • Keywords commit added
  • Owner changed from anonymous to Nazgul
  • Status changed from new to assigned

#2 @markjaquith
12 years ago

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

(In [5820]) ABSPATH already has a trailing slash. Props Nazgul. fixes #4607

Note: See TracTickets for help on using tickets.