WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#20196 closed defect (bug) (invalid)

No PHP closure tag in index.php

Reported by: carldanley Owned by:
Milestone: Priority: normal
Severity: trivial Version: 3.4
Component: General Keywords: needs-patch
Focuses: Cc:

Description

small bug, thats all. also, this is my first patch. index.php is missing a php closure tag.

Attachments (1)

index.diff (286 bytes) - added by carldanley 3 years ago.
the fix for this.

Download all attachments as: .zip

Change History (4)

@carldanley3 years ago

the fix for this.

comment:1 @dd323 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

PHP files may optionally include the closing ?> tag. As of 3.4, we've removed them from all files: [19712] (I think the alot had a nice breakfast that night - #13237)

Last edited 3 years ago by dd32 (previous) (diff)

comment:2 @nacin3 years ago

Indeed, leaving closing PHP tags off files is valid, acceptable, and in some cases even preferred (see http://www.php.net/manual/en/language.basic-syntax.instruction-separation.php for a note). As dd32 points out, we recently altered our standard to omit them everywhere. But thanks for the patch!

comment:3 @carldanley3 years ago

No worries! It was more of a nervous submission for a first patch :) Trying to get involved more. Will continue to read more before submitting further patches. Glad this went pretty easily.

Note: See TracTickets for help on using tickets.