Hi there,

according to http://www.xmlrpc.com/metaWeblogApi, files should be uploaded in base64 encoded format. However, the blogapi module does not decode base64 files. I'm submitting this patch for review. Works for me using the python xml-rpc lib and drupal 6.1. (also tested with ecto)

CommentFileSizeAuthor
#3 metaweblog_upload.patch659 bytesmeba
#2 metaweblog_upload_d6.patch667 bytesmeba
#2 metaweblog_upload_d5.patch899 bytesmeba
#1 metaweblog_upload.patch659 bytesmeba
blogapi.module.base64.patch479 bytesacrollet

Comments

meba’s picture

meba commented
Version: 6.1 » 7.x-dev
StatusFileSize
new metaweblog_upload.patch659 bytes

Patches must be committed to HEAD first.

Simple and straightforward patch, tested and looks good. Rerolling against D7.

(Should be fixed in D5 as it's still supported. And the patch works there too.)

meba’s picture

meba commented
StatusFileSize
new metaweblog_upload_d5.patch899 bytes
new metaweblog_upload_d6.patch667 bytes

Rolling against DRUPAL-6 and DRUPAL-5 CVS.

meba’s picture

meba commented
StatusFileSize
new metaweblog_upload.patch659 bytes

I guess that the last two files might be noticed by testbot now and marked as FAILED against head so uploading D7 patch as last one again...

Status: Needs review » Needs work

The last submitted patch failed testing.

lilou’s picture

lilou commented
Status: Needs work » Closed (won't fix)

Blog API is removed : #537434: Drop the blog API from core
So this won't be necessary anymore.

acrollet’s picture

acrollet commented
Project: Drupal core » BlogAPI Alternative (obsolete)
Version: 7.x-dev » 6.x-0.1
Component: blogapi.module » Code
Status: Closed (won't fix) » Active

Moving this over to blogapi_new, since it seems that it won't get fixed in 6.x core :( - Damien, do you have any interest in applying this patch? The original patch applies and works, (with a little fuzz) I can re-roll against HEAD if you like.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Component: Code » Blogging application integration
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

This project is now obsolete, because someone new stepped up to maintain the official D7 version. Please give that module a try instead. Thank you.