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)

Files: 
CommentFileSizeAuthor
#3 metaweblog_upload.patch659 bytesmeba
Failed: 8503 passes, 1 fail, 0 exceptions View
#2 metaweblog_upload_d6.patch667 bytesmeba
#2 metaweblog_upload_d5.patch899 bytesmeba
#1 metaweblog_upload.patch659 bytesmeba
Unable to apply patch metaweblog_upload.patch View
blogapi.module.base64.patch479 bytesacrollet
Failed: 8503 passes, 1 fail, 0 exceptions View

Comments

meba’s picture

Version: 6.1 » 7.x-dev
FileSize
659 bytes
Unable to apply patch metaweblog_upload.patch View

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

Rolling against DRUPAL-6 and DRUPAL-5 CVS.

meba’s picture

FileSize
659 bytes
Failed: 8503 passes, 1 fail, 0 exceptions View

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

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

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

Component: Code » Blogging application integration
DamienMcKenna’s picture

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.