major rework of wp-app.php

[eliast]

I have a done a ton of rework for the APP implementation. Most importantly, I have broken up the parser from the server.

There are many fixes as a result of the Atom Interop event at Google. But some of them include:

• Better support for XHTML
• More complete query results of entries in feed listings
• Converted tabs to spaces
• Added support for atom:published
• Categories support (both XML document and set/update categories)
• Improved Paging
• Fixed draft support
• Improved role/caps support
• Fixed atomsvc+xml content type
• Fixed Media Collection urls
• Tested with our own test suite:  http://svn.wp-plugins.org/wp-app/test

That's all for now.

Just briefly looking at the atomlib.php, it says it requires PHP 5... WP has a minimum requirement of PHP 4.2, I believe.

rob1n, php5 is a long story. Language-wise it doesn't need php5, but there are differences in XML parsing between 4.2 and 6 that are hard to ignore. This is something we need to decide, although, I have been coding with the lowest common denominator in mind. That said, it works on my dreamhost account with php4.4.2 I think. We can always disable APP support if there's not enough XML support to maintain a stable endpoint. However, that shouldn't stop us from shipping it.

This patch is unreadable because of the needless tab -> spaces changes. Please keep things to WP's coding standards, and try to keep patches as succinct as possible.

@matt,

I fixed the patch considerably to make it more readable. Please let me know if this is ready to be committed.

@rob1n,

The code has always been the same (so it works on php4). The only reason why I added php5 was because that's the parser I have tested and they do behave differently across versions depending on what you compiled php with. However, I'll work to make sure the XML parsing behavior is consistent across versions.

Sam Ruby will be reviewing and testing the patch attached here  http://intertwingly.net/blog/2007/08/03/WordPress-AtomPub-and-PHP5#c1186424251

Patch to bring wp-app.php up to conformance with final Atompub spec

Attached wp-app.php.diff that can be applied against the current version on the trunk. This should bring Atompub support up to the current and final release of the spec, but needs additional testing.

(In [5951]) Bring in atomlib 0.4 and update wp-app to use it. Props eliast and placey. fixes #4191

DD32 to wp-hackers: Might not be a bug, But are you sure you wanted '.*/wp-app.php' ? keeping in mind that str_replace only replaces literals, it doesnt support regular expresions.

Replying to foolswisdom:

DD32 to wp-hackers: Might not be a bug, But are you sure you wanted '.*/wp-app.php' ? keeping in mind that str_replace only replaces literals, it doesnt support regular expresions.

This was dealt with in #4887 and committed in [6004].