<title>- und <meta>-Tags für Kategorien in der obersten Ebene fehlerhaft

S A G

Hallo alle zusammen !

Mir ist soeben direkt nach einer Installation der Version 1.06 aufgefallen, dass die <title>- und <meta>-Tags für Kategorien in der obersten Menü-Ebene nicht in den Quelltext übernommen werden (es wird lediglich ein "-" für den Kategoriennamen als Title eingesetzt, eventuell gesetzte Descriptions und Keywords bleiben leer). Verschiebt man aber eine solche Kategorie in eine andere, so dass diese dann eine Unterkategorie darstellt, werden die besagten Tags korrekt übernommen.

Ist dieses Problem bekannt und bereits eine Lösung vorhanden? Und wenn nicht, wo genau im Quellcode müsste man nachschauen, um den Fehler eventuell beheben zu können?

Viele Grüße,
S A G

Gunah

jop den Fehler kann ich nur bestätigen, ist bei mir auch so

PHP5.2.6-3
Apache 2.2.6
MySQL 5.0.1a-15
Debian

Gunah

habe das Problem behoben.

[...] entfernt siehe unten

Diese kann man ohne Bedenken ändern, Schutz bietet hier die Funktion intval() die nur Ganze Zahlen erlaubt.

Trotzdem Übernehme ich keine Haftung für evtl schäden

Edit://
hier mal meine Neue metatags.php kann auch in die Basis eingebaut werden oder Verändert werden

PHP

<?php


/* -----------------------------------------------------------------------------------------
 $Id: metatags.php 1140 2005-08-10 10:16:00Z mz $


 XT-Commerce - community made shopping
 [url]http://www.xt-commerce.com[/url]


 Copyright (c) 2003 XT-Commerce
 -----------------------------------------------------------------------------------------
 based on:
 (c) 2003	 nextcommerce (metatags.php,v 1.7 2003/08/14); [url]www.nextcommerce.org[/url]


 Released under the GNU General Public License
 ---------------------------------------------------------------------------------------*/
?>
<meta name="robots" content="<?php echo META_ROBOTS; ?>" />
<meta name="language" content="<?php echo $_SESSION['language_code']; ?>" />
<meta name="author" content="<?php echo META_AUTHOR; ?>" />
<meta name="publisher" content="<?php echo META_PUBLISHER; ?>" />
<meta name="company" content="<?php echo META_COMPANY; ?>" />
<meta name="page-topic" content="<?php echo META_TOPIC; ?>" />
<meta name="reply-to" content="<?php echo META_REPLY_TO; ?>" />
<meta name="distribution" content="global" />
<meta name="revisit-after" content="<?php echo META_REVISIT_AFTER; ?>" />
<meta name="robots" content="noodp" />
<?php


if (strstr($_SERVER['PHP_SELF'], FILENAME_PRODUCT_INFO)) {


	if ($product->isProduct()) {
?>
<meta name="description" content="<?php echo $product->data['products_meta_description']; ?>" />
<meta name="keywords" content="<?php echo $product->data['products_meta_keywords']; ?>" />
<title><?php echo $product->data['products_meta_title'].' '.$product->data['products_name'].' '.$product->data['products_model'].' - '.TITLE; ?></title>
	<?php


	} else {
?>
<meta name="description" content="<?php echo META_DESCRIPTION; ?>" />
<meta name="keywords" content="<?php echo META_KEYWORDS; ?>" />
<title><?php echo TITLE; ?></title>
	<?php


	}


} else {
	if ($_GET['cPath']) {
		if (strpos($_GET['cPath'], '_') == '1') {
			$arr = explode('_', xtc_input_validation($_GET['cPath'], 'cPath', ''));
			$_cPath = $arr[1];
		} else {
			//$_cPath=(int)$_GET['cPath'];
			if (isset ($_GET['cat'])) {
				$site = explode('_', $_GET['cat']);
				$cID = $site[0];
				$_cPath = str_replace('c', '', $cID);
			}
		}
		if(empty($_cPath)) $_cPath = intval($_GET['cPath']);

		if(is_array($arr)) {
		rsort($arr);
		$r = '';
			foreach ($arr as $value) {
				$r++;
				//echo $value.'_'.$r.'-';
				$categories_meta_query = xtDBquery("SELECT categories_meta_keywords,
															categories_meta_description,
															categories_meta_title,
															categories_name
															FROM " . TABLE_CATEGORIES_DESCRIPTION . "
															WHERE categories_id='" . $value . "' and
															language_id='" . $_SESSION['languages_id'] . "'");
				$categories_meta = xtc_db_fetch_array($categories_meta_query, true);
				if($r==1) $out .= $categories_meta['categories_meta_title'];
				else $out .= ' - '.$categories_meta['categories_meta_title'];
			}
		}
			$categories_meta_query = xtDBquery("SELECT categories_meta_keywords,
														categories_meta_description,
														categories_meta_title,
														categories_name
														FROM " . TABLE_CATEGORIES_DESCRIPTION . "
														WHERE categories_id='" . $_cPath . "' and
														language_id='" . $_SESSION['languages_id'] . "'");
			$categories_meta = xtc_db_fetch_array($categories_meta_query, true);
			if ($categories_meta['categories_meta_keywords'] == '') {
				$categories_meta['categories_meta_keywords'] = META_KEYWORDS;
			}
			if ($categories_meta['categories_meta_description'] == '') {
				$categories_meta['categories_meta_description'] = META_DESCRIPTION;
			}
			if(!is_array($arr)) {
					$out = $categories_meta['categories_meta_title'];
			}
			$categories_meta['categories_meta_title'] = $out;
?>
<meta name="description" content="<?php echo $categories_meta['categories_meta_description']; ?>" />
<meta name="keywords" content="<?php echo $categories_meta['categories_meta_keywords']; ?>" />
<title><?php echo $categories_meta['categories_meta_title'].' - '.TITLE; ?></title>
<?php


	} else {
		if ($_GET['coID']) {
			$contents_meta_query = xtDBquery("SELECT content_heading
			                      FROM " . TABLE_CONTENT_MANAGER . "
			                      WHERE content_group='" . intval($_GET['coID']) . "' and
			                      languages_id='" . $_SESSION['languages_id'] . "'");
			$contents_meta = xtc_db_fetch_array($contents_meta_query, true);
?>
<meta name="description" content="<?php echo META_DESCRIPTION; ?>" />
<meta name="keywords" content="<?php echo META_KEYWORDS; ?>" />
<title><?php echo $contents_meta['content_heading'].' - '.TITLE; ?></title>
<?php


		} else {
?>
<meta name="description" content="<?php echo META_DESCRIPTION; ?>" />
<meta name="keywords" content="<?php echo META_KEYWORDS; ?>" />
<title><?php echo TITLE; ?></title>
<?php


		}
	}
}
?>

Alles anzeigen

habe einpaar Intvals hinzugefügt, bei Unter Kategorien sehen die Titel nun so aus:
UnterKategorie - MainKategorie - TITLEderSEITE

Forumadmin

Habe es mal übernommen. Bisher keine Probleme festgestellt.

Gunah

mir auch nicht

habe im Orginal aber noch was gefunden und zwar steht es so drin:

Code

WHERE content_group='" . $_GET['coID'] . "' and

So kann man noch in die Datenbank Befehle über den Browser oder ein Exploit rein schreiben...

es sollte besser so aussehen:

Code

WHERE content_group='" . intval($_GET['coID']) . "' and

oder auch so

Code

WHERE content_group='" . (int) $_GET['coID'] . "' and

macht aber das gleiche
habe es auch schon bei xt:commerce eingesendet, hoffe die gucken da schnellst möglich nach, aber wie man auf ecombase lesen kann, sollen die darin nicht so schnell sein...

hier mal die Patch Datei für das Orginal:
http://gunah.org/extra/commerce…/metatags.patch

EDIT://
habe dieses Script gefunden was auch nicht schlecht ist
http://www.gunnart.de/tipps-und-tric…uer-xtcommerce/
allerdings ist hier auch der Bug wie kurz hier beschrieben wurde

edit: habe mal von gunnart die Variante überarbeitet und für meine Bedürfnisse angepasst
http://gunah.org/extra/xtc/304s…ed_by_gunah.zip

Alle änderungen sind Dokumentiert