[wp-trac] Re: [WordPress Trac] #4337: Blog by Email: No content is assumed

WordPress Trac wp-trac at lists.automattic.com
Fri Jun 15 07:52:56 GMT 2007


#4337: Blog by Email: No content is assumed
---------------------------+------------------------------------------------
 Reporter:  jottlieb       |        Owner:  anonymous   
     Type:  defect         |       Status:  new         
 Priority:  normal         |    Milestone:  2.4 (future)
Component:  General        |      Version:  2.2         
 Severity:  normal         |   Resolution:              
 Keywords:  blog by email  |  
---------------------------+------------------------------------------------
Comment (by cfactor):

 jottlieb: I ran into this myself when I was setting up email blogging.
 I'm using dovecot as my pop3 server, if that could mean anything.  I'm
 attaching a patch that solves it for me.  I have not tested it with other
 POP3 servers can test it, it would be great.

 The first change is what really fixes the problem.  For some reason the
 blank line between the headers and the body was matching the original
 /^\s+/, so I changed it to /^\s+\S+/ so it can be used to only match the
 overrun header lines.

 The second change is for something different.  Since the $line doesn't get
 called until after the while check is done, by the time the message ending
 period hits the while check, it had already been added by the previous
 iteration.  I tried adding the fgets line before the while loop and moving
 the existing one to after the current line is added to existing MsgArray,
 but for some reason, this times out.  And yet, the script does exactly
 what I tried in function uidl (though it appears that wordpress never
 calls it.)  So I just added the check where it checks for end of buffer.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/4337#comment:2>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list