Frage zu kanonischen URLs...

[Janosch1408]

Ich wollte mir soeben mal die viewforum.php, viewtopic.php und overall_header.html ändern, damit mein Forum Kanonische URLs hat (oder umschreibt?^^)

das viewforum.php war einfach... bei viewtopic musste ich mir überlegen wie das genau gemeint war: vor "// Output page"

habe ich das so richtig gemacht?:
http://www.jan-schuessler.ch/code.png
habe bedenken, weil dieser code in keiner {} steht... habe aber eig. keine ahnung von programmieren...

und beim overall_header... ich benutze diesen style hier:
http://www.stsoftware.biz/phpbb30_style.ca_halcyon.html

dort gibt es eine zeile die ähnlich ist wie vom tutorial:

code von Tutorial: <link href="{T_THEME_PATH}/large.css" rel="alternate stylesheet" type="text/css" title="A++" />
mein style: <link href="{T_THEME_PATH}/print.css" rel="stylesheet" type="text/css" media="print" title="printonly" />

nach diesem code müsste ich ja diesen einfügen:

<!-- IF U_CANONICAL --><link rel="canonical" href="{U_CANONICAL}" /><!-- ENDIF -->

muss ich den noch ändern da es ein anderer style ist?


besten Dank!

Grüsse Janosch

[Janosch1408]

hm, kann mir da niemand weiterhelfen?

[redbull254]

Hallo erstmal,

hilfreich wäre zuerst einmal ein Link zu dem Code den Du verbauen willst.
Diesen gibt es einmal als Snippet und einmal als Mod.

[larsneo]

zuerst einmal ein paar grundlagen zu rel="canonical" - das link element gibt suchmaschinen ja lediglich einen hinweis auf die bevorzugte adresse einer seite und hat im 'normalen' betrieb keinerlei auswirkung (um es unabhängig vom quälcodeüberhaupt anzuzeigen nutze ich das searchstatus addon für den firefox, siehe auch *klick*).

dein bisheriger einbau sieht auf den ersten blick korrekt aus, wo kann man sich das denn mal live anschauen (templates regenieren nicht vergessen)?

[Janosch1408]

werde es gleich noch fertig stellen und uploaden/aktualisieren...
dann auf http://www.fotografenforum.ch

Danke

[Janosch1408]

so, ist alles drauf... wie kann man jetzt testen ob es funktioniert??^^


PS: den code habe ich vom Tutorial im Forum Snippets

[larsneo]

so, ist alles drauf... wie kann man jetzt testen ob es funktioniert??^^

wer lesen kann ist klar im vorteil *klick* - ein einfacher blick in den quältext der seite hätte dir aber auch schon das fehlen der rel="canonical" angabe gezeigt...

[Janosch1408]

oh, tschuldigung - übersehen.

hm was habe ich denn falsch gemacht?

ausschnitt aus viewforum:

Code: Alles auswählen

$template->assign_vars(array(
	'MODERATORS'	=> (!empty($moderators[$forum_id])) ? implode(', ', $moderators[$forum_id]) : '',
	'U_CANONICAL'   => generate_board_url() . '/' . append_sid("viewforum.$phpEx", "f=$forum_id" . (($start) ? "&start=$start" : ''), true, ''),

	'POST_IMG'					=> ($forum_data['forum_status'] == ITEM_LOCKED) ? $user->img('button_topic_locked', $post_alt) : $user->img('button_topic_new', $post_alt),
	'NEWEST_POST_IMG'			=> $user->img('icon_topic_newest', 'VIEW_NEWEST_POST'),
	'LAST_POST_IMG'				=> $user->img('icon_topic_latest', 'VIEW_LATEST_POST'),
	'FOLDER_IMG'				=> $user->img('topic_read', 'NO_NEW_POSTS'),
	'FOLDER_NEW_IMG'			=> $user->img('topic_unread', 'NEW_POSTS'),
	'FOLDER_HOT_IMG'			=> $user->img('topic_read_hot', 'NO_NEW_POSTS_HOT'),

viewtopic:

Code: Alles auswählen

// We need to do the same with the topic_id. See #53025.
if (empty($_REQUEST['t']) && !empty($topic_id))
{
	$_REQUEST['t'] = $topic_id;
}
$template->assign_var('U_CANONICAL', generate_board_url() . '/' . append_sid("viewtopic.$phpEx", "f=$forum_id&t=$topic_id" . (($start) ? "&start=$start" : ''), true, ''));
// Output the page
page_header($user->lang['VIEW_TOPIC'] . ' - ' . $topic_data['topic_title'], true, $forum_id);

$template->set_filenames(array(
	'body' => ($view == 'print') ? 'viewtopic_print.html' : 'viewtopic_body.html')
);
make_jumpbox(append_sid("{$phpbb_root_path}viewforum.$phpEx"), $forum_id);

page_footer();

?>

overall_header:

Code: Alles auswählen

<link href="{T_THEME_PATH}/print.css" rel="stylesheet" type="text/css" media="print" title="printonly" />
<link href="{T_STYLESHEET_LINK}" rel="stylesheet" type="text/css" media="screen, projection" />
<!-- IF U_CANONICAL --><link rel="canonical" href="{U_CANONICAL}" /><!-- ENDIF -->
<link media="only screen and (max-device-width: 480px)" href="{T_THEME_PATH}/iphone.css" type="text/css" rel="stylesheet" />

index.php:

Code: Alles auswählen

	'FORUM_IMG'				=> $user->img('forum_read', 'NO_NEW_POSTS'),
	'FORUM_NEW_IMG'			=> $user->img('forum_unread', 'NEW_POSTS'),
	'FORUM_LOCKED_IMG'		=> $user->img('forum_read_locked', 'NO_NEW_POSTS_LOCKED'),
	'FORUM_NEW_LOCKED_IMG'	=> $user->img('forum_unread_locked', 'NO_NEW_POSTS_LOCKED'),

	'S_LOGIN_ACTION'			=> append_sid("{$phpbb_root_path}ucp.$phpEx", 'mode=login'),
	'S_DISPLAY_BIRTHDAY_LIST'	=> ($config['load_birthdays']) ? true : false,

	'U_MARK_FORUMS'		=> ($user->data['is_registered'] || $config['load_anon_lastread']) ? append_sid("{$phpbb_root_path}index.$phpEx", 'hash=' . generate_link_hash('global') . '&mark=forums') : '',
	'U_MCP'				=> ($auth->acl_get('m_') || $auth->acl_getf_global('m_')) ? append_sid("{$phpbb_root_path}mcp.$phpEx", 'i=main&mode=front', true, $user->session_id) : '')
);
$template->assign_var('U_CANONICAL', generate_board_url() . '/' . append_sid('', false, true, ''));
// Output page
page_header($user->lang['INDEX']);

$template->set_filenames(array(
	'body' => 'index_body.html')
);

page_footer();

?>