Gäste - Post - Freischaltung

[Bootenks]

zu 2.1: das hab ich schon umgebaut funktioniert auch *grins*

zu deinen Edits: was meinst du mit dem 1 und 2ten???

Edit:

Habe den Mod geupt im Zuge dessen habe ich auch gleichzeitig die Category Hierarchy 2.1.x Kompatiblität vollzogen!

Beide Versionen sind im ersten Post zu finden.

zum Update von 0.0.1: man muss die modcp.php Änderungen komplett ersetzen und die 2 fehlenden Dateien admin/admin_users.php sowie templates/subSilver/admin/user_edit_body.tpl noch bearbeiten!

@ Oxpus weiss immernoch nicht was du meinst Oo

[oxpus]

Wenn ein Gast ein Topic erstellt, werden nach der Installation dieses MODs zwar der Post und Post-Text erstellt (inaktiv), aber das Topic nicht.
Ich teste das nun morgen mit der neuen Version und dann sehe ich weiter...

EDIT
Weiterhin kein Erfolg:
Wenn ein Gast ein neues Topic erstellt, werden in der Datenbank nur der Post und Posttext in die betreffenden Tabellen geschrieben, das Topic nicht.
Somit ist das Topic nirgends zu sehen...

@Bootenks
Teste das mal selber, ob Du den gleichen Fehler hast. Ich kann so das Topic logischerweise nicht mehr wiederfinden, da es schlicht fehlt...

[Bootenks]

achso stimmt... habe gar nicht dran gedacht, dass ein Gast ja auch ein Topic erstellen kann *grins*

Werde mich da mal dransetzen und schauen wie ich das lösen kann. Dank dir schon einmal für den Tipp! ^^

[oxpus]

Okay, danke.

[Bootenks]

So hab mir dazu ne Lösung einfallen lassen:

dazu die functions_admin.php Änderungen nochmal neu so ändern:

Code: Alles auswählen

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

# 
#-----[ FIND ]------------------------------------------------ 
# 
		case 'topic':

# 
#-----[ AFTER, ADD ]------------------------------------------------ 
# 
			$sql = "SELECT topic_id, topic_poster
				FROM " . TOPICS_TABLE . "
				WHERE topic_id = $id";

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

			if ( $row = $db->sql_fetchrow($result) ) $poster_id = $row['topic_poster'];

			if ( $poster_id != -1 ) $sql_line = 'AND post_activate = 1';

# 
#-----[ FIND ]------------------------------------------------ 
# 
			$sql = "SELECT MAX(post_id) AS last_post, MIN(post_id) AS first_post, COUNT(post_id) AS total_posts
				FROM " . POSTS_TABLE . "
				WHERE topic_id = $id";

#
#-----[ REPLACE WITH ]------------------------------------------
#
			$sql = "SELECT MAX(post_id) AS last_post, MIN(post_id) AS first_post, COUNT(post_id) AS total_posts
				FROM " . POSTS_TABLE . "
				WHERE topic_id = $id
				$sql_line";

und noch ne Änderung in der viewforum

Code: Alles auswählen

# 
#-----[ OPEN ]------------------------------------------------ 
# 
viewforum.php

# 
#-----[ FIND ]------------------------------------------------ 
# 
//
// Grab all the basic data (all topics except announcements)
// for this forum
//

# 
#-----[ AFTER, ADD ]------------------------------------------------ 
# 
if ( $userdata['user_level'] == ADMIN || $is_auth['auth_mod'] )
{
	$where_sql = "";
}
else if ( $board_config['guestposts'] == 1 && $userdata['user_allow_showguestposts'] == 1 ) 
{
	$where_sql = " AND p.post_activate = 1 ";
}
else
{
	$where_sql = " AND p.poster_id > -1 ";
}

# 
#-----[ FIND ]------------------------------------------------ 
# 
		AND t.topic_poster = u.user_id
		AND p.post_id = t.topic_first_post_id
		AND p2.post_id = t.topic_last_post_id
		AND u2.user_id = p2.poster_id 
		AND t.topic_type <> " . POST_ANNOUNCE . "

# 
#-----[ AFTER, ADD ]------------------------------------------------ 
# 
		$where_sql 

und zusätzlich noch diese kleine Änderung in der viewtopic.php

Code: Alles auswählen

# 
#-----[ FIND ]------------------------------------------------ 
# 
   include($phpbb_root_path . 'includes/functions_admin.' . $phpEx);
   sync('topic', $topic_id);

# 
#-----[ BEFORE, ADD ]------------------------------------------------ 
# 
   if ( $gp != 0 )
   {

# 
#-----[ AFTER, ADD ]------------------------------------------------ 
# 
   }

Bau bei dir das mal ein Oxpus. Und berichte mir hoffentlich Gutes

Anmerkung: Werde das noch nicht uppen... erst wenn Oxpus das ok dafür gibt ansonsten warte ich darauf wo es noch Fehler kommen.

[tom10]

Wenn Oxpus das o.K. gibt dann uppe das schnell neu, damit ich das auch einbauen kann und dann teste ich wieder mit, denn das Was Oxpus geschrieben hat war ja auch bei mir das Problem! Das hab ich aber schon angesprochen. Ich dachte es hinge mit dem Ch zusammen.

[oxpus]

Schau ich mir heute Nachmittag an. Mein Testboard ist offline (lokal installiert) und ich sitze ja aktuell nicht zu Hause

Aber Danke erstmal für die Änderungen.
Auf den ersten Blick könnte es so funktionieren.
Hier ist aber auch immer noch keine Anpassung für den CH MOD 2.0.x nötig und auch der Split Topic Type MOD (der ja einen Teil der viewforum.php ersetzt) muss nicht angefasst werden. Soviel auch zu den Kompatibilitäten.
In wie weit der UPI2DB hier berücksichtigt werden muss, sehe ich nachher.
Ich denke aber, daß der problemlos hier mitspielt...

@Bootenks
Du hast hier nicht berücksichtigt, daß Ankündigungen ggf. auch von Gästen gepostet werden könnten (Wer das so einstellt, müsste allerdings erschossen werden ). Also ist die 1. Abfrage zu den Topics auch zu berücksichtigen.
Und in der Anleitung für die viewtopic.php hast Du Teile des Codes in der Reihenfolge verdreht. Schau das mal an, man muss nach der ersten Änderung wieder mehr in Richtung Anfang der Datei springen, anstelle wie bei MODs üblich weiter voran...

[Bootenks]

oje das liegt daran dass ich Korrekturen eingebaut habe und die Chronologie wohl nicht beachtet habe ^^... naja das sind ja noch kleine Fische... aber das mit der Ankündigung stimmt natürlich wieder ~.~ Ich sag ja der Mod expandiert immer weiter... Achja mir wäre lieb dass einer mal testet ob die Änderungen der search.php korrekt sind, denn da gabs einige Stellen an den ich nicht genau wusste wann diese eintreten... habe diese deshalb provisorisch auch abgeändert... *thanks* => Änderungen kommen heute Abend wieder denke ich mal

Edit: wenn du Xampp hast kannst du es ja auch auf einem Vanilla phpBB testen ^^ würde mir auf jedenfall weiterhelfen

[oxpus]

Also ich teste immer mit einem vanilla phpBB.
Keine Sorge

Die search.php arbeitet auch hervorragend und korrekt.
Wie schon gesagt:
Offen sind jetzt die Topic-Erstellung durch Gäste (hast Du ja bereits was gepostet) und die Erstellung/Anzeige von Gast-Ankündigungen (kannst Du ja schnell in der Anleitung ergänzen).
Der Rest arbeitet bereits fehlerlos!

Mit den Topics teste ich nachher und baue dazu Deine letzte Änderung ein. Das Ergebnis folgt dann umgehend wieder hier.
Aber: Mach Dir keinen Stress. Auf einen Tag oder 2 kommt es jetzt wirklich nicht an!

EDIT
Also das neue Gast-Topic wird nun erfolgreich mit gespeichert, aber:
Man kann es nicht aktivieren, da man es ja im Forum nirgends sieht
Hier sollte die neue Konsole auch diese Topics anzeigen können, also ein Aufruf für das gesamte Forum möglich sein und nicht nur für das Topic alleine...
Wir nähern uns aber deutlich der Perfektion

EDIT2
Anbei auch mal der erweiterte Fix für die Einbindung von Gast-Ankündigungen:

[ Attachment gelöscht am Fr 24.Feb, 2006 08:51 von oxpus ]

[Bootenks]

Anmerkung:
irgednwas gefällt mir an dem Mod noch nicht aber ich weiss noch nicht was ~.~ hab das Gefühl der Inkonsitenz der Daten ~.~ (irgendwas ändere ich noch in der functions_post.php bzw. hab ich schon geändert)

Also durch meine Änderungen sieht der Admin und die entsprechenden Mods den Topic... dann können die da rein und dort den Post aktivieren und somit ist dann das Topic freigegeben Oo oder wie meinst du das?

für den Fix thanks ^^

[oxpus]

Öh, ich sehe genau diesen Topic eben nicht...
Also:
Ich habe als Gast ein neues Topic erstellt.
In der Datenbank werden Topic, Post und Post-Text angelegt.
Soweit okay.
Aber als Admin wird mir genau dieses Topic nicht angezeigt.
Weder im Forum (wo es als Gasttopic ja zumindest stehen sollte), noch irgendwo anders...
Selbst, wenn ich Topic und Post manuell in der DB auf User-ID 2 setze und aktiviere, bleibt es verschwunden.
Ich werde mal mein Testforum online stellen, dann könnt Ihr selber testen.
Dauert einen Moment...

EDIT
Testforum ist nun unter http://test.oxpus.de zu erreichen.
Anmeldung erfolgt ohne nötige Aktivierung per Email.
Aber bitte an der Gruppe "Moderation" anmelden, um die Funktion testen zu können.
Gäste haben komplette Ansicht- und Post-Rechte.
Spam ist hier ausnahmsweise mal erlaubt

EDIT2
@Bootenks
Das Gast-Topic ist auch hier nun vorhanden. Keine Chance, das zu sehen.
Wenn Du eine neue Konsole hättest, wäre das was anderes...

EDIT3
Oh, auf dem Index als Letztes Topic von Gast "TEST" ist es zu sehen, kann aber nicht geöffnet werden...

[Bootenks]

ja merk es auch gerade und hab schon eine pfiffige Idee wie ich es hinbekomme und dann auch noch glücklich bin aber das dauert ein stück weil ich den Code erstmal austüfteln muss :'(

Edit: So müsste es klappen ~.~

prüfe nochmal die Änderungen der functions_post.php, da müssen einige abgeändert werden

Edit: anbei die neue Version (aber eher nur so für tester nicht offiziell)

[oxpus]

Beinahe

Also:
Die Gast-Topics werden angezeigt.
Soweit schon einmal okay.
Aber beim Löschen kommen Fehlermeldungen.
Ich hänge mal die functions_post.php anbei, damit Du sehen kannst, wo die stecken.
Du hast auf den ersten Blick gesehen nicht geprüft, wo wann die $forum_update_sql bereits gefüllt ist und wo nicht...
Kleinigkeit, denke ich, aber dann wäre der MOD wirklich final, würde ich mal sagen.
Okay das MOD-Image für die Konsole und ein "Alles markieren :: Keine Markieren" für die Auswahl der zu aktivierenden Posts wäre auch ganz nett. Und natürlich auch deutsch...

Dazu habe ich die erste Bedingung in der update_post_stats(); mal etwas angepasst. Z. B. heissen Gäste ANONYMOUS und nicht GUEST in einem phpBB...

[ Attachment gelöscht am Fr 24.Feb, 2006 08:50 von oxpus ]

[Bootenks]

wie mir das da reingekommen ist weiss ich auch nicht ^^ ansonsten verwende ich ja immer Anonymous ^^ aber danke *hoffentlich ist es bald wirklich final* ^^

EDIT:
das dürfte es sein:

functions_post.php

Code: Alles auswählen

# 
#-----[ FIND ]------------------------------------------------ 
# 
	if ($mode == 'delete')
	{

# 
#-----[ AFTER, ADD ]------------------------------------------------ 
# 
		$set_comma = 0;

# 
#-----[ FIND ]------------------------------------------------ 
# 
				$forum_update_sql .= ', forum_topics = forum_topics - 1';

#
#-----[ REPLACE WITH ]------------------------------------------
#
				if ( $post_activate == 0  )
				{
					$sql_line = "";
				}
				else
				{
					$sql_line = ", ";
					$set_comma = 1;
				}
				$forum_update_sql .= $sql_line . 'forum_topics = forum_topics - 1';

# 
#-----[ FIND ]------------------------------------------------ 
# 
					$forum_update_sql .= ($row['last_post_id']) ? ', forum_last_post_id = ' . $row['last_post_id'] : ', forum_last_post_id = 0';

#
#-----[ REPLACE WITH ]------------------------------------------
#
					$sql_add = (( $set_comma == 0 && $post_activate == 0 ) ? "" : ", ");
					$forum_update_sql .= ($row['last_post_id']) ? $sql_add . 'forum_last_post_id = ' . $row['last_post_id'] : ', forum_last_post_id = 0';

die anderen Spielereien bau ich auch ein ^^

[oxpus]

Nicht aktivierte Beiträge bringen hier immer noch eine Fehlermeldung. Zeile 487.
Wieder ein fehlendes Komma.
Aktivierte Beiträge lassen sich hingegen fehlerfrei löschen...

Error in posting

DEBUG MODE

SQL Error : 1054 Unknown column '1forum_last_post_id' in 'field list'

UPDATE phpbb_forums SET forum_topics = forum_topics - 1forum_last_post_id = 13 WHERE forum_id = 1

Line : 487
File : functions_post.php

[Bootenks]

komisch bei mir geht es Oo habe alle Varianten durchgespielt Oo

anbei mal das neue Werk mit Mark_all/Umark_all ...

achso wäre glücklich wenn mir mal einer so ein Bildchen machen würde ^^ und der Name wäre auch noch wichtig!

[tom10]

Ich werde das mal eben einbauen. Wie wäre es mit dem Namen guest_posting_controll oder so ?

[oxpus]

guest_posting_controll

Klingt nicht schlecht.

EDIT
Ich habe die Fehler in der functions_post.php behoben und die Anleitung mal insgesamt korrigiert.
Auch ist nun ein "provisorisches" Image vorhanden, das ersetze ich aber noch durch ein besseres...
Bitte mal alle testen, ob diese Version nun wirklich korrekt ist.
Danke.

[Bootenks]

Habs mal frisch in ein Vanilla eingebaut ^^ Sieht im Moment alles gut aus. Habe gleichzeitig geprüft was du geändert hast. Du magst mein RELPACE nicht gerne sehen ^^ Verstehe ich auch! Liegt wohl daran, dass andere Mods dadurch schnell aus dem Ruder geraten könnten. Zu meiner Verteidigung ist zu sagen, dass ein REPLACE beim Erstellen der Install schnell geht als ein IN-LINE FIND etc.. *grins* Also hast du mehr Textkosmetik gemacht oder??? Das einzigste was mir noch aufgefallen ist, ist dass du das umständliche $set_comma mit einer einfachen Variablenabfrage (biste gefüllt ja oder nein ersetzt hast) ! Ist auch leichter warum ich gestern nicht draufgekommen bin weiss ich nicht... war wohl der Suff ~.~

Ansonsten danke ich dir für dein erfahrendes Auge. Ich hoffe ich komme bald mit diesem INSTALL Geschreibsel auf einen grünen Ast und seh es bald nimmer als meinen größten Erzfeind an. *grins*

[oxpus]

Du magst mein RELPACE nicht gerne sehen

Nicht ich, sondern laut den Vorgaben der phpbb.com-Group ist sowas nicht besonders klug und das kann ich auch verstehen:

Was passiert, wenn ein User eine Abfrage bereits durch einen anderen MOD geändert hat?
Er kann diese Abfrage laut Deinen Angaben nur noch schwer finden und schon garnicht so ohne weiteres komplett ersetzen.
Daher sind die IN-LINE-Befehle da, die eben hier nur einen Teil ersetzen.

Das hat nichts mit mögen oder nicht mögen zu tun. Auch ich würde lieber ein komplettes REPLACE WITH verwenden, aber wie gesagt: Man muss immer an gemoddete Foren denken.
Bau mal aus Spaß Deinen MOD in ein Plus ein. Viel Spaß dabei

Im Ernst:
Die Mimik in der functions_post.php war nett gedacht, brachte aber einige Verwirrung in das Script, so daß weiterhin Fehler beim Löschen angezeigt wurden.
Daher habe ich hier auf die bewährte Methode zurückgegriffen, die da heisst:
Bist Du leer, dann ohne Komma, ansonsten mit Komma voran.
Klappt in 99,99% der Fälle immer.
Das hatte ich allerdings beim ersten Mal auch versucht, nur aber nicht, wie ich vorhin bemerkte, alle Stellen berücksichtigt und daher "Alarm" geschlagen.
Sorry, für die dadurch entstandene Mehrarbeit. Für 2 Minuten zu wenig Nachdenken meinerseits...

Aber schön, wenn der MOD nun funktioniert. Wenn ich nachher noch ein vernünftiges Icon habe, poste ich das hier, damit der MOD endlich abgeschlossen werden kann und auf OXPUS.de ebenfalls Einzug halten kann.
Vielleicht ergeben sich dann noch so 2-5000 Add-Ons für die diversen Ansichten, wie z. B. die letzten Themen auf dem Portal oder die recent.php.
Mal schauen...

[Bootenks]

Ja ich denke auch, dass das hier noch ein Riesenteil wird! *grins* Den Add-Ons steht hier nichts mehr im Weg.

Edit Habe aber gerade noch einen kleinen Fehler gefunden. Ich denke bis heute Abend dürfte die erste Final draussen sein. Nochmals ein großes Dankeschön an dich Oxpus für deine Mithilfe.

[oxpus]

Keine Ursache.
Ich bin ja schliesslich auch stark an dem MOD interessiert...

[Bootenks]

Mal eine andere Frage im Moment ist es ja so, dass wenn im ACP eingestellt wurde, dass die Gästebeiträge nicht ein- und ausgeschaltet werden dürfen, die Gästebeiträge nicht angezeigt werden. (nur Mods und Admins sehen die dann) Ist dies sinnvoll und wenn nein macht es dann nicht mehr Sinn da noch eine zweite Option einzuführen, die dann eine Standardeinstellung (Gästeposts immer anzeigen oder eben nie), die für alle gilt, vorgibt? Ansonsten habe ich die Fehler die ich noch gefunden hatte gefixt (war ein Sprachfehler *grins* und eine sinnlose $lang Variable, sowie eine fehlende Änderung in der functions_admin.php)!

Edit: Habe noch einen ganz bösen Fehler entdeckt ~.~ also den zu sehen war schon eine Kunst ~.~ Mal sehen wie ich ihn "vernichten" werde *grins*

[tom10]

Ups, und ich hab das gerade fertig eingebaut und nun...warte ich lieber auf das erste relace.
Was vielleicht nicht schlecht wäre, ist wenn ein Gast was gepostet hat das dem dann nicht ngezeigt wird, das das Posting erfolgreich eingetragen wurde, sondern das das posting erst vom Mod oder Admin für die öffentlichkeit freigeschalten werden muß um sichtbar zu werden.

[Bootenks]

ja seit deinem Einbau sind schon ne Menge Korrekturen abgearbeitet worden :P Trägt ja nicht ohne Grund den Beta Stempel. Aber zur Beruhigung... Ansich musste nur die functions_post.php auf Änderungen prüfen... Warte aber wirklich lieber aufs Final und bau ihn dann ordentlich ein... ab dann mach ich auch für Updates die nötiigen update.txt

Edit: vorhin genannter Fehler behoben ^^ (nun warte ich noch auf das Icon und auf die Antwort von Oxpus zu meiner zuvor gestellten Frage!) => dann kommt das Release