phpBB.de

Verfasst: **07.06.2007 17:12**

hanker hat geschrieben:
S2B hat geschrieben:Danke für die Icons, ich werde sie beim nächsten Update reinbauen.
Wäre nett, wenn du meinen Namen o.ä. am Rande erwähntest...

Kann ich tun, deinen Namen habe ich übrigens auch im Newseintrag auf meiner Website erwähnt.

Verfasst: **08.06.2007 13:04**

Bin noch nicht dazu gekommen, die modcp.php zu kontrollieren, habe aber ein weiteres Problemchen bei mir (und nur mir?) entdeckt.

Wenn ich ein Posting von mehreren innerhalb auf eine Umfrage archiviere, und dann lösche, erhalte ich ein:

Code: Alles auswählen

Error in deleting poll

DEBUG MODE

SQL Error : 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 2

DELETE FROM phpbb_vote_results WHERE vote_id =

Line : 534
File : functions_post.php

Erstelle ich eine neue Umfrage ohne Folgeposting, archiviere und lösche das dann einzige Topic, tritt der Fehler nicht auf.

Archiviere und lösche ich eine komplette Umfrage auch mit Folgeposting, tritt der Fehler ebenso nicht auf.

Ich denke mal, phpBB löscht bei nur einem geposteten Topic immer gleich den ganzen Thread. Ergo tritt das Problem beim Löschen einzelner Umfragepostings auf. Wenn ich das Archivieren abschalte, habe ich übrigens keine Probleme.

René

Verfasst: **08.06.2007 16:43**

Oha, interessant.

Danke für den Hinweis.

Hier mal eine vorläufige Lösung, wahrscheinlich wird die endgültige Lösung auch so aussehen (ich kann aber für nichts garantieren):
Finde in der includes/functions_archive.php: (Zeile 471)

Code: Alles auswählen

		$topic_data['topic_data']['topic_last_post_id'] = max($topic_data['post_ids']);

Füge danach ein:

Code: Alles auswählen

		if ($mode == 'posts')
		{
			$topic_data['topic_data']['topic_vote'] = 0;
		}

Bei der Untersuchung dieses Fehlers bin ich gleich noch auf einen anderen gestoßen:
Finde: (Zeile 879)

Code: Alles auswählen

			$topic_data['topic_data']['topic_last_post_id'] = max($topic_data['topic_data']['topic_last_post_id'], $topic_stat['topic_last_post_id']);

Füge danach ein:

Code: Alles auswählen

			if ($topic_stat['topic_vote'])
			{
				$topic_data['topic_data']['topic_vote'] = 1;
			}

Edit: Es gibt zu jedem Fix noch eine zweite genau identische Stelle, an der das gleiche nochmal ausgeführt werden muss.

Verfasst: **09.06.2007 00:33**

S2B hat geschrieben:Oha, interessant. Danke für den Hinweis.

Jederzeit zum Bug-Finden bereit!

Darf man Fragen, wofür die zwei Stellen zuständig waren?

René

PS: Und *DANKE* für den Fix, der Fehler wirkt behoben. (Werde dann auch mal endlich meine modcp.php überprüfen.)

Verfasst: **09.06.2007 12:39**

S2B hat geschrieben:Überprüfe doch mal die Änderungen an der modcp.php, vor allem nach dem case 'move': und case 'split':.

Habe das gerade getan. Alles ist so, wie in der Anleitung beschrieben. Nun bin ich verwirrt (also, endgültig, jetzt

).

Ich habe meine modcp.php mal hier raufgeladen:

http://www.rpg-domain.de/temp/modcp.php.txt

René

EDIT: Da sind natürlich noch die von mir oben geposteten Veränderungen drinnen, aber die schalten ja nur die Anzeige der buttons für lock/unlock, move und split aus.

Verfasst: **09.06.2007 13:27**

hanker hat geschrieben:Darf man Fragen, wofür die zwei Stellen zuständig waren?

Die erste Änderung sorgt dafür, dass beim Archivieren/Wiederherstellen (=> zwei Stellen) von einzelnen Beiträgen der topic_vote-Status des Themas auf 0 gesetzt wird, da die Umfrage ja noch beim Rest des Themas ist.

Die zweite Änderung sorgt dafür, dass der Vote-Status beim Archivieren/Wiederherstellen beibehalten wird, wenn eins der beiden Themen, die ja dann quasi zusammengeführt werden müssen, den Vote-Status 0 hat.

Deine Datei schaue ich mir bei Gelegenheit mal an, allerdings kann das ein bisschen dauern, da übermorgen die Schule wieder losgeht.

Verfasst: **09.06.2007 13:56**

Hat a bisserl Zeit, ich bin eh gerade im Noten-Verteilungs-Stress.

René

PS: Dann werde ich Wahnsinniger mich mal drann begeben, und zwischendurch ein paar weitere MODs installieren.

Verfasst: **11.06.2007 15:04**

S2B hat geschrieben:Hier mal eine vorläufige Lösung, wahrscheinlich wird die endgültige Lösung auch so aussehen (ich kann aber für nichts garantieren):

Version 2.0.3 ist oben, die Stellen sind die gleichen geblieben. Ich habe außerdem die neuen Bilder hinzugefügt und die Sache mit dem Thema verschieben gelöst.

Verfasst: **11.06.2007 18:10**

Könntest du etwas genauer werden, was genau bezüglich des Verschiebens geändert wurde? Ich habe ja derzeit eine "Version 2.5", und da gestaltet sich ansonsten ein Update vielleicht schwierig.

René

Verfasst: **11.06.2007 19:06**

hanker hat geschrieben:Könntest du etwas genauer werden, was genau bezüglich des Verschiebens geändert wurde? Ich habe ja derzeit eine "Version 2.5", und da gestaltet sich ansonsten ein Update vielleicht schwierig.

Schau mal in den Changelog, es geht nur um die Ausblendung des Verschieben-Symbols, wenn nur ein Archiv-Forum vorhanden ist.

phpBB.de

Archive MOD 2.0.1 viewtopic.php