[PROBLEM] Color Groups [nivisec]

[ThaRipsta]

Hallo,

ich habe auf unserem board (http://www.btnhboard.com/index.php) kurzzeitig den "Color Groups" MOD von nivisec installiert. Da ich leider sonst keinen Support zu diesem MOD finde, versuche ichs einfach mal hier (hab SEHR gute erfahrungen hier gemacht )

Mein Problem:

nach der Installataion des MODS musste ich feststellen, dass ich den server fast gekillt habe:

Mit 40 aktiven usern (eingelogged) musste ich feststellen, dass ich 245 (!) sql-queries hatte (normal 21-25) [index]

Auf diesem Board sehe ich auch relativ viele queries [index].

Ist das die "eigenart" des MODS, oder habe ich einen fehler gemacht?

Falls es eine eigenart sein sollte: GIBT ES EINE ANDERE LÖSUNG?!?

Gruss

-lilripsta

[oxpus]

hab SEHR gute erfahrungen hier gemacht

Danke danke. Hört man immer wieder gerne

BTW: Schönes Style auf Deinem Board!

Und nochwas:
Du darfst mein Board nicht unbedingt und direkt mit anderen vergleichen. Ich habe über 200 Mods eingebaut Da ist schon einiges mehr am rechnen, auch ohne den ColorGroupMod In der Regel geht es aber, Du siehst bei mir ja auch, wie lange die Seite braucht, um geladen zu werden, trotz der teilweise immensen Anzahl an Queries.

Nun zum Color Groups Mod:
Es wird mit dem Mod immer für jeden User, egal, ob Portal (sofern vorhanden), Index, Forumansicht, Topicansicht oder wo auch immer, innerhalb einer ohnehin schon teilweise recht mächtigen Schleifen die entsprechende Farbe abgefragt.
Das zieht natürlich eine Menge Queries nach sich und bremst das Board aus.

Abhilfe wäre nur schwer möglich. Man könnte vielleicht versuchen, in einer Abfrage festzustellen, welche User überhaupt eine andere Farbe bekommen (am besten in der page_header.php) und dann daraus gezielt die User "einfärben". Dann egal wo.

Eine andere Version:
In dem Userprofil beim Setzen einer Farbe dieses vermerken (wenn das nicht schon passiert) und dann prüfen, ob ein "einfärben" nötig ist. Auch das ist überall möglich, da hier die Prüfung in der zentral vorhandenen Farb-Funktion abgelegt werden kann.

Der 2. Variante werde ich mal nachgehen, ich denke, damit kann man am Besten einige Queries sparen...

Und zum Schluss:

Falls es eine eigenart sein sollte: GIBT ES EINE ANDERE LÖSUNG?!?

Mir ist leider keine bekannt

[ThaRipsta]

Danke für die (WIE GEWÖHNLICH) schnelle und aufschlussrecihe Antwort

Schön, dass dir unser layout gefällt

Leider bin ich einer der phpBB user, die -Schritt für Schritt- (ohne EasyMod *stolzsei*) die mods installiert, ohne viel Ahnung von PHP oder der board architektur an sich zu haben (ok..bin ehrlich: hab weniger als wenig Ahnung )

Von daher wäre mir eine Art "step by step" anleitung zu deinen Ideen (die evtl auf durchführbarkeit getestet wären ) SEEEEHR entgegenkommend

Da ich mir vorstellen kann, dass du viel zu tun hast, und mein Problem nicht dringlich ist, kann ich warten

und nochmal:

DANKÖÖ!

-lilripsta

PS: wie gesagt..habe kaum ahnung..aber wäre es möglich, den Color user MOD irgendwie in die "supermod MOD" schleifen einzubauen (die user werden ja im index farblich angezeigt)

[oxpus]

Keine Angst, ich werde eine Anleitung schreiben, die wie jeder andere Mod auch zu verstehen und installieren ist.

Und zum Super Mod: Der ColorGroupMod versorgt auf Wunsch jeden User mit einer Farbe und man kann ja auch Farbgruppen anlegen, in die alle Super Mods reingepackt werden. Somit sind diese auch gleich farbig dargestellt

[ThaRipsta]

ich meinte eigentlich eine möglichkeit, das script des "superMod-MODs" so zu verändern, dass nicht nur die user (smods, admins, etc) in der "who is online"-liste, sondern AUCH in den Foren (last post) und Topics angezeigt würden.

--> "eingeschränktes" Color Groups look-alike

[oxpus]

Hier die Lösung zur Reduzierung der vielen Queries (Für Version 1.1.0 des ColorGroups Mods!):

Code: Alles auswählen

#
#-----[ OPEN ]-----
#
includes/functions_colorize_groups.php

#
#-----[ FIND ]-----
#
function color_group_colorize_name($user_id, $no_profile = false)
{
	global $board_config, $phpEx, $db, $phpbb_root_path;
	
	// First see if the user is Anon
	if ($user_id != ANONYMOUS)

#
#-----[ REPLACE WITH ]-----
#
function color_group_colorize_name($user_id, $no_profile = false)
{
	global $board_config, $phpEx, $db, $phpbb_root_path, $colorusers, $coloruname;
	
	// First see if the user is Anon
	if ($user_id != ANONYMOUS && $colorusers[$user_id] != 0)

#
#-----[ FIND ]-----
#
	else
	{
		return false;
	}

#
#-----[ REPLACE WITH ]-----
#
	else
	{
		$username = $coloruname[$user_id];
		if ($no_profile)
		{
			$user_link = "<span $style_color>$username</span>";
		}
		else
		{
			$user_link = '<a href="' . append_sid($phpbb_root_path."profile.php?mode=viewprofile&" . POST_USERS_URL . "=$user_id") . '"' . $style_color .'>' . $username . '</a>';
		}
		return $username;
	}

# 
#-----[ OPEN ]------------------------------------------ 
# 
includes/page_header.php

# 
#-----[ FIND ]------------------------------------------ 
#
$template->pparse('overall_header');

?>

# 
#-----[ BEFORE, ADD ]------------------------------------------ 
#
$sql = "SELECT user_id, user_color_group, username FROM " . USERS_TABLE;
if( !($result = $db->sql_query($sql)) )
{
	message_die(GENERAL_ERROR, 'Could not read user color groups', '', __LINE__, __FILE__, $sql);
}

$colorusers = array();
$coloruname = array();
while ( $row = $db->sql_fetchrow($result) )
{
	$userid = $row['user_id'];
	$colorusers[$userid] = $row['user_color_group'];
	$coloruname[$userid] = $row['username'];
}


# 
#-----[ SAVE/CLOSE ALL FILES ]-------------------------- 
#
# EoM

Damit werden in der includes/page_header.php alle User abgefragt und anschliessend bei jedem Aufruf der Farbfunktion geprüft, ob der betreffende User einer Farbgruppe angehört. Bis dahin alles mit nur einer Query gegen die Datenbank!
Ist eine Gruppe beim User eingetragen, so wird für den User die Färbung abgefragt und durchgeführt = 1 Query je User.
Was an dieser Stelle nun wegfällt, sind Queries für die User, die keiner Farbgruppe angehören.

So ergibt sich folgende Beispielrechnung:
Es sind 25 User auf einer Memberlist-Seite dargestellt = 25 Queries nur für die Farbgruppen nach der Originalmethode (ohne alles andere!).

Enthalten sind dabei nur 2 Usern, die einer Farbgruppe angehören = nach der neuen Methode 2 Queries + 1 Query über alle User = 3 Queries.
Ersparnis: 22 Queries. Ist das nichts? Und das alleine bei nur einem User, der die Memberliste aufruft.

Auch ich habe hier jetzt mit dieser o.g. Änderung deutlich Queries abgebaut. Mein Board ist gleich ein gutes Stück schneller geworden... Naja, eher ein Tick, aber spürbar.

[AmigaLink]

Hört sich nach einer feinen sachen an!
werde ich doch glatt bei mir auch einbauen.

[skittles]

Werds auch gleich mal einbauen!

[oxpus]

Wie gesagt, es sind zumindest bei mir weniger Queries.
Ich hoffe nur, ich habe bei der Beschreibung nicht wieder Fehler eingebaut...

[AmigaLink]

WOW
Bei mir sind die Queries, beim aufruf der Mitgliederliste, von 220 auf 77 zurück gegangen!!!

[oxpus]

Na, wer sagts denn...
Und dann überleg mal, was die Datenbank weniger aushalten muss, wenn viele User gleichzeitig Seiten wie Memberliste, Viewtopic, Index, etc. aufrufen

[AmigaLink]

Ja, echt eine prima sache!!!
Ich muss sagen mir war bisher garnicht bewust das es bei der Mitgliederliste soviele Queries waren. :rolleyes: Mir ist eben beinahe das Essen aus dem Gesicht gefallen als ich das getestet habe. Aber zum glück kam ja dann die Lösung durch deinen Fix!

[oxpus]

Was meinst Du, was ich durch solche, bzw. ähnliche Tricke meine viewtopic beschleunigt habe!
Ich habe alle Queries, die für User bei jedem Post durchgeführt werden, vor die eigentliche Post-Schleife gesetzt, dann die entsprechenden Werte wie z.B.

Code: Alles auswählen

$pictures[$row['user_id']] = $row['total'];

erstellt und innerhalb der Schleife dann nur noch mit

Code: Alles auswählen

$user_pics = $pictures[$poster_id];

abgefragt.
So sind 5-6 Abfragen umgebaut worden und ich kam von knapp 300 auf unter 70 Queries bei nur 10-15 Posts pro Seite

Nicht umsonst ist dieses Board auch trotz der sehr zahlreichen Mods immer noch flott unterwegs. Letztendlich aber auch durch den XS-Mod.

[skittles]

Du könntest dies alles ja in einen oxpus mega Speed Mod packen!

[oxpus]

Dafür sind die Boards zu unterschiedlich. Wenn man nichts modded, kann man das Board damit nicht beschleunigen.
Nur wenn Mods in der Viewtopic mit SQL_Queries aufschlagen, kann man da was tun.
Und für jeden Mod eine eigene Anleitung... Nicht so wirklich prickelnd...

[skittles]

auch wahr!, schade eigentlich!

[AmigaLink]

PROBLEM!!!
In der "Wer ist Online?" anzeige (Portal & Forum) werden die Registrierten benutzer nicht mehr angezeigt!!!

// EDIT

Und die Farbmarkierungen stimmen teilweise auch nicht mehr!

[oxpus]

Ähnliches war mir auch schon aufgefallen.
Füge daher mal den Code-Block der page_header.php zusätzlich vor

Code: Alles auswählen

//
// gzip_compression
//

ein. Dann klappt wieder alles.

[AmigaLink]

Nö!
Die "Wer ist Online?" Anzeige geht zwar nun wieder, aber es werden nur die Mitglieder farblich markiert dargestellt, die einzeln einer Farbgruppe zugefügt wurden. Mitglieder von Benutzergruppen werden ignoriert!

[oxpus]

Ist das tragisch oder werden die garnicht mehr angezeigt?
Ich habe selber hier viele User-Anzeigen, die noch nicht einmal gefärbt werden.
Iigendwo muss man auch mal Abstriche machen

[AmigaLink]

Angezeigt werden sie. Nur die entsprechende färbung fehlt!
Das tragische daran ist, das ich "Color Groups" nur Installiert habe um die Mitglieder von Benutzergruppen, welche zum größtem teil wiederum erweiterte rechte in meinem Board haben, farblich makiert darstellen zu können! Wenn das nicht mehr geht (oder nur indem ich die Mitglieder einzeln in eine Farbgruppe aufnehme (was man schnell mal vergessen hat)) dann hat der Mod für mich keinen Sinn, den Admins & Mods wurde ja eh schon Farblich gekenzeichnet.
(ich weis, ich bin schlimm)

[oxpus]

Und genau so ist der Mod ausgelegt:
Farbgruppe anlegen, User zuordnen, Einfärbung.
Gruppen können angeblich auch eingefärbt werden, das habe ich aber aktuell nicht im Kopf wie das geht. Über die Funktion für User-Ränge bin ich jedenfalls vorhin noch gestolpert.

[AmigaLink]

Das meine ich ja!
Du kannst einer Farbgruppe eine (oder auch mehrer) Benutzergruppen zuordnen deren Mitglieder dann die färbung der Farbgruppe erhalten.
Genau das hab ich bei mir auch gemacht und normalerweise geht auch alles wie es soll. Nach deinem Querie-Fix allerdings, werden die Mitglieder einer Benutzergruppe, die wiederum einer Farbgruppe angehört, nicht mehr als Mitglieder der Farbgruppe erkannt und somit nicht eingefärbt!
(Was für ein Satz )

[oxpus]

(Was für ein Satz )

Stimmt: Lang, verschlungen, aber verständlich.

Gut. Mit den Gruppen muss ich mal schauen. Ich weiß nicht, warum das nicht in den Userprofilen gespeichert wird, denn ein User kann ja nur eine Farbe haben.
Also bis später.

EDIT: Was für eine Version verwendest Du? Ich kann bei mir einer Farbgruppe keine Benutzergruppen zuordnen Oder ich bin blind...

EDIT 2: Ja, hatte selber die 1.1.0 installiert. Die 1.2.0 werde ich dann jetzt entsprechend umbauen.

[oxpus]

Und hier ist der Code für die Version 1.2.0 des Color Group Mod (Wer die alte Fassung eingebaut hat, muss diese vorher wieder entfernen!):

Code: Alles auswählen

# 
#-----[ OPEN ]------------------------------------------ 
# 
common.php

# 
#-----[ FIND ]------------------------------------------ 
#
?>

# 
#-----[ BEFORE, ADD ]------------------------------------------ 
#
$sql = "SELECT user_id, user_color_group, username FROM " . USERS_TABLE;
if( !($result = $db->sql_query($sql)) )
{
	message_die(GENERAL_ERROR, 'Could not read user color groups', '', __LINE__, __FILE__, $sql);
}

$colorusers = array();
$coloruname = array();
while ( $row = $db->sql_fetchrow($result) )
{
	$userid = $row['user_id'];
	$colorusers[$userid] = $row['user_color_group'];
	$coloruname[$userid] = $row['username'];
}

#
#-----[ OPEN ]-----
#
includes/functions_colorize_groups.php

#
#-----[ FIND ]-----
#
function color_group_colorize_name($user_id, $no_profile = false)
{
	global $board_config, $phpEx, $db, $phpbb_root_path;
	
	// First see if the user is Anon
	if ($user_id != ANONYMOUS)
	{
		// Get the user info and see if they are assigned a color_group //

#
#-----[ REPLACE WITH ]-----
#
function color_group_colorize_name($user_id, $no_profile = false)
{
	global $board_config, $phpEx, $db, $phpbb_root_path, $colorusers, $coloruname;
	
	// First see if the user is Anon
	if ($user_id != ANONYMOUS)
	{
		$username = $coloruname[$userid];
		if ( $colorusers[$user_id] != 0 )
		{
		// Get the user info and see if they are assigned a color_group //

#
#-----[ FIND ]-----
#
			if (!isset($row['username']))
			{
				//If there was a problem before, we don't want a blank username!
				$sql = 'SELECT username FROM ' . USERS_TABLE . "
				WHERE user_id = $user_id";
				$result = $db->sql_query($sql);
				$row = $db->sql_fetchrow($result);
			}
			$username = $row['username'];

#
#-----[ REPLACE WITH ]-----
#
		}

#
#-----[ FIND ]-----
#
	else
	{
		return false;
	}

#
#-----[ REPLACE WITH ]-----
#
	else
	{
		$username = $coloruname[$user_id];
		if ($no_profile)
		{
			$user_link = "<span $style_color>$username</span>";
		}
		else
		{
			$user_link = '<a href="' . append_sid($phpbb_root_path."profile.php?mode=viewprofile&" . POST_USERS_URL . "=$user_id") . '"' . $style_color .'>' . $username . '</a>';
		}
		return $username;
	}


# 
#-----[ SAVE/CLOSE ALL FILES ]-------------------------- 
#
# EoM

Getestet hatte ich einen User in der Farbgruppe, nur eine Benutzergruppe in der Farbgruppe und zuletzt einen User und eine Benutzergruppe in der Farbgruppe. Jedesmal war der einzelne User nicht Mitglieder der Benutzergruppe.
Ergebnis: Bei mir scheint es zu funktionieren. Zum Ergebnis hatte ich mir die Mitgliederliste und die Benutzergruppe in der Gruppenverwaltung angesehen.