phpBB2 by Przemo

michalman · Haxor **Helped:** 17 times Posts: 206 Location: Warszawa

Code:

##############################################################
## MOD Title: Lista Najbogatszych użytkowników na forum
## MOD Author: michalman ( admin@michalman.pl )
## MOD Descripton: Modyfikacja dodaje oddzielną podstrone z informacją kto ma najwięcej kasy. Wymagany cash mod albo points_system
## MOD Version: 1.0.1 xD
##
## Installation Level: easy
## Installation Time: 3 minut
##
## Files To Edit:
## includes/constants.php,
## language/lang_polish/lang_main.php
##
## Included Files:
## bogacze.php
## templates/NAZWA TWOJEGO STYLU/bogacze.tpl
##
##############################################################
## Before Adding This MOD To Your Forum, You Should Back Up All Files Related To This MOD
##############################################################

Modyfikacja wyświetla liste 5 najbogatszych użytkowników na forum.
Wymagany point_system albo cash mod

Quote:

jeśli chcemy żeby wyświetlało więcej bogaczy to wchodzimy do:

bogacze.php

znajdz:

Code:

DESC LIMIT 5";

i te 5 zamień na tyle ile chcesz mieć wyświetlanych

Quote:

Standardowo ustawiłem na kolumnę user_cash ale jeśli macie inną to wejdzcie w bogacze.php

i wszystkie

Code:

user_cash

zamień na

Code:

user_NAZWA WALUTY

Bogacze Mod.zip
Bogacze

Download 148 Time(s) 2.67 KB

kevin_ · Użyszkodnik **Helped:** 568 times Posts: 2148

Code:

define('IN_PHPBB', 3);

co to jest? IN_PHPBB to stała, definiuj się ją na TRUE bądź FALSE (1:0)

Warunek dla niezarejestrowanych napisałbym go tak:

Code:

if( [b]![/b]$userdata['session_logged_in'])
message_die(GENERAL_MESSAGE, 'niezalogowany')

przez co nie musisz ciągnąć klamry przez cały kod, i od razu będzie przejrzyściej się prezentowało wszystko

poza tym Optymalizacja, nie SLECT * tylko SELECT co_chcesz_pobrac,
bo np. po co Ci ma pobierać informacje o sesji, haśle, ile prób błędnych logowań, ile postów itd ? tylko to co najważniejsze, potrzebne dla MOda pobierz z bazy

W pętli while dałbym $template dla pętl, a dla zwykłego $template (55 linijka) podpiąłbym langi - aby trzymało się standardów phpBB

Poza tym nie trzymaj kodu HTML w pliku PHP. Skoro ekipa phpBBteam dała nam tak coś pięknego jak template system, to oddzielaj to.

Pozdro

#Bucu · PHP <3 **Helped:** 419 times Posts: 1930 Location: Kielce

Code:

$userdata = session_pagestart($user_ip, PAGE_DOWNLOAD);

E to jest podstrona downloadu?

Code:

$page_title = 'Najbogatsi użytkownicy';

to do langow

Code:

phpbb_users

stałe tabel stosuj, a nie taka fuszerke.

Code:

$result = mysql_query($sql);
while ($row = mysql_fetch_assoc($result)

jest cos takiego jak klasa db

Code:

$status = ($row['user_session_time'] > (CR_TIME - 1800) && ($row['user_allow_viewonline'] || $userdata['user_level'] == ADMIN)) ? ' <a href=viewonline.php style="text-decoration: none"><font color="green">Online</font></a>' : ' <font color="red">Offline</font>';

a to juz totalna pomylka, kod html w pliku php.

Ogolnie mod do poprawy.

michalman · Haxor **Helped:** 17 times Posts: 206 Location: Warszawa

O mój boże! Będę pisał moda który ma 3x dłuższy kod po to żeby, no nie wiem po co.
Ktoś potrzebował to napisałem. A może jeszcze niech będzie konfiguracja w PA?
Napisałem na szybko, bo tak mi się podobało

#Bucu · PHP <3 **Helped:** 419 times Posts: 1930 Location: Kielce

michalman, po to zeby nie byl bublem takim jakim jest, jak juz sie bierzesz za robienie modow to zrob to porzadnie - uwierz wygodniejsze, przyznasz mi kiedys racje.

michalman · Haxor **Helped:** 17 times Posts: 206 Location: Warszawa

Dobra macie poprawiony cały mod. Dodałem langi i cały kod poprawiłem. Nie ma juz html w php ;P

Thanks bucu za motywacie :mrgreen:

Nie zapomniałem tego umieścić w instrukcji

Seba123 · **Helped:** 46 times Posts: 623

lsteam · **Helped:** 35 times Posts: 358

Posiadam Points System który działa mi.. zainstalowałem ten mod bogacze i takie coś mi się pokazuje:

http://lssite.pl/bogacze.php

~Boltex · Posted: 13-12-2009, 18:27 **Forum address:** Cały czas w budowie

a demko można?

@krystianr

lsteam, w bogacze.php
znajdź :

Code:

user_cash

zamień na :

Code:

user_points

powinno chodzić

lsteam · **Helped:** 35 times Posts: 358

Screen z modu:

A tutaj demo: lssite.pl/bogacze.php

Seba123 · **Helped:** 46 times Posts: 623

#Bucu · PHP <3 **Helped:** 419 times Posts: 1930 Location: Kielce

Seba123,

Porządek w kodzie - HTML oddzielony od PHP.
Inna filozofia pracy - niektóre rzeczy łatwiej jest zrobić za pomocą systemu szablonów (np. obsługa formularzy).
Łatwość dostosowania do własnych potrzeb wyglądu - wystarczy znajomosc html (ktos nie zna php, pogubi kropki, przecinki etc i ma problem

)

michalman · Haxor **Helped:** 17 times Posts: 206 Location: Warszawa

Seba123 · **Helped:** 46 times Posts: 623

lepiej sprawdzaj po userid, nie username.