Joomla 3.4.1 - 升级URL后,文章ID最后显示文章内容

时间:2015-06-29 12:41:17

标签: php joomla joomla3.4

我们升级了6个Joomla安装,没有任何问题,但我们有一个表现出奇怪的行为。

假设我的文章ID为909.如果我使用有效的文章ID结束URL,则会显示该文章的内容。它应该转到404页面。

domain.com/909/                    <-- Display article id 909
domain.com/about/900/              <-- Display article id 900
domain.com/contact-us/855/         <-- Display article id 855
domain.com/contact-us/blah-blah/   <-- Display 404 page

所有这些URL都会在进入404页面时解析并显示内容。我想也许在某个地方有一个糟糕的路由器,但我确实用新安装替换了核心文件。我仍在接受这种行为。

有什么想法吗?

1 个答案:

答案 0 :(得分:0)

使用此代码完整地编辑com_content / router.php。你的意思是好的。在新的joomla更新后不要忘记重新启动。

&#13;
&#13;
<?php
/**
 * @package     Joomla.Site
 * @subpackage  com_content
 *
 * @copyright   Copyright (C) 2005 - 2014 Open Source Matters, Inc. All rights reserved.
 * @license     GNU General Public License version 2 or later; see LICENSE.txt
 */

defined('_JEXEC') or die;

/**
 * Build the route for the com_content component
 *
 * @return  array  An array of URL arguments
 * @return  array  The URL arguments to use to assemble the subsequent URL.
 * @since    1.5
 */
function ContentBuildRoute(&$query)
{
	$segments = array();

	// get a menu item based on Itemid or currently active
	$app = JFactory::getApplication();
	$menu = $app->getMenu();
	$params = JComponentHelper::getParams('com_content');
	//$advanced = $params->get('sef_advanced_link', 0);//اصلی
	$advanced = $params->get('sef_advanced_link', 1);//تغییر
	// we need a menu item.  Either the one specified in the query, or the current active one if none specified
	if (empty($query['Itemid']))
	{
		$menuItem = $menu->getActive();
		$menuItemGiven = false;
	}
	else
	{
		$menuItem = $menu->getItem($query['Itemid']);
		$menuItemGiven = true;
	}

	// check again
	if ($menuItemGiven && isset($menuItem) && $menuItem->component != 'com_content')
	{
		$menuItemGiven = false;
		unset($query['Itemid']);
	}

	if (isset($query['view']))
	{
		$view = $query['view'];
	}
	else
	{
		// we need to have a view in the query or it is an invalid URL
		return $segments;
	}

	// are we dealing with an article or category that is attached to a menu item?
	if (($menuItem instanceof stdClass) && $menuItem->query['view'] == $query['view'] && isset($query['id']) && $menuItem->query['id'] == (int) $query['id'])
	{
		unset($query['view']);

		if (isset($query['catid']))
		{
			unset($query['catid']);
		}

		if (isset($query['layout']))
		{
			unset($query['layout']);
		}

		unset($query['id']);

		return $segments;
	}

	if ($view == 'category' || $view == 'article')
	{
		if (!$menuItemGiven)
		{
			$segments[] = $view;
		}

		unset($query['view']);

		if ($view == 'article')
		{
			if (isset($query['id']) && isset($query['catid']) && $query['catid'])
			{
				$catid = $query['catid'];
				// Make sure we have the id and the alias
				if (strpos($query['id'], ':') === false)
				{
					$db = JFactory::getDbo();
					$dbQuery = $db->getQuery(true)
						->select('alias')
						->from('#__content')
						->where('id=' . (int) $query['id']);
					$db->setQuery($dbQuery);
					$alias = $db->loadResult();
					$query['id'] = $query['id'] . ':' . $alias;
				}
			}
			else
			{
				// we should have these two set for this view.  If we don't, it is an error
				return $segments;
			}
		}
		else
		{
			if (isset($query['id']))
			{
				$catid = $query['id'];
			}
			else
			{
				// we should have id set for this view.  If we don't, it is an error
				return $segments;
			}
		}

		if ($menuItemGiven && isset($menuItem->query['id']))
		{
			$mCatid = $menuItem->query['id'];
		}
		else
		{
			$mCatid = 0;
		}

		$categories = JCategories::getInstance('Content');
		$category = $categories->get($catid);

		if (!$category)
		{
			// we couldn't find the category we were given.  Bail.
			return $segments;
		}

		$path = array_reverse($category->getPath());

		$array = array();

		foreach ($path as $id)
		{
			if ((int) $id == (int) $mCatid)
			{
				break;
			}

			list($tmp, $id) = explode(':', $id, 2);

			$array[] = $id;
		}

		$array = array_reverse($array);

		if (!$advanced && count($array))
		{
			$array[0] = (int) $catid . ':' . $array[0];
		}

		$segments = array_merge($segments, $array);

		if ($view == 'article')
		{
			if ($advanced)
			{
				list($tmp, $id) = explode(':', $query['id'], 2);
			}
			else
			{
				$id = $query['id'];
			}
			$segments[] = $id;
		}
		unset($query['id']);
		unset($query['catid']);
	}

	if ($view == 'archive')
	{
		if (!$menuItemGiven)
		{
			$segments[] = $view;
			unset($query['view']);
		}

		if (isset($query['year']))
		{
			if ($menuItemGiven)
			{
				$segments[] = $query['year'];
				unset($query['year']);
			}
		}

		if (isset($query['year']) && isset($query['month']))
		{
			if ($menuItemGiven)
			{
				$segments[] = $query['month'];
				unset($query['month']);
			}
		}
	}

	if ($view == 'featured')
	{
		if (!$menuItemGiven)
		{
			$segments[] = $view;
		}

		unset($query['view']);
	}

	// if the layout is specified and it is the same as the layout in the menu item, we
	// unset it so it doesn't go into the query string.
	if (isset($query['layout']))
	{
		if ($menuItemGiven && isset($menuItem->query['layout']))
		{
			if ($query['layout'] == $menuItem->query['layout'])
			{
				unset($query['layout']);
			}
		}
		else
		{
			if ($query['layout'] == 'default')
			{
				unset($query['layout']);
			}
		}
	}

	return $segments;
}

/**
 * Parse the segments of a URL.
 *
 * @return  array  The segments of the URL to parse.
 *
 * @return  array  The URL attributes to be used by the application.
 * @since    1.5
 */
function ContentParseRoute($segments)
{
	$vars = array();

	//Get the active menu item.
	$app = JFactory::getApplication();
	$menu = $app->getMenu();
	$item = $menu->getActive();
	$params = JComponentHelper::getParams('com_content');
	//$advanced = $params->get('sef_advanced_link', 0);//اصلی	
	$advanced = $params->get('sef_advanced_link', 1);//تغییر کرد
	$db = JFactory::getDbo();

	// Count route segments
	$count = count($segments);

	// Standard routing for articles.  If we don't pick up an Itemid then we get the view from the segments
	// the first segment is the view and the last segment is the id of the article or category.
	if (!isset($item))
	{
		$vars['view'] = $segments[0];
		$vars['id'] = $segments[$count - 1];

		return $vars;
	}

	// if there is only one segment, then it points to either an article or a category
	// we test it first to see if it is a category.  If the id and alias match a category
	// then we assume it is a category.  If they don't we assume it is an article
	if ($count == 1)
	{
		// we check to see if an alias is given.  If not, we assume it is an article

/*
//این تیکه کامنت شد		
		if (strpos($segments[0], ':') === false)
		{
			$vars['view'] = 'article';
			$vars['id'] = (int) $segments[0];
			return $vars;
		}
 */
		list($id, $alias) = explode(':', $segments[0], 2);

		// first we check if it is a category
		$category = JCategories::getInstance('Content')->get($id);

		if ($category && $category->alias == $alias)
		{
			$vars['view'] = 'category';
			$vars['id'] = $id;

			return $vars;
		}
		else
		{
			$query = $db->getQuery(true)
				->select($db->quoteName(array('alias', 'catid')))
				->from($db->quoteName('#__content'))
				->where($db->quoteName('id') . ' = ' . (int) $id);
			$db->setQuery($query);
			$article = $db->loadObject();

			if ($article)
			{
				if ($article->alias == $alias)
				{
					$vars['view'] = 'article';
					$vars['catid'] = (int) $article->catid;
					$vars['id'] = (int) $id;

					return $vars;
				}
			}
		}
	}

	// if there was more than one segment, then we can determine where the URL points to
	// because the first segment will have the target category id prepended to it.  If the
	// last segment has a number prepended, it is an article, otherwise, it is a category.
	if (!$advanced)
	{
		$cat_id = (int) $segments[0];

		$article_id = (int) $segments[$count - 1];

		if ($article_id > 0)
		{
			$vars['view'] = 'article';
			$vars['catid'] = $cat_id;
			$vars['id'] = $article_id;
		}
		else
		{
			$vars['view'] = 'category';
			$vars['id'] = $cat_id;
		}

		return $vars;
	}

	// we get the category id from the menu item and search from there
	$id = $item->query['id'];
	$category = JCategories::getInstance('Content')->get($id);

	if (!$category)
	{
		JError::raiseError(404, JText::_('COM_CONTENT_ERROR_PARENT_CATEGORY_NOT_FOUND'));
		return $vars;
	}

	$categories = $category->getChildren();
	$vars['catid'] = $id;
	$vars['id'] = $id;
	$found = 0;

	foreach ($segments as $segment)
	{
		$segment = str_replace(':', '-', $segment);

		foreach ($categories as $category)
		{
			if ($category->alias == $segment)
			{
				$vars['id'] = $category->id;
				$vars['catid'] = $category->id;
				$vars['view'] = 'category';
				$categories = $category->getChildren();
				$found = 1;
				break;
			}
		}

		if ($found == 0)
		{
			if ($advanced)
			{
				$db = JFactory::getDbo();
				$query = $db->getQuery(true)
					->select($db->quoteName('id'))
					->from('#__content')
					->where($db->quoteName('catid') . ' = ' . (int) $vars['catid'])
					->where($db->quoteName('alias') . ' = ' . $db->quote($segment));
				$db->setQuery($query);
				$cid = $db->loadResult();
			}
			else
			{
				$cid = $segment;
			}

			$vars['id'] = $cid;

			if ($item->query['view'] == 'archive' && $count != 1)
			{
				$vars['year'] = $count >= 2 ? $segments[$count - 2] : null;
				$vars['month'] = $segments[$count - 1];
				$vars['view'] = 'archive';
			}
			else
			{
				$vars['view'] = 'article';
			}
		}

		$found = 0;
	}

	return $vars;
}
&#13;
&#13;
&#13;