search

jkuchar / FileDownloader

Allows you to send files to browser from Presenter. (also can be used without Nette) Allows you to log and restrict access to these files. There is also support for partial downloads and files over 4GB!
http://addons.nette.org/jkuchar/filedownloader
Other
28 stars 10 forks source link

Update for Nette 2.x and BigFileTools #2

Closed vhalasu closed 11 years ago

vhalasu commented 11 years ago

Ještě mám nějaké drobnosti, aby se ty příklady daly rovnou spustit (např. v libs by měla být i třída BigFileTools). Mám to přidat ještě k tomuhle pull requestu, anebo vytvořit novou větev?

jkuchar commented 11 years ago

Co jsou ty drobnosti (kromě toho BigFileTools)? :-)

BigFileTools: Přemýšlím, jak to řešit... Composer, nebo submodule? Co je teď zvykem?

vhalasu commented 11 years ago

Typo. V příkladu na MVP chybí adresáře app/cache a app/log, v readme.txt je místo app/cache uvedený app/temp (ale jelikož není nastavený, tak se vezme výchozí hodnota ./cache).

BigFileTools jsem myslela, jestli tam nechcete taky dokopírovat (např. libs/JanKuchar/Tools/BigFileTools.php), když už tam je Nette. Určitě by tam aspoň mělo být napsané, že bez téhle třídy stahování nefunguje. Composer ani submoduly jsem ještě nepoužila.

jkuchar commented 11 years ago

cache/temp: dříve Nette používalo temp. Jakmile se dostanu k vývojářskému počítači, opravím to. (klidně to přidej do pull requestu)

To, že je v repu i Nette není dobře. Určitě to chci změnit... Asi nějak po vzoru MultipleFileUploadu, který nyní na svoje složení používá Composer. (viz https://github.com/jkuchar/MultipleFileUpload/tree/nette20 a příklad je úplně oddělený do druhého repa)

Tedy tento pull-request sloučím s hlavní větví a na ty opravy v readme, klidně založ nový. Jestli jsi to pročítala, budu moc rád, pokud Tvoje postřehy nepřijdou v niveč.

A děkuji moc za čas strávený s opravami. :-) Mimochodem je to první pull-request od všech mých projektů, máš tedy prvenství!