Re: [SM-DEVEL] php avelsieve prob

[Paul Lesniewski]

Patrick Welche wrote:

> Do any of you php gurus know what to do about:
>
> Array ( [type] => all )
> Warning: Cannot modify header information - headers already sent by (output started at /usr/local/share/httpd/htdocs/squirrelmail.new/plugins/avelsieve/include/process_user_input.inc.php:53) in /usr/local/share/httpd/htdocs/squirrelmail.new/plugins/avelsieve/edit.php on line 212
>
> ?
>
> This is when setting a vacation message i.e., match ALL messages.
>
> (Only noteworthy thing I can think of is:
> $avelsieve_oldcyrus = true;
> $avelsieve_enable_envelope_auth = false;
> as I'm running cyrus 2.2.2-beta (and php 5))

And why, may I ask, is this on the devel list?  There is a plugins list.
 That's what it is for.


-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems?  Stop!  Download the new AJAX search engine that makes
searching your log files as easy as surfing the  web.  DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
--
squirrelmail-devel mailing list
Posting Guidelines: http://www.squirrelmail.org/wiki/MailingListPostingGuidelines
List Address: squirrelmail-devel@...
List Archives: http://news.gmane.org/thread.php?group=gmane.mail.squirrelmail.devel
List Archives: http://sourceforge.net/mailarchive/forum.php?forum_id=7139
List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-devel

[Patrick Welche]

On Fri, Dec 02, 2005 at 10:33:44AM -0800, Paul Lesneiwski wrote:

>
>
> Patrick Welche wrote:
> > Do any of you php gurus know what to do about:
> >
> > Array ( [type] => all )
> > Warning: Cannot modify header information - headers already sent by (output started at /usr/local/share/httpd/htdocs/squirrelmail.new/plugins/avelsieve/include/process_user_input.inc.php:53) in /usr/local/share/httpd/htdocs/squirrelmail.new/plugins/avelsieve/edit.php on line 212
> >
> > ?
> >
> > This is when setting a vacation message i.e., match ALL messages.
> >
> > (Only noteworthy thing I can think of is:
> > $avelsieve_oldcyrus = true;
> > $avelsieve_enable_envelope_auth = false;
> > as I'm running cyrus 2.2.2-beta (and php 5))
>
> And why, may I ask, is this on the devel list?  There is a plugins list.
>  That's what it is for.

I kind soul told me off-list that there is probably some print_r debugging
left-over, and sure enough, the attached fixes it.

Cheers,

Patrick

Index: process_user_input.inc.php
===================================================================
RCS file: /squirrelmail/avelsieve/include/process_user_input.inc.php,v
retrieving revision 1.17
diff -u -r1.17 process_user_input.inc.php
--- process_user_input.inc.php 1 Nov 2005 17:29:28 -0000 1.17
+++ process_user_input.inc.php 2 Dec 2005 18:59:07 -0000
@@ -50,7 +50,7 @@
  $match_vars = array('headermatch', 'addressmatch', 'envelopematch', 'sizeamount', 'bodymatch');
  $new_cond_indexes = array();
  foreach($ns['cond'] as $n => $c) {
- print_r($c);
+ /* print_r($c); */
  foreach($match_vars as $m) {
  if(!empty($c[$m]) || $c['type'] == 'all') {
  $new_cond_indexes[] = $n;