collective / wildcard.media

13 stars 12 forks source link

doesn't work with wildcard.foldercontents upload #13

Open djay opened 9 years ago

djay commented 9 years ago

It probably needs to adjust content_type_registry to ensure uploaded videos are created as a video object. However changing that doesn't see to work with wildcard.foldercontents either.

keul commented 9 years ago

I have some old memory from an Aspeli comment on the Plone mailing list that say: "Dexterity have not support for webdav/ftp upload right now".

Maybe something changed meanwhile.

djay commented 9 years ago

The code is there https://github.com/plone/plone.dexterity/blob/master/plone/dexterity/filerepresentation.py but I think that code applies to creating a DX object inside a DX container (I'm using DX inside AT), and I'm also not sure yet if this code is called from wildcard.foldercontents

djay commented 9 years ago

wildcard.foldercontents does try to work with content_type_registry but it doesn't do it very well. It ignores the mimetype of the upload completely. (https://github.com/collective/wildcard.foldercontents/blob/master/wildcard/foldercontents/views.py#L861) I'll raise a bug on mockup.

djay commented 9 years ago

I created issues https://github.com/plone/mockup/issues/394 and https://github.com/plone/mockup/issues/393. Once they are fixed then it would be good for wildcard.media to provide a way to set a content_type_registry predicate (either on install or via control panel) so users have a way to handle large uploads of videos.