Support 'atom_ns' 'atom_head' & 'atom_entry' hooks in Atompub backend

[znarfor]

wp-app.php is not extensible at the moment and by simply adding this hooks, we may be able to expose more content in it through plugins.

Alternatively, we may use dedicated hooks like 'app_ns' 'app_head' and 'app_entry'.

commit, wontfix?

Replying to Denis-de-Bernardy:

commit, wontfix?

Waiting for feedback from joseph.

I need to read up and make sure we're solid with the proper formatting/syntax of namespaces in XML/AtomPub (looks pretty simple, just want to make sure).

I'm also trying to think through the best way to do this. This might be better done as a filter to make it less likely that a mis-behaving plugin would mess up AtomPub responses.

Replying to josephscott:

I need to read up and make sure we're solid with the proper formatting/syntax of namespaces in XML/AtomPub (looks pretty simple, just want to make sure).

I'm also trying to think through the best way to do this. This might be better done as a filter to make it less likely that a mis-behaving plugin would mess up AtomPub responses.

A filter for namespaces sounds easier and safer for a plugin to use.

(Also if we add atom_head we should move the generator call into a filter on it.)

Moving to 2.9 for now as I don't want to change APP now while we don't have a concrete solution.

It's a bit surprising to say using a filter is necessary when the exactly same hook is already existing in feed-atom.php, same for generator.

See yourself: http://core.trac.wordpress.org/browser/trunk/wp-includes/feed-atom.php

I think the refactoring you propose have few chances to be a priority, and then will likely be delayed again and again. Let be pragmatic ?

Can you please review your decision and consider integrating this hooks in WP 2.8 ?

punting per IRC discussion

Can you please excerpt the discussion or link to the relevant IRC logs ?

Can you please excerpt the discussion or link to the relevant IRC logs ?

A target date was set for 2.8, Remaining enhancements and non-critical bugs are being punted to 2.9 (or 2.8.1 if decided to bring it back to the 2.8 series, which is done after its applied to 2.9)

 https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2009-06-03&sort=asc ~5pm onwards.

closed #5195 as dup

Would be nice to integrate this simple and straightforward patch in 2.9.

The patch still applies on -trunk and looks okay.

Do we want these to be the same as the ones in the atom feed or different - my feeling is maybe different so as to give better control for plugins?

As the submitter of this patch, i have nothing against different names for this hooks. Say 'atompub_head' or 'app_head' instead of 'atom_head'.

I didn't catch that the action names were the same as the atom feed, I think they should definitely be different.

(In [12488]) Add some hooks to wp-app.php to allow a plugin to extend the data returned. Fixes #8826 props znarfor.