phpBB2 by Przemo

[Widmo]

Przemo:

Kod:

jak odpisujesz na dany temat, i masz podglad poprzednich postow w IFRAME na dole, to sie pojawia caly topic od poczatku, a nie w odwrotnej kolejnosci od ostatniego postu synchronicznie starsze. i do gabloty powtorz :D

[Gadatliwa Kasia]

[HAPOL]

http://www.przemo.org/php...pic.php?t=21174 > czy moze ktos sie temu przyjzec?

[gufiak]

przemo, możesz mi zdradzić co ten kod robi:

Kod:

groupcp.php; linia 833: if ( $user_ids )         {             $sql = "DELETE FROM " . USER_GROUP_TABLE . "                 WHERE group_id = $group_id                     AND user_id IN($user_ids)";             if ( !($result = $db->sql_query($sql)) )             {                 message_die(GENERAL_ERROR, 'Could not delete group memebership data', '', __LINE__, __FILE__, $sql);             }             sql_cache('clear', 'user_groups');             sql_cache('clear', 'moderators_list');         }

Już pare osób zgłaszało problemy z dodawaniem userów do grup. Sam też z tym problemem się spotkałem, więc postanowiłem się przyjrzeć kodowi. Zauważyłem, że ten fragment kodu usuwa z bazy przed chwilą dodanego usera podczas pobierania danych do wyświetlenia. Nie bardzo rozumiem sens istnienia tego fragmentu kodu w tym miejscu...

P.S. po usunięciu tego kodu bez problemów dodaje usera do grupy a na dodatek jest on później wyświetlany jako członek tej grupy. Wcześniej jak w bazie ręcznie przypisałem usera do grupy, to wszystko działało dobrze oprócz tego, że go nie wyświetlało jako członka tej grupy.

[Radek]

gufiak, nie mozesz tego usuwac
czesniej jesczze jest kawalek kodu, ktory wg mnie dziala tak:
jezeli np. w automatycznym dodawaniu do gurpy ustawisz od 20 postow i zamiast "dodaj nowego uzytkownika" zaznaczysz "usun uztkownika" to wtedy forum sprawdza, czy ten user ma wiecej postow niz np. 20 i jezeli jest zaznaczone w grupie usuwaj userow to usuwa

tak to chyba dziala, doglebnie sie nie przygladalem i moze zle cos pisze, jak cos to poprawcie

[ccn]

no to bedzie to forum do pobrania, czy go nie bedzie?

[gufiak]

Możliwe. Jakbym miał nieco więcej czasu, to bym dokładniej przeanalizował ten kod. Teraz niestety egzaminy, a po nich kolejne egzaminy. Ale jeśli znajdę trochę czasu zanim przemo to poprawi, to postaram się poszukać błędu. Bo napewno jest on gdzieś w tej okolicy. Może brakuje jakiegoś warunku, może czegoś innego, a może czegoś jest za dużo...

[gufiak]

No cóż, przyjrzałem się i faktycznie ten kod jest odpowiedzialny za usuwanie użytkownika, jeśli nie ma wystarczającej ilości postów. Dopiero teraz zauważyłem, że dla tej grupy ustawiłem przyjmowanie po okreslonej ilości postów. Jak wyłączyłem to, to dodaje bez problemów. Jednak uważam, że trochę źle to działa. Bo jako admin powinienem mieć możliwość dodania usera do danej grupy nawet wtedy, gdy nie ma wystarczającej ilości postów, bo np. na członkostwo zasłużył w inny sposób. W tym momencie bez sensu jest tworzenie oddzielnej grupy dla tych userów. Poza tym dobrze by było, żeby pojawiała się informacja o niewystarczającej ilości postów, bo już parę osób zgłaszało ten problem i pewnie tak jak ja każdy zastanawia się co jest nie tak, dlaczego nie chce dodać usera. Na dodatek wysyła maila z informacją o przyjęciu do danej grupy, a tak naprawdę nie przyjmuje do niej. Więc myślę, że tą funkcję przydałoby się nieco poprawić.

[Przemo]

gufiak, ten kod kasuje userów zawartych w zmiennej: $user_ids w której sa uzytkownicy pobrani zapytaniem: WHERE ug.group_id = $group_id ... AND u.user_posts < " . $group_info['group_count'] czyli kasuje tych którzy mają mniej postów niz wymagana ilość, jeśli grupa ma automatyczne dodawanie i ustawioną ilość postów. Ten kod był zawsze w posting.php ale przeniosłem go tutaj bo byly zgloszenia o automatycznym kasowaniu z grupy.

[gufiak]

Przemo, to już wiem, przyjrzałem się dokładniej zapytaniu przed tym kodem. Ale przeczytaj mój poprzedni post. Moim zdaniem przydałoby się to nieco poprawić.

[Przemo]

No więc kiedy ma być user kasowany skoro nie automatycznie? Musiala by powstac nowa kolumna w bazie, ze user zostal dodany recznie przez admina i wtedy by ni ekasowalo. Po prostu jak ustawiamy grupe od jakiejs ilosci postow, to nie dodajemy recznie ani nie zezwalamy na dolaczanie do grupy.

[gufiak]

Hmm.. Niebardzo rozumiem dlaczego user ma być kasowany automatycznie. User sam przecież nie może w żaden sposób dodać się do grupy, w najlepszym przypadku musi zostać zaakceptowany przez moderatora. Opusczenie grupy też jest albo we własnym zakresie możliwe, albo poprzez usunięcie przez moderatora/admina. Więc w jakim przypadku ma być on automatycznie kasowany? Nie wiem czy się nie mylę, ale ten kod się wykonuje chyba tylko w przypadku ręcznego dodania usera przez moderatora/admina. Popraw mnie jeśli jestem w błędzie.
A skoro już tak, to by się nie dało przenieść tego kodu do bloku gdzie jest user dodawany do grupy? Możnaby wtedy uniknąć niepotrzebnego wysyłania na emaila powiadomienia o przyjęciu usera do grupy.

[Przemo]

Automatyczne kasowanie nastepuje w trakcie przegladania grupy, gdy dana grupa ma automatyczne dodawanie po postach i gdy user ma tych postow mniej. Co w tym zlego?

[DenVer89]

dajcie przemowi spokój niech robi tą wersje bo 1.12.3 nidgy nie wyjdzie jest wszystko

[niedzwiedz_2]

Co do grup, to miałem u siebie coś takiego:
Jest gr. otwarta (bez automatycznego dod. itp) i user zgłosił chęć wstąpienia.
Zaakceptowałem go i teoretycznie był w gr. jednak nie miał kolorów.
Usunąlem go i dodałem ręcznie i wszystko było OK.

Teraz będę to testować.

Mam forum 1.12.2 udostępnioną tu na forum.

[Gadatliwa Kasia]

[Przemo]

niedzwiedz_2, sprawdź to jeszcze raz, zarejestruj sam usera i dolacz. Nie powinno tak byc, moze go jednak nie bylo w grupie, moze dalej oczekiwał?