Raw Image gets unicodedecodeerror on logging debug data

[Odin Hørthe Omdal (Velmont)]

Forgot to encode to utf-8 first.

Have patch.

[Christopher Allan Webber]

Hm. I think this may be backwards! Encoding to utf-8 means encoding to a bytestring, which it looks like yeah, the parent is also a bytestring in python2, so not surprising this is happening.

However, I think it might be better if we did the reverse... instead,

'Wrote new file from {0} to preview (jpg) {1}' should become u'Wrote new file from {0} to preview (jpg) {1}'

In that case, I don't think you need to encode (if anything, maybe decode, but you shouldn't decode something if it's already unicode, which it looks like this probably is?)

Could you test that and see if it works? If so, I'd be happy to merge this in.

[Odin Hørthe Omdal (Velmont)]

lol, true.

I am in Python 3 land myself, unicode is simpler there :) Of course you're very right. I'll change, test and upload. :)

[Odin Hørthe Omdal (Velmont)]

Don't know how to delete the old patch, -- since I changed the name on this new one.

[ShawnRisk]

This should always be put in review if there is patch added to the ticket and this needs to be checked by others.

[Odin Hørthe Omdal (Velmont)]

This really is more or less reviewed, since I did just what Chris said :)

Also have the commit on-web (not in its own branch, but on my own patch-list that I rebase on top of upstream/master):

​https://github.com/Velmont/mediagoblin/commit/6907027ec1a9d24d9dd7fc8c1816782ab9cc00a2

[Christopher Allan Webber]

Merged, rebased, pushed. Thank you!