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

CommentFileSizeAuthor
#3 352872-remove-xmlrpc-wrapper.patch4.34 KBDamien Tournoud
Passed: 11162 passes, 0 fails, 0 exceptions View
remove-xmlrpc-wrapper.patch4.37 KBDamien Tournoud
Failed: Failed to install HEAD. View
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

Dave Reid’s picture

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

Status: Needs review » Needs work

The last submitted patch failed testing.

Damien Tournoud’s picture

Status: Needs work » Needs review
FileSize
4.34 KB
Passed: 11162 passes, 0 fails, 0 exceptions View

Rerolled!

sun’s picture

Status: Needs review » Reviewed & tested by the community

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

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Damien Tournoud’s picture

Status: Fixed » Needs work
Issue tags: +Needs Documentation

Needs upgrade documentation.

jhodgdon’s picture

Assigned: Unassigned » jhodgdon

Adding this to the module upgrade guide...

jhodgdon’s picture

Assigned: jhodgdon » Unassigned
Status: Needs work » Needs review

I've added this to the module upgrade guide (and categorical page):
http://drupal.org/node/224333#xmlrpc-wrapper

A review would be nice. Then please mark fixed if the text there is OK.

Damien Tournoud’s picture

Status: Needs review » Fixed

Looks good.

Status: Fixed » Closed (fixed)
Issue tags: -Needs Documentation

Automatically closed -- issue fixed for 2 weeks with no activity.