Skip to content
Snippets Groups Projects
Commit c81c925b authored by gnaegi's avatar gnaegi
Browse files

OO-323 removed mime-type accept limitation on file upload, does not work...

OO-323 removed mime-type accept limitation on file upload, does not work stable enough. Checked on server side after upload anyway.
parent f16becfc
No related branches found
No related tags found
No related merge requests found
...@@ -106,17 +106,19 @@ public class FileElementRenderer implements ComponentRenderer { ...@@ -106,17 +106,19 @@ public class FileElementRenderer implements ComponentRenderer {
sb.append(" onchange=\"b_handleFileUploadFormChange(this, this.form.fake_").append(id).append(", this.form.upload)\""); sb.append(" onchange=\"b_handleFileUploadFormChange(this, this.form.fake_").append(id).append(", this.form.upload)\"");
} }
// Add mime type restriction // Add mime type restriction
Set<String> mimeTypes = fileElem.getMimeTypeLimitations(); //TODO: This does unfortunately not work in the different browsers. Not even video/* does work properly in safari.
if (mimeTypes.size() > 0 ) { // We do not limit the selection here, check file type later on the server which is way more stable than this fancy "accept" attribute
sb.append(" accept=\""); // Set<String> mimeTypes = fileElem.getMimeTypeLimitations();
Iterator iterator = mimeTypes.iterator(); // if (mimeTypes.size() > 0 ) {
while (iterator.hasNext()) { // sb.append(" accept=\"");
String type = (String) iterator.next(); // Iterator iterator = mimeTypes.iterator();
sb.append(type); // while (iterator.hasNext()) {
if (iterator.hasNext()) sb.append(","); // String type = (String) iterator.next();
} // sb.append(type);
sb.append("\""); // if (iterator.hasNext()) sb.append(",");
} // }
// sb.append("\"");
// }
// Add pseudo focus marker on fake file chooser button // Add pseudo focus marker on fake file chooser button
sb.append(" onfocus=\"this.form.fake_").append(id).append(".nextSibling.style.border = '1px dotted black';\""); sb.append(" onfocus=\"this.form.fake_").append(id).append(".nextSibling.style.border = '1px dotted black';\"");
sb.append(" onblur=\"this.form.fake_").append(id).append(".nextSibling.style.border = '0';\""); sb.append(" onblur=\"this.form.fake_").append(id).append(".nextSibling.style.border = '0';\"");
......
...@@ -357,11 +357,11 @@ ...@@ -357,11 +357,11 @@
</mime-mapping> </mime-mapping>
<mime-mapping> <mime-mapping>
<extension>m4a</extension> <extension>m4a</extension>
<mime-type>audio/x-aac</mime-type> <mime-type>audio/x-m4a</mime-type>
</mime-mapping> </mime-mapping>
<mime-mapping> <mime-mapping>
<extension>m4v</extension> <extension>m4v</extension>
<mime-type>video/mp4</mime-type> <mime-type>video/x-m4v</mime-type>
</mime-mapping> </mime-mapping>
</web-app> </web-app>
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment