WordPress.org
Get WordPress

Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#4607 closed enhancement (fixed)

ABSPATH already has a trailing slash

Reported by: nazgul's profile Nazgul Owned by: nazgul's profile Nazgul
Milestone: 2.3 Priority: low
Severity: normal Version: 2.3
Component: Optimization Keywords: has-patch commit
Focuses: Cc:

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 17 years ago.

Download all attachments as: .zip

Change History (3)

@Nazgul
17 years ago

#1 @Nazgul
17 years ago

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

#2 @markjaquith
17 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.