Fehler in Toolbar

[Overhead]

Hallo Leute,

gerade bemerkt und das ist eigentlich nur eine optische Sache...

Wenn jemand eine PN bekommt zeigt das die MiniToolbar so an:



was ist da wohl schief gelaufen?


cu Overhead

[oxpus]

Vermutlich ein Fehler in der Link-Definition.
Also wohl in der viewtopic.php zu suchen...

[Overhead]

... mh viewtopic - da ist das als einigstes drin....

Code: Alles auswählen

if ( $can_watch_topic )
{
	$uw_parm = $is_watching_topic ? 'unwatch' : 'watch';
	$tlbr_more = array(
		'watch' => array('link_pgm' => 'viewtopic', 'link_parms' => array(POST_TOPIC_URL => intval($topic_id), $uw_parm => 'topic', 'start' => intval($start)), 'txt' => $is_watching_topic ? 'Stop_watching_topic' : 'Start_watching_topic', 'img' => $is_watching_topic ? 'tlbr_un_watch' : 'tlbr_watch'),'pdf' => array('link_pgm' => 'pdf', 'link_parms' => array(POST_TOPIC_URL => intval($topic_id)), 'txt' => 'PDF', 'img' => 'tlbr_pdf'),

	);
}
build_toolbar('viewtopic', $l_privmsgs_text, $s_privmsg_new, $forum_id, $tlbr_more);

cu Overhead

[oxpus]

AH!
Du hast ja diesen MOD im Einsatz...

Nun, dieser Ausschnitt ist NICHT direkt der betreffende Part, der das Icon für die Privaten Nachrichten erstellt.
Interessant ist vielmehr, was in $l_privmsgs_text und $s_privmsg_new enthalten ist.

[Overhead]

[quote="oxpus";p="79426"]AH!
Du hast ja diesen MOD im Einsatz...

Interessant ist vielmehr, was in $l_privmsgs_text und $s_privmsg_new enthalten ist.[/quote]

grml - bin ich aber nun schlauer? - eigentlich nicht - die Files werden in der viewtopic nicht angesprochen.... das heisßt wohl ich muss wo völlig anders suchen...

cu Overhead

[oxpus]

Na, irgendwo in der viewtopic.php müssen die Variablen doch gefüllt werden.
Die kommen doch nicht von ungefähr.
Vielleicht sind die aber auch in der page_header.php definiert.
Wäre eine Möglichkeit.

[Overhead]

@ Oxpus,

wenn dann aus der functions.php

Code: Alles auswählen

function build_toolbar($mode, $l_privmsgs_text='', $s_privmsg_new=0, $forum_id=0, $tlbr_more='')
{
	global $userdata, $template, $lang, $images, $phpEx;

	// restrict mode input to valid options
	$mode = ( in_array($mode, array('default', 'index', 'viewforum', 'viewtopic')) ) ? $mode : '';

	if ( !empty($mode) && $userdata['session_logged_in'] )
	{
		// init vars
		$s_toolbar = '';

		// toolbar actions details display
		$toolbar_actions = array(
			'inbox' => array('link_pgm' => 'privmsg', 'link_parms' => array('folder' => 'inbox'), 'txt' => $l_privmsgs_text, 'img' => !$s_privmsg_new ? 'tlbr_no_new_pm' : 'tlbr_new_pm'),
			'unanswered' => array('link_pgm' => 'search', 'link_parms' => array('search_id' => 'unanswered'), 'txt' => 'Search_unanswered', 'img' => 'tlbr_unanswered'),
			'newposts' => array('link_pgm' => 'search', 'link_parms' => array('search_id' => 'newposts'), 'txt' => 'Search_new', 'img' => 'tlbr_new'),
			'egosearch' => array('link_pgm' => 'search', 'link_parms' => array('search_id' => 'egosearch'), 'txt' => 'Search_your_posts', 'img' => 'tlbr_self'),
'pdf' => array('link_pgm' => 'pdf', 'link_parms' => array(POST_TOPIC_URL => intval($topic_id)), 'txt' => 'PDF', 'img' => 'tbr_pdf'),
			'forums' => array('link_pgm' => 'index', 'link_parms' => array(POST_FORUM_URL => intval($forum_id), 'mark' => 'forums'), 'txt' => 'Mark_all_forums', 'img' => 'tlbr_markall', 'cond' => $mode == 'index'),
			'topics' => array('link_pgm' => 'viewforum', 'link_parms' => array(POST_FORUM_URL => intval($forum_id), 'mark' => 'topics'), 'txt' => 'Mark_all_topics', 'img' => 'tlbr_markall', 'cond' => !empty($forum_id) && ($mode == 'viewforum' || $mode == 'viewtopic')),
			'viewonline' => array('link_pgm' => 'viewonline', 'link_parms' => '', 'txt' => 'Who_is_Online', 'img' => 'tlbr_viewonline', 'cond' => $mode != 'viewtopic'),
		);

		// add additional actions in toolbar so existing
		if ( !empty($tlbr_more) && is_array($tlbr_more) )
		{
			$toolbar_actions = array_merge($toolbar_actions, $tlbr_more);
		}

		// let's go
		foreach ( $toolbar_actions as $action => $data )
		{
			if ( !isset($data['cond']) || $data['cond'] )
			{
				// build url parms
				$url_parms = '';
				if ( !empty($data['link_parms']) )
				{
					foreach ( $data['link_parms'] as $key => $val )
					{
						if ( !empty($key) && !empty($val) )
						{
							$url_parms .= (empty($url_parms) ? '?' : '&') . $key . '=' . $val;
						}
					}
				}

				// build toolbar
				$s_toolbar .= '<a href="' . append_sid($data['link_pgm']. '.' . $phpEx . $url_parms) . '"><img src="' . $images[ $data['img'] ] . '" alt="' . ( $action == 'inbox' ? $data['txt'] : $lang[ $data['txt'] ] ) . '" title="' . ( $action == 'inbox' ? $data['txt'] : $lang[ $data['txt'] ] ) . '" border="0" /></a>';
			}
		}

		// send to template
		if ( !empty($s_toolbar) )
		{
			// constants
			$template->assign_block_vars('toolbar', array(
				'S_TOOLBAR' => $s_toolbar,
			));
		}
	}
}

aber das sieht eigentlich völlig ok aus...

cu Overhead

[oxpus]

Nicht die Funktion ist hier wichtig, sondern was in den genannten Variablen drinnen ist!
Denn wenn dort bereits ein Link definiert wird, wird Link in Link verschachtelt, was dann folglich zu den von Dir genannten Fehler führt.

[Overhead]

@ Oxpus

in der pageheader.php

wir folgendes definiert:

Code: Alles auswählen

$s_privmsg_new = 1;
			$icon_pm = $images['pm_new_msg'];
		}
		else
		{
			$s_privmsg_new = 0;
			$icon_pm = $images['pm_new_msg'];
		}
	}
	else
	{
		$l_privmsgs_text = $lang['No_new_pm'];

		$s_privmsg_new = 0;
		$icon_pm = $images['pm_no_new_msg'];
	}

	if ( $userdata['user_unread_privmsg'] )
	{
		$l_message_unread = ( $userdata['user_unread_privmsg'] == 1 ) ? $lang['Unread_pm'] : $lang['Unread_pms'];
		$l_privmsgs_text_unread = sprintf($l_message_unread, $userdata['user_unread_privmsg']);
	}
	else
	{
		$l_privmsgs_text_unread = $lang['No_unread_pm'];
	}
}
else
{
	$icon_pm = $images['pm_no_new_msg'];
	$l_privmsgs_text = $lang['Login_check_pm'];
	$l_privmsgs_text_unread = '';
	$s_privmsg_new = 0;
}

Da tauchen also beide Sachen auf - in der viewtopic hingegen sind beide nicht enthalten - also wenn dann muss doch hier etwas falsch sein - wobei mit nicht klar ist wo das sein sollte.

cu Overhead

[oxpus]

Was steht in der lang_main.php hinter $lang['Unread_pm'] und $lang['Unread_pms']?

[Overhead]

@ Oxpus

das steht in der lang_main

Code: Alles auswählen

// Private Messaging
//
$lang['Private_Messaging'] = 'Private Nachrichten';

$lang['Login_check_pm'] = 'Einloggen, um private Nachrichten zu lesen';
$lang['New_pms'] = '<EMBED SRC="post.wav" AUTOSTART=true LOOP=FALSE WIDTH=0 HEIGHT=0></EMBED>Du hast %d neue Nachrichten'; // You have 2 new messages
$lang['New_pm'] = '<EMBED SRC="post.wav" AUTOSTART=true LOOP=FALSE WIDTH=0 HEIGHT=0></EMBED>Du hast 1 neue Nachricht'; // You have 1 new message
$lang['No_new_pm'] = 'Du hast keine neuen Nachrichten';
$lang['Unread_pms'] = 'Du hast %d ungelesene Nachrichten';
$lang['Unread_pm'] = 'Du hast 1 ungelesene Nachricht';
$lang['No_unread_pm'] = 'Du hast keine ungelesenen Nachrichten';
$lang['You_new_pm'] = 'Eine neue private Nachricht befindet sich in deinem Posteingang';
$lang['You_new_pms'] = 'Es befinden sich neue private Nachrichten in deinem Posteingang';
$lang['You_no_new_pm'] = 'Es sind keine neuen privaten Nachrichten vorhanden';

cu Overhead

[oxpus]

Soweit alles okay.

Häng mal bitte die cfg-Datei des Styles an (nicht die theme.cfg, sondern die mit dem Stylenamen).
Wenn dort die Bilder anders definiert sind, als nur mit den Pfadangaben, kann daher der Fehler rühren.

[Overhead]

@ Oxpus

bitteschön...

cu Overhead

[ Attachment gelöscht am Di 27.Nov, 2007 15:09 von oxpus ]

[oxpus]

Hier die Datei zurück.
Wie richtig vermutet, waren die Images nicht korrekt definiert, so daß der Navi-MOD hier Probleme bekommt.

[Overhead]

@ Oxpus

Danke - da muss mal aber erst auch darauf kommen... da hätte ich niemals gesucht.


cu Overhead

[oxpus]

Deshalb hatte ich ja auch Punkt für Punkt abgeklappert, um alle anderen Fehler auszuschliessen.

Erfahrung, die man im Laufe der Jahre sammelt.
Ist nicht einfach für Anfänger oder weniger erfahrene Admins.

[Overhead]

@ Oxpus

wieso warst Du eigentlich verwundert das ich die Mod benutze - das sieht fein aus und schließlich hast Du hier ja etwas ähnliches am laufen. - Ich finde wenn es etwas gibt das die Bedienung erleichtert und dann noch schick aussieht kann man das ruhig nutzen - imo gibt es viele Mods die nur ein Forum aufblasen und im Verhältnis dazu eigentlich zuwenig Nutzen bringen - oder nach Einbau (z.B. Album) von den Admins erhöhte Aufmerksamkeit erfordern.

cu Overhead

[oxpus]

Verwundert nicht, daß Du ihn einsetzt, sondern daß es ihn gibt.
Ich habe das alles "manuell" erstellt, also Links in der viewtopic.php und die Definitionen in der passenden tpl dazu.
Da sich auch nichts ändert, hatte ich mir jeglichen weiteren MOD hierzu geschenkt, bzw. erst gar nicht weiter danach gesucht.

Sonst ist alles easy...

[Overhead]

@ Oxpus

im Original ist das mit dem pdf nicht eingebaut - da hattest Du mit geholfen - ich lege die Mod mal bei evtl. willst Du die ja in Deiner Downloadsammlung verewigen - ich hatte Deine Lösung immer bewundert und wusste das Du die selber gebastelt hast - Du hast ja in irgendeinem Beitrag geschrieben das Du die Sache nicht public machen willst weil Du das Teil exclusiv für Dein Forum gebaut hast. - Also hab ich mich auf die Suche gemacht, und habe mal meine rudimentären Sprachkenntnisse zusammengekratzt bis ich schließlich in Frankreich fündig geworden bin. btw. Bei denen und in Brasilien gibt es einige megaheftig gemoddete phpbb Foren auf denen u.a. auch über CT, CH und andere Nettigkeiten heftig diskutiert und Einbaufehlerbereinigt wird - Schade nur das der Austausch der Modder untereinander, wie Du, Amiga, Hotischi und CB das pflegen dort anscheinend völlig außen vor ist - da wurstelt jeder vor sich hin - und ist auch nicht bereit sich von anderen Moddern helfen zu lassen.

cu Overhead

[oxpus]

Ich sage dazu nur: Andere Länder, andere Sitten.

Nun, Downloads nehme ich bei mir nur auf, wenn ich sie selber verbaut habe oder hatte.
Ansonsten bleiben die aussen vor.
Aber hier im Beitrag kann den ja auch jeder finden...

[Overhead]

@ Oxpus

irgendwie ist ein Wurm in der Sache - cfg. hatte ich ja übernommen - der Darstellungsfehler war weg nun ist er wieder da. Also nochmal die von Dir bearbeitete sfg. gezogen und eingebunden - aber diesmal bleibt der Fehler - an den anderen Sache hatte ich eigentlich nichts gemacht. -- das ist irgendwie zu Mäusemelken...

cu Overhead

[oxpus]

Da ich diesen MOD kaum kenne, muss ich hier auch passen...

Hinweis: Sind die Images in der cfg-Datei des Styles korrekt eingetragen?
Also ohne zusätzliche Hochkommas?

[Overhead]

@ oxpus

die stehen so drin:

Code: Alles auswählen

//-- mod : toolbar -------------------------------------------------------------
//-- add
$images['tlbr_markall'] = "$current_template_images/tlbr_markall.gif";
$images['tlbr_new'] = "$current_template_images/tlbr_new.gif";
$images['tlbr_new_pm'] = "$current_template_images/tlbr_pm_new.gif";
$images['tlbr_no_new_pm'] = "$current_template_images/tlbr_pm.gif";
$images['tlbr_self'] = "$current_template_images/tlbr_self.gif";
$images['tlbr_un_watch'] = "$current_template_images/tlbr_un_watch.gif";
$images['tlbr_pdf'] = "$current_template_images/tlbr_pdf.gif";
$images['tlbr_unanswered'] = "$current_template_images/tlbr_unanswered.gif";
$images['tlbr_viewonline'] = "$current_template_images/tlbr_viewonline.gif";
$images['tlbr_watch'] = "$current_template_images/tlbr_watch.gif";
//-- fin mod : toolbar ---------------------------------------------------------

cu Overhead

[oxpus]

Und die sind nicht noch einmal an zweitere Stelle definiert?

[Overhead]

@ oxpus


In der cfg. stehen die nur einmal drin -

das ist es ja was mich so wundert...


cu Overhead