[wp-trac] [WordPress Trac] #13034: Add class-wp-importer.php to wp-admin/includes directory

WordPress Trac wp-trac at lists.automattic.com
Fri Apr 16 21:27:00 UTC 2010


#13034: Add class-wp-importer.php to wp-admin/includes directory
---------------------------+------------------------------------------------
 Reporter:  briancolinger  |       Owner:  ryan     
     Type:  enhancement    |      Status:  assigned 
 Priority:  normal         |   Milestone:  3.1      
Component:  Import         |     Version:  3.0      
 Severity:  normal         |    Keywords:  has-patch
---------------------------+------------------------------------------------

Comment(by nacin):

 Random thoughts: min_whitespace and stop_the_insanity are both global
 scope. Should they be? If so, they should get better names and should
 probably be moved out of admin-only includes. I'd think they can be class
 methods however. Also, min_whitespace does not appear to be UTF-8
 friendly, versus {{{[\r\n\t ]}}}.

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


More information about the wp-trac mailing list