#20042 closed enhancement (fixed)
Move AtomServer class to class-atomserver.php
| Reported by: | ryan | Owned by: | ryan |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.4 |
| Component: | AtomPub | Version: | |
| Severity: | normal | Keywords: | has-patch dev-feedback |
| Cc: | Focuses: |
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
@
15 years ago
- Cc 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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)