A possible fix for #46

[mapio]

This seems to fix the issue. Not sure it is the right way to go, though…

[rozza]

I still think that flask-admin shouldnt use text area for binary - probably a file upload instead.

[mapio]

This is a good point… I'll see if I'll be able to implement it and submit a pull-request.

[rozza]

Will merge if you can add a test case (so we dont regress) and use txt_type(valuelist[0]) instead :) 👍

[mapio]

I'm surely missing something: BinaryField does not appear at all in flask-admin code…

The "translation" between (mongo) BinaryField and (wtf) TextArea seems to happen in

flask-mongoengine/flask_mongoengine/wtf/orm.py

Line 141 in 2f49872

@converts('BinaryField')

[mapio]

I believe it should be bin_type, not txt_type… am I wrong?

[rozza]

Re: Flask-admin I dont think it uses flask-mongoengine under the hood - so #46 will have to be fixed in their repo as well.

yes binary_type probably best :)

[mapio]

Test added (if you remove the wrapping with bin_type it fails).

And I don't think that flask-admin needs any fix in their repo: the testcase in #46 is based on flask-admin and works once this fix is applied (without touching flask-admin code).