The xmlrpc() wrapper is not welcome...

Damien Tournoud - December 31, 2008 - 12:08
Project:Drupal
Version:7.x-dev
Component:base system
Category:task
Priority:normal
Assigned:Unassigned
Status:needs work
Issue tags:Needs Documentation
Description

... now that we have a clean autoloader :)

AttachmentSize
remove-xmlrpc-wrapper.patch4.37 KB
Testbed results
remove-xmlrpc-wrapper.patchfailedFailed: Failed to install HEAD. Detailed results

#1

Dave Reid - January 4, 2009 - 07:37

It's worth noting that in the patch, a call to drupal_function_exists('xmlrpc'); is required to use the function.

#2

System Message - January 8, 2009 - 00:50
Status:needs review» needs work

The last submitted patch failed testing.

#3

Damien Tournoud - May 2, 2009 - 11:39
Status:needs work» needs review

Rerolled!

AttachmentSize
352872-remove-xmlrpc-wrapper.patch 4.34 KB
Testbed results
352872-remove-xmlrpc-wrapper.patchpassedPassed: 11162 passes, 0 fails, 0 exceptions Detailed results

#4

sun - May 3, 2009 - 20:30
Status:needs review» reviewed & tested by the community

Very nice clean-up. Testbot passes, so we're good to go.

#5

Dries - May 4, 2009 - 10:38
Status:reviewed & tested by the community» fixed

Committed to CVS HEAD. Thanks.

#6

Damien Tournoud - May 4, 2009 - 10:58
Status:fixed» needs work

Needs upgrade documentation.

 
 

Drupal is a registered trademark of Dries Buytaert.