phpBB2 by Przemo

[Grochonorte]

Kod:

############################################################## ## MOD Title: Admin notes in viewprofile ## MOD Author: Grochonorte < grochonorte@gmail.com > (Patryk Groszek) http://grochonorte.pl ## MOD Description: Mod dodaje w profilu użytkownika notatki administratora odnoszące się do niego. ## MOD Version: 1.0 ## ## Installation Level: Easy ## Installation Time: ~ 5 minutes ## Files To Edit (3): ##        /includes/usercp_viewprofile.php ##        /language/lang_polish/lang_main.php ##        /templates/your_template/profile_view_body.tpl ## ## License: Creative Commons BY-NC-SA 3.0 Polska.(http://creativecommons.org/licenses/by-nc-sa/3.0/pl/) ##############################################################

[ Komentarz dodany przez: Michael2318: 14-04-2012, 14:53 ]
Modyfikacja nie jest dobrze napisana pod względem bezpieczeństwa, jeśli autor się zdecyduje poprawić - niech napisze do mnie na PW (Michael2318).

[Gadatliwa Kasia]

[mariusz18d]

Już zainstalowałem, działa sprawnie. Dziękuje bardzo za dodatek

[Deotho]

Modyfikacja działa jak należy. Dobra robota, ale jak dla mnie zbędna.

[Flamaster]

Te notatki są widoczne dla wszystkich użytkowników czy tylko dla admina ?

[Grochonorte]

Flamaster, tylko admina.

[bolek i lolek]

Kod:

$sql = "SELECT * FROM " . USERS_TABLE . " WHERE user_id = " . $user_id . " ORDER BY user_admnote";

Co ma na celu to całe ORDER BY user_admnote co chcesz sortować ? Nie lepiej:

Kod:

$profiledata['user_admnote']

W załączniku moja koncepcja na szybko.

Kod:

$sql = "SELECT * FROM " . USERS_TABLE . " WHERE user_id = " . $user_id . " ORDER BY user_admnote";

Co ma na celu to całe ORDER BY user_admnote co chcesz sortować ? Nie lepiej:

Kod:

$profiledata['user_admnote']

W załączniku moja koncepcja na szybko.

[Gadatliwa Kasia]

[NineX]

Kod:

define('USERS_TABLE', $table_prefix . 'users');

Po co ta linijka w instrukcji?

Kod:

$text = trim($HTTP_POST_VARS['admnote']); $sql = "UPDATE " . USERS_TABLE . "  SET  user_admnote =  '" . $text . "' WHERE  " . USERS_TABLE . " .user_id = " . $user_id;

Żadnego zabezpieczenia?

Kod:

'S_ACTION' => append_sid("profile.$phpEx?mode=viewprofile&u=" . $user_id),

u powinno być zastąpione czymś innym, zerknij w constants.php.

Na koniec, pierwszy sql można umieścić w else. Nie widzę potrzeby aby trzymać go poza kodem.