#20042 closed enhancement (fixed)
Move AtomServer class to class-atomserver.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.4 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | AtomPub | Keywords: | has-patch dev-feedback |
| Focuses: | Cc: |
Description
Move the class to its own file, much as was done with the xmlrpc server class. wp-app.php would then be just the basic endpoint handling bits.
Attachments (2)
Change History (7)
#2
@
14 years ago
- Cc kpayne@… added
- Keywords has-patch added
20042.patch does the following:
- separates the atom server into a separate class
- removes the log_api calls per #20051
- updates the query filter per IRC chat with nacin, ryan
- replaces the
exitcalls withwp_diecalls for future unit tests - installs an override handler for
wp_die - allows a plugin to replace the atom class with a
wp_atom_server_classhook - whitespace cleanup
It passes the Gorilla test:
Gorilla is beginning testing on http://127.0.0.1/wordpress-trunk/wp-app.php...
testServiceDocumentExists with data set #0
------------------------------------------
[info] Service document found
[result] success
testCollectionsExist with data set #0
-------------------------------------
[info] Collections found:
Test Blog Posts accepts application/atom+xml;type=entry
Test Blog Media accepts image/*, audio/*, video/*
[result] success
testHasJPEGCollection with data set #0
--------------------------------------
[info] JPEG collections found:
Test Blog Media
[result] success
Gorilla is beginning testing on http://127.0.0.1/wordpress-trunk/wp-app.php...
testListPosts
-------------
[info] Post collections found:
Test Blog Posts
[info] Using Test Blog Posts at http://127.0.0.1/wordpress-trunk/wp-app.php/posts
[info] Checking current
[result] success
testSparsePost
--------------
[info] Testing basics
[info] Posting of new entry reported success, location: http://127.0.0.1/wordpress-trunk/wp-app.php/post/1848
[result] success
testPostFromTemplate with data set #0
-------------------------------------
[info] Creating from Gorilla/lib/tests/data/posts/example-1.xml
[info] Posting of new entry reported success, location: http://127.0.0.1/wordpress-trunk/wp-app.php/post/1851
[info] Examining the new entry as returned in the POST response
[info] Slug was used in server-generated URI
[info] Checking at the new location
[result] success
Note: See
TracTickets for help on using
tickets.
I like the idea of this.
If we are touch the APP code this much we should look at merging in #7652 and testing out Gorilla (https://github.com/rmccue/Gorilla) for testing it (As last I checked APE was too broken to test with)