search

contributte / fileupload

🆙 File uploads on steroids for Nette Framework (@nette). Implements blueimp/jquery-file-upload.
https://contributte.org/packages/contributte/fileupload.html
MIT License
28 stars 23 forks source link

Vícenásobný upload #33

Closed JZechy closed 3 years ago

JZechy commented 6 years ago

Pří výběru více souborů naráz neprojde kontrole na počet nahrávaných souborů.

https://forum.nette.org/cs/27287-jquery-fileupload-snadny-upload-souboru?p=3#p193902

pitr82 commented 6 years ago

Ahoj, daří se ti to řešit?

JZechy commented 6 years ago

@pitr82 Ahoj, promin, ale k uploaderu jsem se ted dlouho nedostal.

Attanon commented 6 years ago 
canUploadNextFile: function () {
    return this.uploaded < this.config.maxFiles || this.addedFiles < this.config.maxFiles;
},

Pravděpodobně se jedná o toto, proč je tam to || proč tam není &&, ono to projde tím, že uploaded = 0 když jich naklikneš víc najednou a tím pádem tahle podmínka projde i když je addedFiles víc jak maxFiles

JZechy commented 6 years ago

@Attanon Zkusil bys to napravit a poslat PR případně?

KminekMatej commented 4 years ago

Teď jsem si to opravil, moc se mi kvůli jednomu řádku nechce forkovat, nabízím tuto variantu:

canUploadNextFile: function () { return this.addedFiles < this.config.maxFiles; },

f3l1x commented 3 years ago

Solved https://github.com/contributte/fileupload/pull/52

Kdyby byl problem, otevrete novou issue.