Ocena skryptu Opcje

[cisu]

Witam,
do szkoły na 6 napisałem skrypt. Dostałem za niego 6, nauczycielka w ogóle w php nie siedzi więc trudno nie było ;]
Postanowiłem dać go Wam do oceny. Adres:

Kod

http://itickets.icisu.net/

Kod

Przykładowe konto usera:
Login: testuser
Hasło: haslousera

Przykładowe konto admina:
Login: testadmin
Hasło: hasloadmina

Kilka słów o skrypcie: jest to system ticketów, czyli zapytań do Administracji w danej sprawie. Admin dodaje użytkowników, może zmieniać ich dane i usuwać. Użytkownik pisze tickety, odpowiada itd. Możliwość edycji konta, wysyłania powiadomień o odpowiedzi na email (phpMailer) itd.

Teraz tak: ja wiem, że np. nie ma przy dodawaniu userów walidacji znaków, maila itd, ale może to robić tylko Admin, więc uznałem, że nie ma potrzeby takiego zabezpieczania, bo Adminem miałem tam być tylko ja no i to było tylko do szkoły, jak coś to mogę dodać.

Aha, i na cele testowe nie można zmieniać ustawień systemowych, wyłączyłem to (IMG:style_emoticons/default/tongue.gif)

Poza tym od strony użytkownika jest zabezpieczone wysyłanie np. skryptów javascript albo takie tam.

Ja proszę o łagodną ocenę, nie mam takiego doświadczenia dużego w php jak niektórzy tutaj (IMG:style_emoticons/default/tongue.gif)

--------

Jak ktoś znajdzie poważniejsze błędy, proszę o napisanie, postaram się poprawić.

Ten post edytował cisu 7.12.2009, 19:37:11

[cisu]

Ogólnie to nie powiedziałem, już uzupełniam.

Chyba chodziło o użytkowników, nie da się zmieniać danych administratorów, tylko użytkowników. Można sobie zmienić w bazie danych, no tak zrobiłem i tego raczej nie zmienię.

Kolorystyka to jest czysty CSS, łatwo pozmieniać. Ale według mnie jest OK, może już ślepy jestem ale ja bym nie zmieniał kolorów.

A sortowania nie robiłem bo nie przewidywałem, że kiedykolwiek nawet tutaj o tym wspomnę. Może i by się przydało, ale raczej nie będę dopisywał. Zawsze można CTRL+F (IMG:style_emoticons/default/tongue.gif)

A zarządzanie ticketami chyba działa, nie ma edycji, ale jest czytanie i odpowiadanie.