[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