Closed (fixed)
Project:
File Entity (fieldable files)
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Jul 2013 at 09:00 UTC
Updated:
11 Oct 2013 at 15:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
adammaloneattaching first prospective POC patch
Comment #2
adammaloneComment #3
aaron commentedAfter testing, even though the message below the add file interface reads with the new limit, I am still able to upload larger files.
Comment #4
tobiasb* removed whitespace
The filesize validation is skipped for user 1. Therefore use a different user to test the patch.
Comment #5
aaron commentedThis works for me.
Comment #6
dave reidThis should be altered into admin/config/media/file-system instead of adding a new page.
Comment #7
tobiasbAdded the settings to file_entity_form_system_file_system_settings_alter().
Comment #8
aaron commentedThis works as well.
Comment #9
dave reidLikely need to be re-rolled for #2066459: Move settings to our own page at admin/config/media/file-settings
Comment #10
tobiasbMoved the settings to the new form.
Comment #11
aaron commentedWorks as well.
Comment #12
devin carlson commentedI've only found a few issues with #10; it looks good otherwise.
Now that File entity has its own administration settings form setting a weight is unnecessary.
If the supplied
$options['max_filesize']pass theif{}statement then theelseif{}will never trigger, allowing other modules to bypass the user defined file size limit.The new
file_entity_max_filesizevariable should also be added to the list of variables to delete during uninstall.Comment #13
devin carlson commented#12: add-max-filesize-configuration-option-2045605-12.patch queued for re-testing.
Comment #14
devin carlson commentedCommitted #12 to File entity 7.x-2.x.
http://drupalcode.org/project/file_entity.git/commit/af6c714