Forum PHP.pl

Witam, ciekaw jestem waszej opinii na temat skryptu (narzędzia) które niedawno napisałem.

Ów skrypt nazywa się Filemon JS i jest to darmowy menadżer plików napisany w JavaScript (na framework'u ExtJS 2.0.2) z dodatkiem odrobiny kodu PHP do zarządzania plikami po stronie serwera.

Strona WWW: http://mobilegb.eu/pl/filemon-js
Demo: http://mobilegb.eu/pl/filemon-js/demo

Ciężko coś oceniać - podstawowa funkcja managera plików to ich ładowanie i zmiana, a to jest w demie niedostępne.

proponował bym dodać przechwycenie prawego przycisku myszy, coś jak jest w google docs

właściwości nie działają, pliki mają po 0 bajtów

Dzięki za informację. Problem już naprawiony. Chociaż do błędu się nie przyznaję - to wina jakiegoś idioty który umieścił w PHP funkcję magic_quotes_gpc



Skromne menu pod prawym klawiszem jest. Pojawia się po kliknięciu na plik (kliknięcie w pustą przestrzeń daje domyślne menu przeglądarki).



Niestety w demie musiałem zablokować możliwość jakiejkolwiek edycji danych. Wiem, że poszedłem trochę na łatwiznę ale miałbym niezły problem z utrzymaniem bezpieczeństwa gdyby każdy mógł wgrywać na serwer dowolne pliki.

W Browser.php:

[PHP] pobierz, plaintext 
    public function uploadFile($dir)
        if (is_uploaded_file($_FILES['uploadFile']['tmp_name'])) {
[PHP] pobierz, plaintext 

Nie operuj bezspośrednio na globalnych zmiennych w tej metodzie. Zamiast tego użyj po prostu:

[PHP] pobierz, plaintext 
public function uploadFile($uploadFile, $dir) {}
[PHP] pobierz, plaintext 

Będzie bardziej uniwersalne i o wiele ładniejsze.

W ogóle ta metoda nie powinna zwracać jakieś kody błędów a wyrzucać wyjątki z tymi kodami błędów.
Zamiast

[PHP] pobierz, plaintext 
return self::UPLOAD_ERR_EXTENSION;
[PHP] pobierz, plaintext 

daj

[PHP] pobierz, plaintext 
throw new Exception(self::UPLOAD...)
[PHP] pobierz, plaintext 

itp.

Poza tym kwiatki również w metodzie
public function download($files)
- dziwne exity...

W Mime.php:
dodaj metodę

[PHP] pobierz, plaintext 
public static function get($key) {
  return $this->_mimes[$key];
}
[PHP] pobierz, plaintext 

tyle z szybkiego rzutu okiem na część php.

Ten post edytował ano 17.01.2012, 19:18:34

fakt jest ale pojawiają się oba