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

CommentFileSizeAuthor
#3 352872-remove-xmlrpc-wrapper.patch4.34 KBdamien tournoud
remove-xmlrpc-wrapper.patch4.37 KBdamien tournoud

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

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

damien tournoud commented
Status: Needs work » Needs review
StatusFileSize
new 352872-remove-xmlrpc-wrapper.patch4.34 KB

Rerolled!

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Reviewed & tested by the community

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

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

damien tournoud’s picture

damien tournoud commented
Status: Fixed » Needs work
Issue tags: +Needs documentation

Needs upgrade documentation.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon

Adding this to the module upgrade guide...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
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

damien tournoud commented
Status: Needs review » Fixed

Looks good.

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

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

Issue tags: +Needs documentation

Restoring issue tags, see #2125755: System messages removed all issue tags during D7 upgrade.