Gäste - Post - Freischaltung

[Bootenks]

War inhaltlich viel falsch also am Code oder nur die install.txt?? interessiert mich muss ja wissen in wie weit ich da noch an mir arbeiten müsste ^^

Edit: noch eine Anmerkung: Ich fand es halt nur komisch, da alles funktionierte ausser dem include der page_header ... da war ich irgendwie doof... Oo Also hätte man doch nur den code einfach in den else Teil gepackt und fertig wäre er gewesen. Deshalb meine interessierete Nachfrage... Er funkzt ja bei mir Oo Deshalb wundert mich jetzt dein einiges ~.~ Also ich kann mit Kritik umgehen nur verstehen müsste ich sie auch *grins* Deswegen würde ich gern wissen was du geändert hast. *so das wars aber jetzt auch mit der Nachfrage*

[tom10]

Also ich bin da gerade mal dran den Code einzubauen...
Laut Anleitung soll ich dies machen:

#
#-----[ IN-LINE FIND ]------------------------------------------------
#
WHERE poster_id IN ($matching_userids)";

#
#-----[ IN-LINE REPLACE WITH ]------------------------------------------------
#
WHERE poster_id IN ($matching_userids)
$where_guestpost";

Aber fehlt da nicht nach

WHERE poster_id IN ($matching_userids)

ein Punkt ?
So das es dann so aussieht :

Code: Alles auswählen

				WHERE poster_id IN ($matching_userids).
					$where_guestpost";

[Bootenks]

nein! wenn ein . kommen soll dann müsstest du es so schreiben:

Code: Alles auswählen

WHERE poster_id IN ($matching_userids)" .
$where_guestposts;

Da schwierige ist zu erkennen wann der Compiler die $variablen in einer Zeichenkette interpretiert und wann nicht... Das kenne ich auch zu genüge *schlimm schlimm* ^^

[oxpus]

@Bootenks
Nochmal:
Wenn Du die page_header.php includierst, werden alle META an die overall_header.tpl übergeben, die zu diesem Zeitpunkt der Klasse $template bekannt sind ( $template->assign_vars('META' =>...); ).
Da die page_header.php auch die overall_header.tpl parst, kannst Du nach dem Include der Datei eben nunmal keine weiteren META-Angaben mehr an die overall_header.tpl schicken, da die ja bereits an den Client (Webbrowser des Users) ausgeliefert wurde !

Jetzt verstanden?

Daher habe ich eben das Includieren der page_header.php in der modcp.php erst dort eingesetzt, wo auch die Verarbeitung losgeht und nicht vor dem Schritt, wo das META übergeben werden soll.
Die Funktion message_die(...); ruft schliesslich die page_header.php selber schon auf

[Bootenks]

hä hast du meine Frage falsch verstanden Oo Ich habe das schon verstanden warum das include dahin musste Oo Ich wollte nur fragen was du einiges geändert hast also sonst noch Oo weil nebst diesem Problem war mir kein anderes bekannt. That was my question ~.~

[oxpus]

Hmpf, was ich noch geändert hatte...
Hauptsächlich die Anleitung eben selber.

Ach ja, im Groben:
Verwende keine kurzen $_VARS, wie $_POST oder $_GET. Das phpBB blockiert mittlerweile deren Inhalte! Nimm statt dessen halt wieder $HTTP_POST_VARS oder $HTTP_GET_VARS, oder oder oder...
Dazu bei Bedingungen auch immer schön die Klammern verwenden.
Also nicht so:

Code: Alles auswählen

if($bedingung == TRUE) $true = 1;

sondern so:

Code: Alles auswählen

if($bedingung == TRUE)
{
     $true = 1;
}

Dient der Übersichtlichkeit und man kann besser nachträglich Änderungen vornehmen, sofern möglich, nötig, sinnvoll...

Ansonsten nur nochmal das übliche.
Du solltest bei eine Suchstelle, wie in der usercp_register.php immer auch nur möglichst eine Zeile bei FIND angeben, ausser, es wäre nicht eindeutig genug.
Denn andere MODs oder Eigenentwicklungen der Forenbereiber könnten hier schon neuen Code dazwischen eingefügt haben.
Du hattest z. B. $viewemail... und $allowviewonline... zusammen angegeben.
Es exitieren MOD-Vorlagen für Profiländerungen, die nur nach $viewemail suchen und solche MODs würden Deine Anleitung gnadenlos torpedieren.

Das wars im Allgemeinen.
Mehr wüsste ich jetzt auch nicht mehr, was ich geändert hatte.
Und zuletzt noch ein Tip:
Verwende für neue MODs immer ein ungemoddetes phpBB der neuesten Version.
Auch ist die letzte Version PHP 5.x und ein Apache 2 auf einer MySQL 4 oder 5 Datenbank sinnvoll.
Diese absolut moderne Forum-Umgebung verzeiht keine Fehler und somit ist mir z. B. der riesige Bug in der modcp.php erst aufgefallen.
Denn bei mir war eben META (ausser dem Standard) komplett leer und damit passierte auch keine Weiterleitung...

[Christian_N]

Oxpus unser Genie von ihm kann man viel lernen
Also, wenns dann soweit ist und Final, baue ich es nächste Woche mal auch ein, bis dahin dürfte ich meine änderrungen soweit fertig haben

[oxpus]

Aus meiner Sicht ist der MOD bereits final...

[Christian_N]

Oki,
dann schau ich nächste woche mal, tja die Zeit eben ist der Feind, das kennt ihr alle ja nur allzu gut mit der Zeit

[Bootenks]

@oxpus. Thanks für die Aufklärung... zum Thema $_POST oder $_GET verwenden laut Vorschrift werde ich mal aber mein Senf dazu geben... Sollte sie dies wirklich torpedieren dann gute Nacht... Ich verwende nur die neuen GLOBAL Varibalen die laut php sicherer sind. Verstehe ich nicht wieso dies torpediert werden sollten. Aber das andere, welches du aufgezählt hast, macht durchaus Sinn... Ist für mich als Programmierer aber ein Dorn im Auge ... hilft aber anderen eben mein Proggy zu installieren. That's why I can understand it ~.~

PS: ich werde die neue Version in den ersten POST "uppen"

[oxpus]

Öhm, ich gebe Dir in Bezug auf die kurzen $_VARS Recht, was PHP angeht, phpBB behandelt die aber nunmal in der 2.0.xer Version noch anders und lässt aus Kompatibilitätsgründen nur die langen Versionen zu.
Daher: Besser noch so verwenden.
Und GLOBALS wurde auch unter PHP nie gerne gesehen. Zeugt von unsauberer Programmierung, auch, wenn man register_globals = On verwendet und ausnutzt. Das sind Sicherheitslücken, die man sich nur schwer ausmalen kann, aber dann jammert, wenn das Board gehackt wurde.
Auf phpbb.com und .de jeden Tag neu zu lesen...

Aber lange Rede kurzer Unsinn: Ich werde die nun aktuelle Version hier so schnell ich kann in OXPUS.de einbauen und für alle damit nicht abgedeckten Stellen ein Add-On zaubern.
Kann man dann dabei legen oder eben separat anbieten...

Ich bedanke mich jedenfalls für diesen nun genialen MOD. Ja ja, ich habe geholfen, aber die Hauptarbeit und die wirklich guten Ideen bei der Umsetzung lagen nunmal bei Dir. Also: !

[Bootenks]

thanks *freu mich wenns klappt und dir und den anderen hilft*

[tom10]

So und ich komme leider beim Einbau nicht wirklich weiter. Ich soll laut Anleitung das machen:

Code: Alles auswählen

							$sql = "SELECT post_id
								FROM " . POSTS_TEXT_TABLE . "
								WHERE post_text LIKE '$match_word'
								$search_msg_only";

# 
#-----[ REPLACE WITH ]------------------------------------------------ 
# 
							$sql = "SELECT p.post_id, p.post_activate, pt.post_id
								FROM " . POSTS_TEXT_TABLE . " pt, " . POSTS_TABLE . " p
								WHERE pt.post_text LIKE '$match_word'
								AND p.post_id = pt.post_id
								$where_guestpost
								$search_msg_only";

Und finde bei mir in der search.php aber nur eine Stelle die geringe änlichkeit mit der zu suchenden Stelle aufweist.

Code: Alles auswählen

						}
						else
						{
							$match_word =  addslashes('%' . str_replace('*', '', $split_search[$i]) . '%');
							$search_msg_only = ( $search_fields ) ? "OR pt.post_subject LIKE '$match_word'" : ''; 
							$search_msg_only .= ($only_bluecards) ? " AND p.post_bluecard>0 AND pt.post_id=p.post_id " : ""; 
                     						$sql = "SELECT pt.post_id 
                        					FROM " . POSTS_TEXT_TABLE . "
 								pt " . (($only_bluecards) ? ','.POSTS_TABLE . ' p ' : '') . "
                        					WHERE pt.post_text LIKE '$match_word'
								$search_msg_only";
						}
						if ( !($result = $db->sql_query($sql)) )
						{
							message_die(GENERAL_ERROR, 'Could not obtain matched posts list', '', __LINE__, __FILE__, $sql);
						}

Kann mir einer von euch helfen das dort richtig rein zu bekommen ?

[oxpus]

Nimm diesen Block hier:

Code: Alles auswählen

						}
						else
						{
							$match_word =  addslashes('%' . str_replace('*', '', $split_search[$i]) . '%');
							$search_msg_only = ($search_fields) ? " OR pt.post_subject LIKE '$match_word' " : ""; 
							$search_msg_only .= ($only_bluecards) ? " AND p.post_bluecard > 0 " : ""; 
                     						$sql = "SELECT p.post_id, p.post_activate, pt.post_id 
                        					FROM " . POSTS_TEXT_TABLE . " pt ,".POSTS_TABLE . " p
                        					WHERE pt.post_text LIKE '$match_word'
                        						AND pt.post_id = p.post_id
                        						$where_guestpost
									$search_msg_only";
						}
						if ( !($result = $db->sql_query($sql)) )
						{
							message_die(GENERAL_ERROR, 'Could not obtain matched posts list', '', __LINE__, __FILE__, $sql);
						}

EDIT
Gute Nachrichten:
Ich habe diesen MOD in meiner lokal vorhandenen Testkopie von OXPUS.de erflgreich installiert.
Für den Split Topic Type MOD werde ich gleich noch ein Add-On zaubern, bzw. die Anleitung dafür kopieren und in der modcp.php habe ich noch eine kleinen Fehler gefunden, den ich dann gleich mit ausbügeln werde.
Die neuen Versionen packe ich dann nachher hier an, wenn ich OXPUS.de auch online aktualisiert habe.
Also bis nachher...

EDIT2
So, der MOD ist nun auch hier online und anbei die ergänzte Version für installierte Split Topic Type MODs.
Sind nur 2 Dateien des besagten MODs, die geändert werden müssen, dafür die viewforum_body.tpl nicht mehr

[tom10]

Ich erhalte eine Fehlermeldung:

Could not get post ID

DEBUG MODE

SQL Error : 1054 Unknown column 't.topic_activate' in 'where clause'

SELECT MAX(p.post_id) AS last_post, COUNT(p.post_id) AS total FROM plus_posts p, plus_topics t WHERE p.forum_id = 11 AND p.topic_id = t.topic_id AND t.topic_activate = 1 AND post_activate = 1

Line : 104
File : functions_admin.php

Und die zeilen um die 104 sehen so aus:

Code: Alles auswählen

	  	case 'forum':
			$sql = "SELECT MAX(p.post_id) AS last_post, COUNT(p.post_id) AS total 
				FROM " . POSTS_TABLE . " p, " . TOPICS_TABLE . " t
				WHERE p.forum_id = $id
				AND p.topic_id = t.topic_id
				AND t.topic_activate = 1
				AND post_activate = 1";
			if ( !($result = $db->sql_query($sql)) )
			{
				message_die(GENERAL_ERROR, 'Could not get post ID', '', __LINE__, __FILE__, $sql);
			}

			if ( $row = $db->sql_fetchrow($result) )
			{
				$last_post = ( $row['last_post'] ) ? $row['last_post'] : 0;
				$total_posts = ($row['total']) ? $row['total'] : 0;
			}
			else
			{
				$last_post = 0;
				$total_posts = 0;
			}

Und ein 2tes Problem tut sich auf:

Could not get topic information!

DEBUG MODE

SQL Error : 1054 Unknown column 't.topic_activate' in 'field list'

SELECT DISTINCT t.topic_id, t.topic_title, t.topic_activate, p.topic_id, p.post_activate, f.forum_id, f.forum_name FROM plus_topics t, plus_posts p, plus_forums f WHERE t.topic_id = p.topic_id AND t.forum_id = f.forum_id AND p.post_activate = 0 ORDER BY f.forum_id ASC

Line : 1431
File : modcp.php

Das sind dann die zeilen:

Code: Alles auswählen

			}
			$db->sql_freeresult($result);

			$sql = "SELECT DISTINCT t.topic_id, t.topic_title, t.topic_activate, p.topic_id, p.post_activate, f.forum_id, f.forum_name
				FROM " . TOPICS_TABLE . " t, " . POSTS_TABLE . " p, " . FORUMS_TABLE . " f
				WHERE t.topic_id = p.topic_id
				AND t.forum_id = f.forum_id
				AND p.post_activate = 0
				ORDER BY f.forum_id ASC";

			if ( !($result = $db->sql_query($sql)) )
			{
				message_die(GENERAL_ERROR, 'Could not get topic information!', '', __LINE__, __FILE__, $sql);
			}

			$old_forumname = "";

[oxpus]

Hast Du die SQL-Anweisungen ausgeführt?

[tom10]

Kannst Du mir bitte mal eine Rückgängikeits SQL Anweisung erstmal hierfür schreiben :

ALTER TABLE `phpbb_posts` ADD `post_activate` TINYINT( 1 ) NOT NULL DEFAULT '1';
ALTER TABLE `phpbb_users` ADD `user_allow_showguestposts` TINYINT( 1 ) NOT NULL DEFAULT '1' AFTER `user_allow_viewonline`;
INSERT INTO `phpbb_config` ( `config_name` , `config_value` ) VALUES ('guestposts', '1');

[oxpus]

Entferne diese Felder in der pphbb_users doch einfach mittels phpmyadmin und den Datensatz aus der phpbb_config ebenfalls ...

[tom10]

O.K. hab es raus bekommen und nun hab ich die neue SQl Anweisung ausgeführt.
Ich hatte die alten SQL Daten noch drin. Nun scheint erstmal keine Fehlermeldung zu kommen.
Ich teste mal weiter... Wenn Du auch mal schauen willst: www.ford-welt.de

[Bootenks]

Freut mich dass er scheinbar funktioniert ^^ *auch auf Oxpus tatkräftiger Unterstützung*

[tom10]

Also so weit ich das nun getestet hab funktioniert der Mod richtig gut !
Respekt an euch beide !
Ein paar kleine "Verbesserung" hätte ich noch als Vorschlag.
Zum einen wäre es richtig gut wenn man direkt nach der aktivierung des Beitrages eine kleine zwischenanzeige bekommen könnte wo z.b. drin Steht "Der oder die Beiträge wurden erfolgreich bearbeitet" oder so änlich, denn der springt ja gleich zum Fenster mit (wie bei mir) "Keine zu aktivierenden nachrichten" ! Das verwirrt ein wenig. Man denkt es hat was nicht geklappt und die Nachricht ist weg.
2. Kann man es vielleicht so regeln das Mods und Admins im header z.b. ein blinkendes Icon oder Text dort eingebaut bekommen wenn Gästepostings generell noch freizuschalten sind?
So in der Art wie bei neuen privaten PM`s....

[oxpus]

@Bootenks
Danke danke...

@tom10
1. Hm, muss Bootenks entscheiden. Ich persönlich finde das überflüssig.
2. Da würde ich auch vorschlagen, hier eine Anzeige im Board-Kopf oder Fuß zu hinterlegen, in der steht, daß noch Topics zu aktivieren sind. Wäre ja nur eine kleine Zählung...

[tom10]

Ich frage euch gleich noch mal was...
Und zwar wegen der viewforum_body.tpl dort hab ich das so gemacht... ist das dort richtig ?

Code: Alles auswählen

{BOARD_INDEX}
{TOPICS_LIST_BOX}
<table border="0" cellpadding="0" cellspacing="0" class="tbl"><tr><td class="tbll"><img src="images/spacer.gif" alt="" width="8" height="4" /></td><td class="tblbot"><img src="images/spacer.gif" alt="" width="8" height="4" /></td><td class="tblr"><img src="images/spacer.gif" alt="" width="8" height="4" /></td></tr></table>
<table width="100%" cellspacing="2" cellpadding="2" border="0">
<tr>
<td><a href="{U_POST_NEW_TOPIC}"><img src="{POST_IMG}" alt="{L_POST_NEW_TOPIC}" title="{L_POST_NEW_TOPIC}" /></a></td>
{topicrow.TOPIC_GUEST_POST}
<td align="left" valign="middle" class="nav" width="100%"><span class="nav"><a href="{U_INDEX}" class="nav">{L_INDEX}</a>{NAV_CAT_DESC}</span></td>
<td nowrap="nowrap" class="nav">{PAGINATION}</td>
</tr>
</table>
<table width="100%" border="0" cellspacing="0" cellpadding="2">
<tr>
<td><br />{JUMPBOX}</td>
<td class="gensmall" align="right" valign="top"><strong>{U_MARK_READ}</strong><br />
{L_MODERATOR}: {MODERATORS}<br />
{TOTAL_USERS_ONLINE}<br/>{LOGGED_IN_USER_LIST}
</td>
</tr>
</table>

Laut Anweisung ist es ja so zu machen :

#
#-----[ FIND ]------------------------------------------------
#
<td class="row1" width="100%"><span class="topictitle">{topicrow.NEWEST_POST_IMG}{topicrow.TOPIC_TYPE}<a href="{topicrow.U_VIEW_TOPIC}" class="topictitle">{topicrow.TOPIC_TITLE}</a></span><span class="gensmall"><br />

#
#-----[ IN-LINE FIND ]------------------------------------------------
#
<td class="row1" width="100%" valign="middle"><span class="topictitle">

#
#-----[ IN-LINE AFTER, ADD ]------------------------------------------------
#
{topicrow.TOPIC_GUEST_POST}

[Bootenks]

zu 1. Finde ich auch albern, da jedem der einen Post aktiviert hat doch klar sein sollte, dass dieser dann nicht mehr in der Liste steht...

zu 2. ja ist denkbar und sicherlich auch nicht schwer... nur wo sollte es blinken?

@tom10: zu deiner letzten Frage: Ja sieht gut aus wenn es da keine grafischen Probleme gibt... kenn jetzt deine Tabellenstruktur nicht...

[oxpus]

Ich würde den Link irgendwo unterhalb des "Default" Boardmenüs einstellen.
Blinken muss es ja noch nicht einmal.
Oder, besser vielleicht neben oder unter-/oberhalb des Admin-Links im Fuß.
Letzteres wäre vielleicht wirklich die beste Lösung, denn dann kann man in der page_tail.php zählen lassen, die Seite ist bis dahin dann schon aufgebaut und dennoch wird angezeigt, daß noch Gastposts zu aktivieren sind.
Als Link, versteht sich...