We have some checker in Upload Question tye : maybe we can create an helper or an extension ?

Sure, that's good idea.

If you found time ;)

I think this is failing as well on the Token CSV import

I think it's failling as well onn VVimport :)

Fix applied to:

Theme import (Themes list and Theme editor)
File upload in Theme editor (uploading files to a theme)
Image upload (in global theme options)
Image upload (in survey theme options
There are other places where this should be applied. I have some already fixed. Want to review these first.

These are fixed and pendign to be applied:

CPDB (Import CSV)
Labelset import
Token import
Question import (had to add 'action' and 'sid' to the URL)
These are pending to be fixed:

UploaderController
questiongroups
surveyadmin - Import survey - Need to split the method (used for copy and import), or move the 'action' to the URL.
frontend_helper, checkUploadedFileValidity ?
dataentry - Import VV file (need to split the method or assume an upload action when the request is POST)

https://github.com/LimeSurvey/LimeSurvey/pull/1633

See comments on github.

Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=30710

Fixed in Release 3.25.0+201117

Reopening for applying the patch on the other uploads, now that the main part has been merged

Now applying the patch to:

• CPDB (Import CSV)
• Labelset import
• Token import
• Question import (had to add 'action' and 'sid' to the URL)

PR: https://github.com/LimeSurvey/LimeSurvey/pull/1658

We should port this to LS4 after. Right?

Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=30771

Fixed in Release 3.25.3+201208

This breaks the file upload on themes. Cannot upload files anymore.

I think it was there all time, since long time ago.
Created new ticket: https://bugs.limesurvey.org/view.php?id=16948

ok, determined the issue existed before. Closing again.