[wp-trac] [WordPress Trac] #21866: Remove AtomPub from core

WordPress Trac wp-trac at lists.automattic.com
Tue Sep 11 16:28:48 UTC 2012


#21866: Remove AtomPub from core
--------------------------------------------------+------------------
 Reporter:  nacin                                 |       Owner:
     Type:  task (blessed)                        |      Status:  new
 Priority:  normal                                |   Milestone:  3.5
Component:  AtomPub                               |     Version:  2.3
 Severity:  normal                                |  Resolution:
 Keywords:  has-patch dev-feedback needs-testing  |
--------------------------------------------------+------------------

Comment (by nacin):

 [attachment:atom.diff] looks good. I'll want to study the class-wp.php
 changes carefully — I got the gist, but could you explain them to the
 class? (Random note - no need to check isset() in order to unset().)

 We can probably ditch log_app(), actually, as it's only available inside
 wp-app.php. The plugin will need to define it. Good use of pluggable-
 deprecated.php for the old wp_atom_server class.

 You can go ahead and leave the deletion of wp-app.php and class-wp-atom-
 server.php out of future patches, to make them easier to review.

 I too considered `isset(
 $GLOBALS['wp_settings_fields']['writing']['remote_publishing'] )`. Though
 I think we can just lump it in next to `do_settings_fields('writing',
 'default');`.

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


More information about the wp-trac mailing list