[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