[wp-trac] Re: [WordPress Trac] #5678: Respectfully strip newlines in some importers

WordPress Trac wp-trac at lists.automattic.com
Wed Jan 16 23:46:32 GMT 2008


#5678: Respectfully strip newlines in some importers
------------------------------+---------------------------------------------
 Reporter:  jdub              |        Owner:  hansengel
     Type:  enhancement       |       Status:  assigned 
 Priority:  normal            |    Milestone:  2.5      
Component:  General           |      Version:  2.5      
 Severity:  normal            |   Resolution:           
 Keywords:  has-patch tested  |  
------------------------------+---------------------------------------------
Comment (by mdawaffe):

 This seems like a good idea, but the function looks overly complicated.
 {{{$skip}}} and {{{$next}}} are never actually used.

 I'd also appreciate a comment for the function saying that it's intended
 for fully marked up, valid HTML only.  The function can't be trusted to
 strip lines from unencoded text, pre-wpautop half HTML/half text, or
 invalid HTML.  That's fine, it should just be noted.

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


More information about the wp-trac mailing list