.install file: Cannot modify header information

ryan_courtnage - December 9, 2008 - 15:30
Project:pgapi - Payment Gateway API
Version:6.x-1.2-beta4
Component:Code
Category:bug report
Priority:normal
Assigned:gor
Status:closed
Description

The .install files for pgapi and it's sub-modules contain blank lines after the closing "?>" php tag. This results in php errors like:

Cannot modify header information - headers already sent by (output started at /home/rcourtna/workspace/drupal/sites/all/modules/pgapi/pgapi.install:52) in /home/rcourtna/workspace/drupal/includes/common.inc on line 319.

(See "PHP Code Tags" section of http://drupal.org/coding-standards).

The attached patch removes the closing php tags from all .install files.

AttachmentSize
pgapi_closephp.patch1.86 KB

#1

mjvs@groups.dru... - December 21, 2008 - 16:27
Status:needs review» reviewed & tested by the community

Yep, patch looks good.

#2

gor - November 25, 2009 - 20:00
Version:6.x-1.x-dev» 6.x-1.2-beta4
Assigned to:Anonymous» gor
Status:reviewed & tested by the community» fixed

#3

gor - November 25, 2009 - 22:20
Status:fixed» closed
 
 

Drupal is a registered trademark of Dries Buytaert.