Archive MOD 2.0.1 viewtopic.php

hanker · Beitrag von **hanker** » 06.06.2007 20:23

Beim Archive MOD 2.0.1 hat sich noch mindestens ein weiteres Problem eingeschlichen, und zwar wird beim Anschauen eines Topics im Archiv auch "Verschieben", "Entsperren" und "Teilen" angezeigt, die ohne Funktion oder ohne sinnvolle Funktion sind. Abhilfe tut das hier:

Ersetze

Code: Alles auswählen

if ( $is_auth['auth_mod'] )
{
	$s_auth_can .= sprintf($lang['Rules_moderate'], "<a href=\"modcp.$phpEx?" . POST_FORUM_URL . "=$forum_id&sid=" . $userdata['session_id'] . '">', '</a>');

	if (!$forum_topic_data['forum_enable_archive'] || $forum_topic_data['forum_status'] == FORUM_ARCHIVE)
	{

	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=delete&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_delete'] . '" alt="' . $lang['Delete_topic'] . '" title="' . $lang['Delete_topic'] . '" border="0" /></a>&nbsp;';
	}
	else
	{
		$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=archive&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_archive'] . '" alt="' . $lang['Archive_topic_title'] . '" title="' . $lang['Archive_topic_title'] . '" border="0" /></a>&nbsp;';
	}

	if ($forum_topic_data['forum_status'] == FORUM_ARCHIVE && count($archive_info) > 0)
	{
		$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=restore&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_restore'] . '" alt="' . $lang['Restore_topic_title'] . '" title="' . $lang['Restore_topic_title'] . '" border="0" /></a>&nbsp;';
	}


	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=move&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_move'] . '" alt="' . $lang['Move_topic'] . '" title="' . $lang['Move_topic'] . '" border="0" /></a>&nbsp;';

	$topic_mod .= ( $forum_topic_data['topic_status'] == TOPIC_UNLOCKED ) ? "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=lock&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_lock'] . '" alt="' . $lang['Lock_topic'] . '" title="' . $lang['Lock_topic'] . '" border="0" /></a>&nbsp;' : "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=unlock&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_unlock'] . '" alt="' . $lang['Unlock_topic'] . '" title="' . $lang['Unlock_topic'] . '" border="0" /></a>&nbsp;';

	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=split&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_split'] . '" alt="' . $lang['Split_topic'] . '" title="' . $lang['Split_topic'] . '" border="0" /></a>&nbsp;';
}

durch

Code: Alles auswählen

if ( $is_auth['auth_mod'] )
{
	$s_auth_can .= sprintf($lang['Rules_moderate'], "<a href=\"modcp.$phpEx?" . POST_FORUM_URL . "=$forum_id&sid=" . $userdata['session_id'] . '">', '</a>');

	if (!$forum_topic_data['forum_enable_archive'] || $forum_topic_data['forum_status'] == FORUM_ARCHIVE)
	{

	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=delete&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_delete'] . '" alt="' . $lang['Delete_topic'] . '" title="' . $lang['Delete_topic'] . '" border="0" /></a>&nbsp;';
	}
	else
	{
		$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=archive&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_archive'] . '" alt="' . $lang['Archive_topic_title'] . '" title="' . $lang['Archive_topic_title'] . '" border="0" /></a>&nbsp;';
	}

	if ($forum_topic_data['forum_status'] == FORUM_ARCHIVE && count($archive_info) > 0)
	{
		$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=restore&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_restore'] . '" alt="' . $lang['Restore_topic_title'] . '" title="' . $lang['Restore_topic_title'] . '" border="0" /></a>&nbsp;';
	}
	else
	{
	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=move&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_move'] . '" alt="' . $lang['Move_topic'] . '" title="' . $lang['Move_topic'] . '" border="0" /></a>&nbsp;';

	$topic_mod .= ( $forum_topic_data['topic_status'] == TOPIC_UNLOCKED ) ? "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=lock&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_lock'] . '" alt="' . $lang['Lock_topic'] . '" title="' . $lang['Lock_topic'] . '" border="0" /></a>&nbsp;' : "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=unlock&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_unlock'] . '" alt="' . $lang['Unlock_topic'] . '" title="' . $lang['Unlock_topic'] . '" border="0" /></a>&nbsp;';

	$topic_mod .= "<a href=\"modcp.$phpEx?" . POST_TOPIC_URL . "=$topic_id&mode=split&sid=" . $userdata['session_id'] . '"><img src="' . $images['topic_mod_split'] . '" alt="' . $lang['Split_topic'] . '" title="' . $lang['Split_topic'] . '" border="0" /></a>&nbsp;';
	}
}

(Im Endeffekt muss bloß ein "else" um die letzten "$topic_mod .=" gesetzt werden.)

René

hanker · Beitrag von **hanker** » 06.06.2007 21:49

Das Problem tritt auch bei modcp_body.tpl bzw. modcp.php auf:

FALSCH (modcp_body.tpl):

Code: Alles auswählen

	<tr align="right"> 
	  <td class="catBottom" colspan="5" height="29"> {S_HIDDEN_FIELDS} 
		<!-- BEGIN switch_archive -->
		&nbsp; 
		<input type="submit" name="archive" class="liteoption" value="{L_ARCHIVE}" />
		<!-- END switch_archive -->
		<!-- BEGIN switch_delete -->
		&nbsp;

		<input type="submit" name="delete" class="liteoption" value="{L_DELETE}" />
		<!-- END switch_delete -->
 		<!-- BEGIN switch_restore -->
		&nbsp;
		<input type="submit" name="restore" class="liteoption" value="{L_RESTORE}" />
		<!-- END switch_restore -->

		&nbsp; 
		<input type="submit" name="move" class="liteoption" value="{L_MOVE}" />
		&nbsp; 
		<input type="submit" name="lock" class="liteoption" value="{L_LOCK}" />
		&nbsp; 
		<input type="submit" name="unlock" class="liteoption" value="{L_UNLOCK}" />
	  </td>
	</tr>

RICHTIG (modcp_body.tpl):

Code: Alles auswählen

	<tr align="right"> 
	  <td class="catBottom" colspan="5" height="29"> {S_HIDDEN_FIELDS} 
		<!-- BEGIN switch_archive -->
		&nbsp; 
		<input type="submit" name="archive" class="liteoption" value="{L_ARCHIVE}" />
		&nbsp; 
		<input type="submit" name="move" class="liteoption" value="{L_MOVE}" />
		&nbsp; 
		<input type="submit" name="lock" class="liteoption" value="{L_LOCK}" />
		&nbsp; 
		<input type="submit" name="unlock" class="liteoption" value="{L_UNLOCK}" />
		<!-- END switch_archive -->
		<!-- BEGIN switch_delete -->
		&nbsp;
		<input type="submit" name="delete" class="liteoption" value="{L_DELETE}" />
		&nbsp; 
		<input type="submit" name="move" class="liteoption" value="{L_MOVE}" />
		&nbsp; 
		<input type="submit" name="lock" class="liteoption" value="{L_LOCK}" />
		&nbsp; 
		<input type="submit" name="unlock" class="liteoption" value="{L_UNLOCK}" />
		<!-- END switch_delete -->
 		<!-- BEGIN switch_restore -->
		&nbsp;
		<input type="submit" name="delete" class="liteoption" value="{L_DELETE}" />
		&nbsp;
		<input type="submit" name="restore" class="liteoption" value="{L_RESTORE}" />
		<!-- END switch_restore -->
	  </td>
	</tr>

FALSCH (modcp.php):

Code: Alles auswählen

		if ($forum_status == FORUM_ARCHIVE)
		{
			$template->assign_block_vars('switch_delete', array());
			$template->assign_block_vars('switch_restore', array());
		}
		else
		{
			if (!$forum_enable_archive)
			{
				$template->assign_block_vars('switch_delete', array());
			}
			else
			{
				$template->assign_block_vars('switch_archive', array());
			}
		}

RICHTIG (modcp.php):

Code: Alles auswählen

		if ($forum_status == FORUM_ARCHIVE)
		{
			$template->assign_block_vars('switch_restore', array());
		}
		else
		{
			if (!$forum_enable_archive)
			{
				$template->assign_block_vars('switch_delete', array());
			}
			else
			{
				$template->assign_block_vars('switch_archive', array());
			}
		}

René, inzwischen schon fast angenervt

Beitrag von **S2B** » 07.06.2007 13:00

hanker hat geschrieben:Beim Archive MOD 2.0.1 hat sich noch mindestens ein weiteres Problem eingeschlichen, und zwar wird beim Anschauen eines Topics im Archiv auch "Verschieben", "Entsperren" und "Teilen" angezeigt, die ohne Funktion oder ohne sinnvolle Funktion sind.

Nix da, wenn du mehrere Archiv-Foren hast, dann kann man damit Themen zwischen Foren verschieben. Außerdem kann man damit Archiv-Themen aufteilen oder sperren/entsperren.

Ach ja: Sowas

hanker hat geschrieben:René, inzwischen schon fast angenervt

finde ich ehrlich gesagt völlig unangemessen.

Keiner zwingt dich, in deinen Augen fehlerhafte MODs zu installieren.

hanker · Beitrag von **hanker** » 07.06.2007 14:04

Siehe Smiley!

(war nicht ernst gemeint)

Wenn das Verschieben funktionieren soll, so tat's diese Option dann zumindest bei mir nicht (phpBB 2.0.22).

Ich habe das ausprobiert, es wurde kein mögliches Verschiebungsverzeichnis angezeigt (nur "Verschieben nach:", aber kein Auswahlmenü).

Das Trennen funktionierte zumindest bei mir auch nicht richtig, allerdings habe ich das auch nicht innerhalb des Archiv-Forums versucht, sondern einen Teil in ein anderes Forum verschoben. Daraufhin fehlte einer der zwei verschobenen Posts, die aneinander hingen. Als ich das Topic dann wieder archivierte, waren sie wieder da.

Vielleicht eine Sicherheitsabfrage, so dass man nur innerhalb / in Archivforen verschieben / trennen kann? Oder müsste das generell funktionieren?

Wie gesagt: Die "Bugs" (ich bin mal ganz vorsichtig) traten bei mir in meinem phpBB 2.0.22 auf. MODs, die mit deinem interferieren habe ich meines Wissens nach nicht installiert (wenn du willst, kann ich hier eine Liste posten).

Aber Danke für die Rückmeldung, wollte dir wirklich nicht auf den Schlips treten, eher nochmals VIELEN DANK für den tollen MOD!

René

EDIT: Auch das Entsperren eines Archiv-Topics funktionierte bei mir nicht. Das Topic war am Anfang gesperrt, viewtopic.php und modcp.php lieferten jedoch ein mögliches "Sperren". Wählte man dies aus, konnte man "Entsperren", aber das führte zu keiner Entsperrung des Forums.

Beitrag von **S2B** » 07.06.2007 14:11

hanker hat geschrieben:Aber Danke für die Rückmeldung, wollte dir wirklich nicht auf den Schlips treten, eher nochmals VIELEN DANK für den tollen MOD!

Na also, es geht doch.

Normalerweise sollten bei den Verschieben/Aufteilen-Dialogen nur Archiv-Foren auswählbar sein. Ich habe mir das ganze nochmal in meinem lokalen Testforum angeschaut (auch phpBB 2.0.22) und da werden, wenn ich mehrere Archiv-Foren eingestellt habe, nur Archiv-Foren in der Liste angezeigt. Wenn du natürlich nur ein Archiv-Forum eingestellt hast, wird beim Verschieben keine Auswahlbox angezeigt, das Aufteilen sollte allerdings auch nur mit einem Archiv-Forum funktionieren.

hanker · Beitrag von **hanker** » 07.06.2007 14:18

Ah! Da kommt das her! Mein Fehler (teilweise)!

Vielleicht sollte man dann für Blödians wie mich verhindern, dass man "Verschieben" bei nur einem Archivforum überhaupt auswählen kann - wenn das denn ohne größere Probleme geht!

Ansonsten: Ich konnte derzeit definitiv aufgeteilte Topics auch in Nicht-Archiv-Foren verschieben. Bei nur einem vorhandenen Archivforum.

René

PS: Vielleicht sollte man für das "Wiederherstellen" ein anderes Icon einsetzen. Dass es dasselbe wie "Verschieben" ist, irritiert (zumindest mich) ein wenig. Wenn du willst schaue ich mal, ob ich eines hinbekomme (denke da an einen Pfeil mit heller anstatt dunkler Füllung).

Beitrag von **S2B** » 07.06.2007 14:29

Ich habe gerade Version 2.0.2 hochgeladen, mir der drei deiner gemeldeten Fehler behoben wurden.

hanker hat geschrieben:Vielleicht sollte man dann für Blödians wie mich verhindern, dass man "Verschieben" bei nur einem Archivforum überhaupt auswählen kann - wenn das denn ohne größere Probleme geht!

Das gleiche Problem gibt es übrigens beim Standard-phpBB auch, wenn du nur ein Forum hast und auf Verschieben klickst. Nichts desto trotz werde ich das irgendwann verbessern (=> steht jetzt auf der ToDo-Liste).

hanker hat geschrieben:Ansonsten: Ich konnte derzeit definitiv aufgeteilte Topics auch in Nicht-Archiv-Foren verschieben. Bei nur einem vorhandenen Archivforum.

Kann eigentlich nicht sein, dann hast du etwas in der Installationsanleitung übersehen.

hanker hat geschrieben:PS: Vielleicht sollte man für das "Wiederherstellen" ein anderes Icon einsetzen. Dass es dasselbe wie "Verschieben" ist, irritiert (zumindest mich) ein wenig. Wenn du willst schaue ich mal, ob ich eines hinbekomme (denke da an einen Pfeil mit heller anstatt dunkler Füllung).

Ja, für neue Icons bin ich immer offen. Wenn du Zeit und Lust hast, kannst du gerne welche entwerfen und mir schicken.

hanker · Beitrag von **hanker** » 07.06.2007 15:14

S2B hat geschrieben:Ich habe gerade Version 2.0.2 hochgeladen, mir der drei deiner gemeldeten Fehler behoben wurden.

Danke!

hanker hat geschrieben:Ansonsten: Ich konnte derzeit definitiv aufgeteilte Topics auch in Nicht-Archiv-Foren verschieben. Bei nur einem vorhandenen Archivforum.
Kann eigentlich nicht sein, dann hast du etwas in der Installationsanleitung übersehen.

Kann eigentlich nicht sein, habe das ganze mit EaysMOD gemacht, und hinterher die offensichtlichen Fehler ausgebügelt. [räusper]

(Soll heißen: Kann durchaus sein. Dann wäre aber EasyMOD schuld.

Ja, für neue Icons bin ich immer offen. Wenn du Zeit und Lust hast, kannst du gerne welche entwerfen und mir schicken.

Da ich mich immer gerne von der Arbeit abhalten lasse:

Überarbeitete restorepost von dir (passt einen *kleinen* Tacken besser zu den offiziellen (Schatten verstärkt)):

[ externes Bild ]

Anderer Pfeil:

[ externes Bild ]

Und dasselbe für den Topic:

[ externes Bild ]

Hoffe, sie gefallen!

René

EDIT: Wenn du schon dabei bist, kannst du mir dann auch nochmal kurz sagen, wo diese Icons eigentlich eingetragen werden müssen? Wusste es gestern noch, und hab's schon wieder vergessen...

Beitrag von **S2B** » 07.06.2007 16:21

Danke für die Icons, ich werde sie beim nächsten Update reinbauen.

hanker hat geschrieben:Kann eigentlich nicht sein, habe das ganze mit EaysMOD gemacht, und hinterher die offensichtlichen Fehler ausgebügelt. [räusper]

(Soll heißen: Kann durchaus sein. Dann wäre aber EasyMOD schuld.

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

hanker hat geschrieben:EDIT: Wenn du schon dabei bist, kannst du mir dann auch nochmal kurz sagen, wo diese Icons eigentlich eingetragen werden müssen? Wusste es gestern noch, und hab's schon wieder vergessen...

templates/subSilver/subSilver.cfg

hanker · Beitrag von **hanker** » 07.06.2007 16:30

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...

Das in der modcp.php werde ich überprüfen, ansonsten *Danke* wegen des Hinweises bezüglich der subSilver.cfg. Habe die Icons bei mir eingebaut, finde, dass es gut aussieht (der "restorepost"-Pfeil könnte noch ein *etwas* anderes Blau vertragen, mal sehen, ob ich nochmal Zeit und Lust habe).

Melde mich wieder, sobald ich die modcp.php unter die Lupe genommen habe.

René

phpBB.de

Archive MOD 2.0.1 viewtopic.php

Archive MOD 2.0.1 viewtopic.php

Dasselbe Problem an anderer Stelle...

Re: Archive MOD 2.0.1 viewtopic.php