PATCH MM_S3 TEST: Cannot modify header information - headers already sent

InternetPro - August 26, 2009 - 06:41
Project:Media Mover
Version:6.x-1.0-beta2
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I was testing Amazon S3 in version 6--1-0-BETA2

I received the following error:
Cannot modify header information - headers already sent by (output started at /var/www/vhosts/ezydex.com/sites/default/modules/media_mover/contrib/mm_s3/mm_s3_test.inc:1) in /var/www/vhosts/ezydex.com/includes/common.inc on line 328.

This was caused by a space character on line 1 of mm_s3_test.inc before <?php

When I corrected this issue, there was a syntax error (spelling) for the function drupal_set_message in 2 locations.

I have corrected these issues in the attached patch, which I diff'd from the modules/media_mover directory

Please review, and comment on my changes.

Cheers!

AttachmentSize
mm_s3_test.inc-1.patch749 bytes

#1

arthurf - September 15, 2009 - 04:42

Wow. Can't believe I let that one slide. Committed this to 6.1x and 6.2x Thanks!

 
 

Drupal is a registered trademark of Dries Buytaert.