Don't allow uploading empty files (allowEmptyFiles option)

[tunnckoCore]

Source of the #428 and suggested to have some option like allowEmptyFiles: false.

[quantumsheep]

Is an empty file really a file though?

[tunnckoCore]

Good question. So actually allowEmpty sounds better, really. Also it is a familiar option from other libraries/servers and such.

[xarguments]

@quantumsheep, anything that has file structure in file-system table is a file, no matter the size :)
@olstenlarck, in case of allowEmpty it wouldn't be obvious whether its for files, fields, or both. (they seem to be different config names)
So allowEmptyFiles/minFileSize (and allowEmptyFields/minFieldLength ??) seem to be clear in intentions.

[mohemos]

This is a serious issue that needs prompt attention because the empty files just mess up the entire folder, I have to run fs.unlink() every time I submit a form that has file with an empty value.

[leonardovillela]

Can i take this issue?

[tunnckoCore]

Yes of course.

[nakedtoast]

Has this feature request been added yet?

[tunnckoCore]

@nakedtoast unfortunately, seems like we do not have any PRs regarding that.

[shaneosullivan]

Has there been any movement on this?

[tunnckoCore]

@shaneosullivan nope, nothing on my end, sorry.

@leonardovillela anything?

[leonardovillela]

Hey, sorry for the late response. I only have time to see the notification now. I will take a look at it this weekend.

[leonardovillela]

@tunnckoCore or @GrosSacASac I want to contribute more to this project, there is another issue you recommend to me to work on?

[GrosSacASac]

#609