Switche

[Holger]

Hi!

an einer Stelle habe ich folgendes:

Code: Alles auswählen

            <td valign="middle" nowrap="nowrap" align="right">
<!-- BEGIN switch_user_logged_in -->
 {postrow.UPI2DB_IGNORE_USER_IMG} {postrow.HIDE_IMG} 
<!-- END switch_user_logged_in -->
{postrow.PROFILE_IMG} {postrow.PM_IMG} {postrow.EMAIL_IMG} {postrow.WWW_IMG} {postrow.USER_PIX_IMG} {postrow.ADS_IMG} {postrow.AIM_IMG} {postrow.YIM_IMG} {postrow.MSN_IMG} {postrow.PHOOGLE_IMG} {postrow.ICQ_IMG}</td> 

Es werden aber {postrow.UPI2DB_IGNORE_USER_IMG} {postrow.HIDE_IMG} nicht angezeigt, trotz eingeloggtem Zustand.
So wird alles angezeigt:

Code: Alles auswählen

            <td valign="middle" nowrap="nowrap" align="right">{postrow.UPI2DB_IGNORE_USER_IMG} {postrow.HIDE_IMG} {postrow.PROFILE_IMG} {postrow.PM_IMG} {postrow.EMAIL_IMG} {postrow.WWW_IMG} {postrow.USER_PIX_IMG} {postrow.ADS_IMG} {postrow.AIM_IMG} {postrow.YIM_IMG} {postrow.MSN_IMG} {postrow.PHOOGLE_IMG} {postrow.ICQ_IMG}</td> 

Was mache ich verkehrt?

/Holger

[oxpus]

Du packst einen Switch in einen Switch und verwendest nicht davon abhängige Token.
Das macht das Template-System des phpBB 2 so nicht mit (im phpBB 3 wäre dsa schon was ganz anderes, aber auch anders und geschickter lösbar).

Wenn schon {postrow.UPI2DB_IGNORE_USER_IMG} und {postrow.HIDE_IMG} komplett in der viewtopic.php erstellt werden, warum dann nicht dort prüfen, ob der User angemeldet ist und die Icons entsprechend darstellen, bzw. weglassen?

[Holger]

Hummmm, ja, gute Idee!
Der wichtige Teil:

Code: Alles auswählen

		'UPI2DB_IGNORE_USER_IMG' => ignore_user($poster_id, $post_id, $topic_id, $postrow[$i]['user_level'], 'viewtopic'),

HIDE_IMG kommt mehrfach vor (HIDE_IMG ist aber nicht so wichtig!):

Code: Alles auswählen

		if ($userdata['session_logged_in'])
		{
			$temp_url = append_sid("viewtopic.$phpEx?hide=hide&t=" . $postrow[$i]['topic_id'] . "&hidee_id=" . $postrow[$i]['user_id']);
			$hide_img = '<a href="' . $temp_url . '"><img src="' . $images['icon_hide'] . '" alt="' . $lang['Hide_sigav'] . '" title="' . $lang['Hide_sigav'] . '" border="0" /></a>';
			$hide = '<a href="' . $temp_url . '">' . $lang['Hide_sigav'] . '</a>';
		}
		else
		{
			$temp_url = "";
			$hide_img = "";
			$hide = "";
		}

und

Code: Alles auswählen

   if ( isset($hid_ids[$postrow[$i]['user_id']]) && $hid_ids[$postrow[$i]['user_id']] )
   {

		$poster_avatar = "";
		$user_sig = "";

		$temp_url = append_sid("viewtopic.$phpEx?hide=unhide&t=" . $postrow[$i]['topic_id'] . "&hidee_id=" . $postrow[$i]['user_id']);
		$hide_img = '<a href="' . $temp_url . '"><img src="' . $images['icon_unhide'] . '" alt="' . $lang['Unhide_user'] . '" title="' . $lang['Unhide_user'] . '" border="0" /></a>';
		$hide = '<a href="' . $temp_url . '">' . $lang['Unhide_user'] . '</a>';
	}

und

Code: Alles auswählen

		'HIDE_IMG' => $hide_img,

Bei dem Ersten ist das ja schon gelöst, aber bei dem 2. nicht.


Das Problem ist, dass der UPI2DB_IGNORE_USER_IMG-Link auch bei Google auftaucht. Wenn ein User also einen eigenen Post anklickt, drückt er sich selber auf die Ignore-Liste ...

[oxpus]

Den 2. Block würde ich so ändern:

Code: Alles auswählen

   if ( isset($hid_ids[$postrow[$i]['user_id']]) && $hid_ids[$postrow[$i]['user_id']] )
   {

		$poster_avatar = "";
		$user_sig = "";

		if ($userdata['session_logged_in'])
		{
			$temp_url = append_sid("viewtopic.$phpEx?hide=unhide&t=" . $postrow[$i]['topic_id'] . "&hidee_id=" . $postrow[$i]['user_id']);
			$hide_img = '<a href="' . $temp_url . '"><img src="' . $images['icon_unhide'] . '" alt="' . $lang['Unhide_user'] . '" title="' . $lang['Unhide_user'] . '" border="0" /></a>';
			$hide = '<a href="' . $temp_url . '">' . $lang['Unhide_user'] . '</a>';
		}
		else
		{
			$hide_img = '';
			$hide = '';
		}
	}

Für die UPI2DB würde ich das so basteln:

Code: Alles auswählen

'UPI2DB_IGNORE_USER_IMG' => ($userdata['session_logged_in']) ? ignore_user($poster_id, $post_id, $topic_id, $postrow[$i]['user_level'], 'viewtopic') : '',

[Holger]

Danke! Scheint perfekt zu funktionieren!

[oxpus]

Sag ich doch