Sugestie [Update & Default group]

Moderator: Moderators

Locked
T-z3P
Nou Venit
Posts: 32
Joined: 25 May 2012, 18:21

Sugestie [Update & Default group]

Post by T-z3P » 13 Jul 2012, 01:04

1. Ai putea adauga un mesaj gen 'Update succesfull', iar dupa 5 secunde, redirectionare la pagina de login. Sincer nu-mi dau seama daca a actualizat baza de date sau nu.
2. Recomand modificarea flag-ului grupului 'Default' din Z in oricare din lista de mai jos pentru buna functionare a modificarii aduse de mine pluginului 'admin_sql'.

Code: Select all

m - custom level A (for additional plugins)
n - custom level B
o - custom level C
p - custom level D
q - custom level E
r - custom level F
s - custom level G
t - custom level H
In plugin folosesc o functie numita 'is_user_admin' care stabileste daca userul este admin in functie de flag-urile care le are. In cazul nostru tot ce este infara de flag-ul Z inseamna admin.

Code: Select all

stock is_user_admin(id)
{
	new __flags=get_user_flags(id);
	return (__flags>0 && !(__flags&ADMIN_USER));
}
Sper sa nu gresesc, sunt destul de obosit acum.
User avatar
vim
Mini tehnicus
Posts: 2019
Joined: 22 Nov 2007, 20:05

Re: Sugestie [Update & Default group]

Post by vim » 13 Jul 2012, 01:54

1. Sistemul de update este rudimentar si deocamdata se ocupa doar cu actualizarea DB-ului. Notificari sunt afisate utilizatorului chiar daca exista sau nu un update disponibil. Pe scurt, procesul se desfasoara cam asa:
  1. Pt a accesa pagina de update este necesar ca user-ul sa fie autentificat si sa aibe acces de Owner.
  2. Accesezi pagina /update/, se verifica daca exista un update pt DB disponibil si esti redirectionat pt prima pagina.
  3. Daca un update este disponibil, apare o notificare care te anunta acest lucru si un link care trebuie accesat pt pornirea update-ului.
  4. Daca nu exista un update disponibil, apare o notificare care te anunta acest lucru.
Sa inteleg ca tie nu iti apar aceste notificari ? Apare doar banda colorata ca in trecut sau nu apar deloc ?

2. In grupul "default" nici nu doresc sa existe admini. Acest grup este folosit doar pt a stoca temporar userii care existau in grupuri ce au fost sterse. Practic cand se sterge un grup ce contine useri, toti sunt mutati automat in grupul `default`, urmand ca un owner sa ii mute in alte grupuri dupa bunul plac. Acest grup este folosit ca un fel de "recycle bin" si flag-ul "z" este fortat ca un nivel de protectie pt cei care nu vor intelege functia acelui grup.

Daca flag-ul nu era blocat automat la "z", un owner care nu intelegea functia acestui grup, putea sa aloce access de Moderator pe grupul "default" si la ceva timp cand stergea grupul "sloturi rezervate" erau mutati toti useri in grupul "default" ; se trezeau cu access de moderator peste noapte.

Astfel, grupul avand accesul "z" fortat, oricine este inclus in acel grup este recunoscut ca un user simplu de AMXX => sanse minime sa apara probleme cu un owner care nu se da in vant dupa citit documentatie. Deasemenea grupul se poate utiliza pt a "suspenda" temporar accesul unui admin nabadaios.

Cam asta a fost cauza aparitiei acelui grup si motivul pt care acel grup are accesul blocat la "z". Asadar din punctul meu de vedere, cine este inclus in grupul "default", nu trebuie sa fie recunoscut de AMXX ca administrator, ci ca user simplu.

Este OK pt tine asa sau trebuie sa modific ceva ?

Code: Select all

[root@gw-03 ~]# man woman
Segmentation Fault (core dumped)
I retired from here
T-z3P
Nou Venit
Posts: 32
Joined: 25 May 2012, 18:21

Re: Sugestie [Update & Default group]

Post by T-z3P » 13 Jul 2012, 18:35

1. Eu incercam sa fac update-ul fara sa ma loghez si imi aparea pagina alba, apoi redirect catre pagina de login.
2. Eu am propus schimbarea flag-ului Z deoarece, sunt destul de sigur ca userii inregistrati (care tot flag Z au) vor primi mesajul acela de avertizare, in ciuda faptului ca ei deja s-au inregistrat. Ca idee lista accea de flag-uri care am insiruit-o de la M pana la T nu este folosita de nici un plugin. Acelea sunt accese "goale", nu ai acces la nici o comanda.
User avatar
vim
Mini tehnicus
Posts: 2019
Joined: 22 Nov 2007, 20:05

Re: Sugestie [Update & Default group]

Post by vim » 14 Jul 2012, 17:49

1. Da, este corect. Daca nu esti autentificat, esti redirectionat catre pagina de autentificare, iar dupa autentificare esti redirectionat inapoi catre "/update/", unde incepe procesul descris in postul meu anterior.

2. Am inteles. Eu sunt dispus sa modific flag-ul de la grupul default in altceva, dar nu crezi ca or sa apara probleme cu cei care folosesc si acele flag-uri "nefolosite" insiruite de tine anterior ? Daca consideri ca nu or sa apara eventuale probleme, propune o litera din lista aia si fac modificarea in script.

Code: Select all

[root@gw-03 ~]# man woman
Segmentation Fault (core dumped)
I retired from here
T-z3P
Nou Venit
Posts: 32
Joined: 25 May 2012, 18:21

Re: Sugestie [Update & Default group]

Post by T-z3P » 15 Jul 2012, 02:10

1. Sunt logat cu contul principal de administrare si primesc urmatoarea eroare.
Fatal error: Call to a member function row() on a non-object in C:\httpd\g1\regnick\core\libraries\Migration.php on line 325
2. Asta este recomandarea mea. Pot sa garantez in proportie de 99.99% ca nu o sa fie probleme deoarece, 99.99% din autorii de pluginuri o sa aleaga unul din cele 4 mari flaguri si anume: C (ADMIN_KICK), D (ADMIN_BAN), G (ADMIN_CVAR) sau L (ADMIN_RCON) cand concep o comanda/functie, nicidecum unul din lista enumerata de mine mai sus. Restul de 0.01% sunt cazuri foarte, foarte, foarte rare sau chiar cazul meu, unde am ales sa folosesc flagul T (ADMIN_LEVEL_H) pentru un plugin custom, unde eu consider ca aceasta comanda este la fel de importanta ca si amx_ban (amx_hack) si din acest motiv vreau sa am libertatea sa o scot sau sa i-o adaug unui user. Propunerea mea ar fi sa atribui grupului 'Default' flagul R (ADMIN_LEVEL_F) de la 'regnick'. Pentru acel 0.01% care totusi exista te rog sa adaugi un avertisment sub forma de text in documentatia scriptului, cum ca daca flagul R este folosit, se recomada schimbarea lui cu unul din lista enumerata de mine (se poate verifica asta prin accesarea fisierului 'cmdacces.ini' din /cstrike/addons/amxmodx/configs/).
User avatar
vim
Mini tehnicus
Posts: 2019
Joined: 22 Nov 2007, 20:05

Re: Sugestie [Update & Default group]

Post by vim » 15 Jul 2012, 19:35

1. Reparat in ba62e2

O sa postez dupa ce fac modificarile si pt punctul #2.

--- 15 Jul 2012, 19:27 ---

2. Modificat in 0eca30a.

git commit log

Code: Select all

changed default group flag from z to r. 
added DEFAULT_GROUP_FLAG constant
Am adaugat si constanta DEFAULT_GROUP_FLAG care reprezinta litera atribuita grupului default. Momentan aceasta litera este 'r', dupa recomandarea ta.

Pt actualizarea DB-ului, nu uita sa accesezi si /update/ dupa ce actualizezi fisierele.

Code: Select all

[root@gw-03 ~]# man woman
Segmentation Fault (core dumped)
I retired from here
T-z3P
Nou Venit
Posts: 32
Joined: 25 May 2012, 18:21

Re: Sugestie [Update & Default group]

Post by T-z3P » 17 Jul 2012, 12:48

Adauga si asta te rog in documentatie...
Pentru acel 0.01% care totusi exista te rog sa adaugi un avertisment sub forma de text in documentatia scriptului, cum ca daca flagul R este folosit, se recomada schimbarea lui cu unul din lista enumerata de mine (se poate verifica asta prin accesarea fisierului 'cmdacces.ini' din /cstrike/addons/amxmodx/configs/).
User avatar
vim
Mini tehnicus
Posts: 2019
Joined: 22 Nov 2007, 20:05

Re: Sugestie [Update & Default group]

Post by vim » 17 Jul 2012, 15:22

Voi adauga in documentatia finala, de care trebuie sa ma apuc in curand sa o scriu. Bleah, urasc sa scriu documentatie.

Code: Select all

[root@gw-03 ~]# man woman
Segmentation Fault (core dumped)
I retired from here
Locked