e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
321 stars 214 forks source link

download plugin can't upload bigger file #1809

Open Jimmi08 opened 8 years ago

Jimmi08 commented 8 years ago

I have file cca 69MB. My setting for upload is 180MB image

My php.ini is 128MB image image image

But I get this error: image

Uploading is working, I had no problem to upload 20MB file. Thanks

tgtje commented 8 years ago

Just test @Jimmi08 i have ( local) max filesize on prefs > empty > let php ini take it. ( Settings in php ini accordingly ( by size ). (also of course filetyp xml)

Go to . . /e107_web/js/core > mediaManager (js) about line 533 > max_file_size : "20mb"
Change and try again... Keep in mind time !!!

(have tested with up to 182MB zip; and 113MB mp3) Both with success screen of mp3 : yow

Jimmi08 commented 8 years ago

Thank you for help. I have max filesize on prefs empty image

I set php.ini and filexml to 128MB image

It worked on localhost.

But not on live site. I've double check that I upload my changes.

Moc commented 8 years ago

@Jimmi08 Maybe a caching issue on the live version?

Jimmi08 commented 8 years ago

Uff, you were right. I delete cache (not enabled, but for sure) in e107 admin and it worked. THANKS TO ALL. But this is still bug, I will lose this change with next update, won't I?

Moc commented 8 years ago

Yep labelled as bug. Waiting for a review from Cameron to ensure this is the right fix, then he or I will commit it.

Thanks for reporting :+1:

tgtje commented 8 years ago

Need to add info ( for readers), that this (pref setting for php ini) only is applicable for hosts that do allow such (value).. ( mine not, so i tested locally). (The mentioned time! >= low bandwidth >takes time to upload ; if slow one could reach/go over setting)

Jimmi08 commented 8 years ago

I need to customize next parameter, file extension, it's ignored filetype.xml image

image

tgtje commented 8 years ago

Thats a good pointer @Jimmi08 :+1: So basically theres filetypes (allowed or not) and a filter that overlaps, but are NOT identical. two methods that CAN create issues when certain types are being used. While (found perhaps a new bug on upload itself,needs testing first> using upload php itself, file goes to to system/temp. but no message or record...), plupload js IN combination with filet.xml works (i can upload a m4a) ; this issue relates to mediaManager upload/filter.

Admin should try to make this a separate issue !!

Moc commented 8 years ago

Summary:

chimcen commented 7 years ago

Stupid question - but could this be a possible reason:? In mediaManager.js Line 555 ends the filetypes list with a comma (other lines with filetypes end without)...

{title : "Media files", extensions: 'mp3,mp4,wav,ogg,webm,mid,midi,'},

Another thing - it would be great when adding tiff, 7z and avi to defaults...

Moc commented 7 years ago

The issue here is that these values (size and file type) are hard coded in a place where they shouldn't be. They should be customisable through the filetypes.xml file, in which you can also add or change current filetypes and sizes.

rica-carv commented 7 years ago

Or why not use a preferences field for it? That way, all could be donne via admin interface area...

Moc commented 7 years ago

Yeah there was some issue with this in the past. Not sure what it was, maybe security reasons, and then it was decided to use a physical file rather than prefs. Anyhow, I'll leave this for someone else to answer.

CaMer0n commented 7 years ago

I don't think plupload supports different file-sizes by type. Recent changes I made should allow uploading via media-manager with the limit stored in the pref. We may need to allow ALL file-types through the media-manager.

Jimmi08 commented 6 years ago

to have it in one place (it's something different, I know, but this will be solved, just think on other ideas) https://github.com/e107inc/e107/issues/3169

CaMer0n commented 5 years ago

It's currently at 400mb.