Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 15 years ago

#10106 closed defect (bug) (invalid)

wp-config-sample.php missing closing PHP tag

Reported by: sojweb's profile sojweb Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: Cc:

Description

The config sample file included with 2.8 is missing the closing PHP tag. Someone just needs to stick it back in.

Attachments (1)

wp-config-sample.php.2.diff (328 bytes) - added by sojweb 16 years ago.
wp-config-sample.php with closing PHP tag

Download all attachments as: .zip

Change History (6)

@sojweb
16 years ago

wp-config-sample.php with closing PHP tag

#1 @filosofo
16 years ago

  • Resolution set to invalid
  • Status changed from new to closed
  • Version 2.8 deleted

This is a feature, not a bug.

First, closing PHP tags are optional.

More importantly, since this is a user-edited file, it's highly likely that someone's text editor will add a line break after the closing tag, causing error messages that aren't immediately traceable. Leaving off the closing tag avoids that headache.

#2 @filosofo
16 years ago

By user-edited, I mean that people copy wp-config-sample.php and edit it as wp-config.php

#3 @demetris
16 years ago

Maybe it would be a good idea to put a comment it its place, so that people don’t think that something is missing or that the file is incomplete? Something like:

//  END OF FILE

Just a thought...

#4 @Denis-de-Bernardy
16 years ago

  • Milestone Unassigned deleted

#5 @sojweb
15 years ago

I'll be damned. Learn something every day. Thanks.

Note: See TracTickets for help on using tickets.