.install file: Cannot modify header information

ryan_courtnage - December 9, 2008 - 15:30
Project:pgapi - Payment Gateway API
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:reviewed & tested by the community
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.

 
 

Drupal is a registered trademark of Dries Buytaert.