diff --git a/core/includes/bootstrap.inc b/core/includes/bootstrap.inc index eac2f97..8d61dec 100644 --- a/core/includes/bootstrap.inc +++ b/core/includes/bootstrap.inc @@ -2478,6 +2478,9 @@ function drupal_container(Container $new_container = NULL, $rebuild = FALSE) { ->register('keyvalue.database', 'Drupal\Core\KeyValueStore\KeyValueDatabaseFactory') ->addArgument(new Reference('database')); + $container->register('path.alias_manager', 'Drupal\Core\Path\AliasManager') + ->addArgument(new Reference('database')); + // Register the EntityManager. $container->register('plugin.manager.entity', 'Drupal\Core\Entity\EntityManager'); } diff --git a/core/includes/common.inc b/core/includes/common.inc index 540f42b..d73dcb7 100644 --- a/core/includes/common.inc +++ b/core/includes/common.inc @@ -2217,7 +2217,7 @@ function url($path = NULL, array $options = array()) { } elseif (!empty($path) && !$options['alias']) { $langcode = isset($options['language']) && isset($options['language']->langcode) ? $options['language']->langcode : ''; - $alias = drupal_get_path_alias($original_path, $langcode); + $alias = drupal_container()->get('path.alias_manager')->getPathAlias($original_path, $langcode); if ($alias != $original_path) { $path = $alias; } @@ -4935,9 +4935,6 @@ function _drupal_bootstrap_full($skip = FALSE) { // current_path(). drupal_language_initialize(); - // Initialize current_path() prior to invoking hook_init(). - drupal_path_initialize(); - // Let all modules take action before the menu system handles the request. // We do not want this while running update.php. if (!defined('MAINTENANCE_MODE') || MAINTENANCE_MODE != 'update') { diff --git a/core/includes/menu.inc b/core/includes/menu.inc index d28316b..836fda9 100644 --- a/core/includes/menu.inc +++ b/core/includes/menu.inc @@ -2995,7 +2995,7 @@ function _menu_delete_item($item, $force = FALSE) { * @param $item * An associative array representing a menu link item, with elements: * - link_path: (required) The path of the menu item, which should be - * normalized first by calling drupal_get_normal_path() on it. + * normalized first by calling drupal_container()->get('path.alias_manager')->getSystemPath() on it. * - link_title: (required) Title to appear in menu for the link. * - menu_name: (optional) The machine name of the menu for the link. * Defaults to 'tools'. diff --git a/core/includes/path.inc b/core/includes/path.inc index dc878c0..ebb8ef5 100644 --- a/core/includes/path.inc +++ b/core/includes/path.inc @@ -2,7 +2,7 @@ /** * @file - * Functions to handle paths in Drupal, including path aliasing. + * Functions to handle paths in Drupal. * * These functions are not loaded for cached pages, but modules that need * to use them in hook_boot() or hook exit() can make them available, by @@ -10,289 +10,6 @@ */ /** - * Initializes the current path to the proper normal path. - */ -function drupal_path_initialize() { - // At this point, the current path is either the request path (due to - // drupal_environment_initialize()) or some modified version of it due to - // other bootstrap code (e.g., language negotiation), but it has not yet been - // normalized by drupal_get_normal_path(). - $path = _current_path(); - - // If on the front page, resolve to the front page path, including for calls - // to current_path() while drupal_get_normal_path() is in progress. - if (empty($path)) { - $path = config('system.site')->get('page.front'); - _current_path($path); - } - - // Normalize the path. - _current_path(drupal_get_normal_path($path)); -} - -/** - * Given an alias, return its Drupal system URL if one exists. Given a Drupal - * system URL return one of its aliases if such a one exists. Otherwise, - * return FALSE. - * - * @param $action - * One of the following values: - * - wipe: delete the alias cache. - * - alias: return an alias for a given Drupal system path (if one exists). - * - source: return the Drupal system URL for a path alias (if one exists). - * @param $path - * The path to investigate for corresponding aliases or system URLs. - * @param $langcode - * Optional language code to search the path with. Defaults to the page language. - * If there's no path defined for that language it will search paths without - * language. - * - * @return - * Either a Drupal system path, an aliased path, or FALSE if no path was - * found. - */ -function drupal_lookup_path($action, $path = '', $langcode = NULL) { - // Use the advanced drupal_static() pattern, since this is called very often. - static $drupal_static_fast; - if (!isset($drupal_static_fast)) { - $drupal_static_fast['cache'] = &drupal_static(__FUNCTION__); - } - $cache = &$drupal_static_fast['cache']; - - if (!isset($cache)) { - $cache = array( - 'map' => array(), - 'no_source' => array(), - 'whitelist' => NULL, - 'system_paths' => array(), - 'no_aliases' => array(), - 'first_call' => TRUE, - ); - } - - // Retrieve the path alias whitelist. - if (!isset($cache['whitelist'])) { - $cache['whitelist'] = state()->get('system.path_alias_whitelist', NULL); - if (!isset($cache['whitelist'])) { - $cache['whitelist'] = drupal_path_alias_whitelist_rebuild(); - } - } - - // If no language is explicitly specified we default to the current URL - // language. If we used a language different from the one conveyed by the - // requested URL, we might end up being unable to check if there is a path - // alias matching the URL path. - $langcode = $langcode ? $langcode : language(LANGUAGE_TYPE_URL)->langcode; - - if ($action == 'wipe') { - $cache = array(); - $cache['whitelist'] = drupal_path_alias_whitelist_rebuild(); - } - elseif ($cache['whitelist'] && $path != '') { - if ($action == 'alias') { - // During the first call to drupal_lookup_path() per language, load the - // expected system paths for the page from cache. - if (!empty($cache['first_call'])) { - $cache['first_call'] = FALSE; - - $cache['map'][$langcode] = array(); - // Load system paths from cache. - $cid = current_path(); - if ($cached = cache('path')->get($cid)) { - $cache['system_paths'] = $cached->data; - // Now fetch the aliases corresponding to these system paths. - $args = array( - ':system' => $cache['system_paths'], - ':langcode' => $langcode, - ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, - ); - // Always get the language-specific alias before the language-neutral - // one. For example 'de' is less than 'und' so the order needs to be - // ASC, while 'xx-lolspeak' is more than 'und' so the order needs to - // be DESC. We also order by pid ASC so that fetchAllKeyed() returns - // the most recently created alias for each source. Subsequent queries - // using fetchField() must use pid DESC to have the same effect. - // For performance reasons, the query builder is not used here. - if ($langcode == LANGUAGE_NOT_SPECIFIED) { - // Prevent PDO from complaining about a token the query doesn't use. - unset($args[':langcode']); - $result = db_query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode = :langcode_undetermined ORDER BY pid ASC', $args); - } - elseif ($langcode < LANGUAGE_NOT_SPECIFIED) { - $result = db_query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid ASC', $args); - } - else { - $result = db_query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid ASC', $args); - } - $cache['map'][$langcode] = $result->fetchAllKeyed(); - // Keep a record of paths with no alias to avoid querying twice. - $cache['no_aliases'][$langcode] = array_flip(array_diff_key($cache['system_paths'], array_keys($cache['map'][$langcode]))); - } - } - // If the alias has already been loaded, return it. - if (isset($cache['map'][$langcode][$path])) { - return $cache['map'][$langcode][$path]; - } - // Check the path whitelist, if the top_level part before the first / - // is not in the list, then there is no need to do anything further, - // it is not in the database. - elseif (!isset($cache['whitelist'][strtok($path, '/')])) { - return FALSE; - } - // For system paths which were not cached, query aliases individually. - elseif (!isset($cache['no_aliases'][$langcode][$path])) { - $args = array( - ':source' => $path, - ':langcode' => $langcode, - ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, - ); - // See the queries above. - if ($langcode == LANGUAGE_NOT_SPECIFIED) { - unset($args[':langcode']); - $alias = db_query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode = :langcode_undetermined ORDER BY pid DESC", $args)->fetchField(); - } - elseif ($langcode > LANGUAGE_NOT_SPECIFIED) { - $alias = db_query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid DESC", $args)->fetchField(); - } - else { - $alias = db_query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid DESC", $args)->fetchField(); - } - $cache['map'][$langcode][$path] = $alias; - return $alias; - } - } - // Check $no_source for this $path in case we've already determined that there - // isn't a path that has this alias - elseif ($action == 'source' && !isset($cache['no_source'][$langcode][$path])) { - // Look for the value $path within the cached $map - $source = FALSE; - if (!isset($cache['map'][$langcode]) || !($source = array_search($path, $cache['map'][$langcode]))) { - $args = array( - ':alias' => $path, - ':langcode' => $langcode, - ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, - ); - // See the queries above. - if ($langcode == LANGUAGE_NOT_SPECIFIED) { - unset($args[':langcode']); - $result = db_query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode = :langcode_undetermined ORDER BY pid DESC", $args); - } - elseif ($langcode > LANGUAGE_NOT_SPECIFIED) { - $result = db_query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid DESC", $args); - } - else { - $result = db_query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid DESC", $args); - } - if ($source = $result->fetchField()) { - $cache['map'][$langcode][$source] = $path; - } - else { - // We can't record anything into $map because we do not have a valid - // index and there is no need because we have not learned anything - // about any Drupal path. Thus cache to $no_source. - $cache['no_source'][$langcode][$path] = TRUE; - } - } - return $source; - } - } - - return FALSE; -} - -/** - * Cache system paths for a page. - * - * Cache an array of the system paths available on each page. We assume - * that aliases will be needed for the majority of these paths during - * subsequent requests, and load them in a single query during - * drupal_lookup_path(). - */ -function drupal_cache_system_paths() { - // Check if the system paths for this page were loaded from cache in this - // request to avoid writing to cache on every request. - $cache = &drupal_static('drupal_lookup_path', array()); - if (empty($cache['system_paths']) && !empty($cache['map'])) { - - // @todo Because we are not within the request scope at this time, we cannot - // use current_path(), which would give us the system path to use as the - // key. Instead we call _current_path(), which may give us the alias - // instead. However, at lookup time the system path will be used as the - // key, because it uses current_path(), and so it will be a cache miss. - // There is a critical issue for fixing the path alias logic here: - // http://drupal.org/node/1269742 - - // Generate a cache ID (cid) specifically for this page. - $cid = _current_path(); - // The static $map array used by drupal_lookup_path() includes all - // system paths for the page request. - if ($paths = current($cache['map'])) { - $data = array_keys($paths); - $expire = REQUEST_TIME + (60 * 60 * 24); - cache('path')->set($cid, $data, $expire); - } - } -} - -/** - * Given an internal Drupal path, return the alias set by the administrator. - * - * If no path is provided, the function will return the alias of the current - * page. - * - * @param $path - * An internal Drupal path. - * @param $langcode - * An optional language code to look up the path in. - * - * @return - * An aliased path if one was found, or the original path if no alias was - * found. - */ -function drupal_get_path_alias($path = NULL, $langcode = NULL) { - // If no path is specified, use the current page's path. - if ($path == NULL) { - $path = current_path(); - } - $result = $path; - if ($alias = drupal_lookup_path('alias', $path, $langcode)) { - $result = $alias; - } - return $result; -} - -/** - * Given a path alias, return the internal path it represents. - * - * @param $path - * A Drupal path alias. - * @param $langcode - * An optional language code to look up the path in. - * - * @return - * The internal path represented by the alias, or the original alias if no - * internal path was found. - */ -function drupal_get_normal_path($path, $langcode = NULL) { - $original_path = $path; - - // Lookup the path alias first. - if ($source = drupal_lookup_path('source', $path, $langcode)) { - $path = $source; - } - - // Allow other modules to alter the inbound URL. We cannot use drupal_alter() - // here because we need to run hook_url_inbound_alter() in the reverse order - // of hook_url_outbound_alter(). - foreach (array_reverse(module_implements('url_inbound_alter')) as $module) { - $function = $module . '_url_inbound_alter'; - $function($path, $original_path, $langcode); - } - - return $path; -} - -/** * Check if the current page is the front page. * * @return @@ -379,36 +96,6 @@ function current_path() { } /** - * Rebuild the path alias white list. - * - * @param $source - * An optional system path for which an alias is being inserted. - * - * @return - * An array containing a white list of path aliases. - */ -function drupal_path_alias_whitelist_rebuild($source = NULL) { - // When paths are inserted, only rebuild the whitelist if the system path - // has a top level component which is not already in the whitelist. - if (!empty($source)) { - $whitelist = state()->get('system.path_alias_whitelist', NULL); - if (isset($whitelist[strtok($source, '/')])) { - return $whitelist; - } - } - // For each alias in the database, get the top level component of the system - // path it corresponds to. This is the portion of the path before the first - // '/', if present, otherwise the whole path itself. - $whitelist = array(); - $result = db_query("SELECT DISTINCT SUBSTRING_INDEX(source, '/', 1) AS path FROM {url_alias}"); - foreach ($result as $row) { - $whitelist[$row->path] = TRUE; - } - state()->set('system.path_alias_whitelist', $whitelist); - return $whitelist; -} - -/** * Fetch a specific URL alias from the database. * * @param $conditions @@ -424,77 +111,7 @@ function drupal_path_alias_whitelist_rebuild($source = NULL) { * - langcode: The language code of the alias. */ function path_load($conditions) { - if (is_numeric($conditions)) { - $conditions = array('pid' => $conditions); - } - elseif (is_string($conditions)) { - $conditions = array('source' => $conditions); - } - elseif (!is_array($conditions)) { - return FALSE; - } - $select = db_select('url_alias'); - foreach ($conditions as $field => $value) { - $select->condition($field, $value); - } - return $select - ->fields('url_alias') - ->execute() - ->fetchAssoc(); -} - -/** - * Save a path alias to the database. - * - * @param $path - * An associative array containing the following keys: - * - source: The internal system path. - * - alias: The URL alias. - * - pid: (optional) Unique path alias identifier. - * - langcode: (optional) The language code of the alias. - */ -function path_save(&$path) { - $path += array('langcode' => LANGUAGE_NOT_SPECIFIED); - - // Load the stored alias, if any. - if (!empty($path['pid']) && !isset($path['original'])) { - $path['original'] = path_load($path['pid']); - } - - if (empty($path['pid'])) { - drupal_write_record('url_alias', $path); - module_invoke_all('path_insert', $path); - } - else { - drupal_write_record('url_alias', $path, array('pid')); - module_invoke_all('path_update', $path); - } - - // Clear internal properties. - unset($path['original']); - - // Clear the static alias cache. - drupal_clear_path_cache($path['source']); -} - -/** - * Delete a URL alias. - * - * @param $criteria - * A number representing the pid or an array of criteria. - */ -function path_delete($criteria) { - if (!is_array($criteria)) { - $criteria = array('pid' => $criteria); - } - $path = path_load($criteria); - $query = db_delete('url_alias'); - foreach ($criteria as $field => $value) { - $query->condition($field, $value); - } - $query->execute(); - module_invoke_all('path_delete', $path); - drupal_clear_path_cache($path['source']); + return drupal_container()->get('path.crud')->load($conditions); } /** @@ -597,14 +214,3 @@ function drupal_valid_path($path, $dynamic_allowed = FALSE) { return $item && $item['access']; } -/** - * Clear the path cache. - * - * @param $source - * An optional system path for which an alias is being changed. - */ -function drupal_clear_path_cache($source = NULL) { - // Clear the drupal_lookup_path() static cache. - drupal_static_reset('drupal_lookup_path'); - drupal_path_alias_whitelist_rebuild($source); -} diff --git a/core/lib/Drupal/Core/CacheDecorator/AliasManagerCacheDecorator.php b/core/lib/Drupal/Core/CacheDecorator/AliasManagerCacheDecorator.php new file mode 100644 index 0000000..21fbfd1 --- /dev/null +++ b/core/lib/Drupal/Core/CacheDecorator/AliasManagerCacheDecorator.php @@ -0,0 +1,116 @@ +aliasManager = $alias_manager; + $this->cache = $cache; + } + + /** + * Implements \Drupal\Core\CacheDecorator\CacheDecoratorInterface::writeCache(). + * + * Cache an array of the system paths available on each page. We assume + * that aliases will be needed for the majority of these paths during + * subsequent requests, and load them in a single query during path alias + * lookup. + */ + public function writeCache() { + $path_lookups = $this->getLookupMap(); + // Check if the system paths for this page were loaded from cache in this + // request to avoid writing to cache on every request. + if (empty($this->systemPaths) && !empty($path_lookups) && !empty($this->requestPaths)) { + // Use the system path of the current request for the cache ID (cid). + $cid = end($this->requestPaths); + // The $map array includes all system paths for the page request. + if ($paths = current($path_lookups)) { + $data = array_keys($paths); + $expire = REQUEST_TIME + (60 * 60 * 24); + $this->cache->set($cid, $data, $expire); + } + } + // We are at the end of the request, so pop off the last request path. + array_pop($this->requestPaths); + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getSystemPath(). + */ + public function getSystemPath($path, $path_language = NULL) { + $system_path = $this->aliasManager->getSystemPath($path, $path_language); + // Because this class is only used by the PathSubscriber, when this method + // is called we know it is being called to get the system path of the + // current request. Therefore the result of the path lookup is the cache + // key we'll need to use when writing the system path cache. Pop it onto + // our stack of request paths. + $this->requestPaths[] = $system_path; + // We also need to pass the list of previously cached system paths for this + // request path on to the alias manager for use in subsequent lookups. + $cached = $this->cache->get($system_path); + $cached_paths = !empty($cached) ? $cached->data : array(); + $this->setSystemPaths($cached_paths); + return $system_path; + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getPathAlias(). + */ + public function getPathAlias($path = NULL, $path_language = NULL) { + return $this->aliasManager->getPathAlias($path, $path_language); + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getLookupMap(). + */ + public function getLookupMap() { + return $this->aliasManager->getLookupMap(); + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::setSystemPaths(). + */ + public function setSystemPaths(array $system_paths) { + $this->systemPaths = $system_paths; + $this->aliasManager->setSystemPaths($system_paths); + } +} diff --git a/core/lib/Drupal/Core/CacheDecorator/CacheDecoratorInterface.php b/core/lib/Drupal/Core/CacheDecorator/CacheDecoratorInterface.php new file mode 100644 index 0000000..7ed0b89 --- /dev/null +++ b/core/lib/Drupal/Core/CacheDecorator/CacheDecoratorInterface.php @@ -0,0 +1,17 @@ +register('nested_matcher', 'Drupal\Core\Routing\NestedMatcher') ->addTag('chained_matcher', array('priority' => 5)); + $container + ->register('cache.path', 'Drupal\Core\Cache\CacheBackendInterface') + ->setFactoryClass('Drupal\Core\Cache\CacheFactory') + ->setFactoryMethod('get') + ->addArgument('path'); + + $container->register('cache_decorator.alias_manager', 'Drupal\Core\CacheDecorator\AliasManagerCacheDecorator') + ->addArgument(new Reference('path.alias_manager')) + ->addArgument(new Reference('cache.path')); + + $container->register('path.crud', 'Drupal\Core\Path\Path') + ->addArgument(new Reference('database')) + ->addArgument(new Reference('path.alias_manager')); + // The following services are tagged as 'nested_matcher' services and are // processed in the RegisterNestedMatchersPass compiler pass. Each one // needs to be set on the matcher using a different method, so we use a @@ -107,6 +122,7 @@ public function build(ContainerBuilder $container) { $container->register('maintenance_mode_subscriber', 'Drupal\Core\EventSubscriber\MaintenanceModeSubscriber') ->addTag('event_subscriber'); $container->register('path_subscriber', 'Drupal\Core\EventSubscriber\PathSubscriber') + ->addArgument(new Reference('cache_decorator.alias_manager')) ->addTag('event_subscriber'); $container->register('legacy_request_subscriber', 'Drupal\Core\EventSubscriber\LegacyRequestSubscriber') ->addTag('event_subscriber'); diff --git a/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php index 7688107..efb6561 100644 --- a/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/PathSubscriber.php @@ -7,8 +7,10 @@ namespace Drupal\Core\EventSubscriber; +use Drupal\Core\CacheDecorator\AliasManagerCacheDecorator; use Symfony\Component\HttpKernel\KernelEvents; use Symfony\Component\HttpKernel\Event\GetResponseEvent; +use Symfony\Component\HttpKernel\Event\PostResponseEvent; use Symfony\Component\EventDispatcher\EventSubscriberInterface; /** @@ -16,26 +18,33 @@ */ class PathSubscriber extends PathListenerBase implements EventSubscriberInterface { + protected $aliasManagerCacheDecorator; + + public function __construct(AliasManagerCacheDecorator $alias_manager_cache_decorator) { + $this->aliasManagerCacheDecorator = $alias_manager_cache_decorator; + } + /** * Resolve the system path. * - * @todo The path system should be objectified to remove the function calls in - * this method. - * * @param Symfony\Component\HttpKernel\Event\GetResponseEvent $event * The Event to process. */ public function onKernelRequestPathResolve(GetResponseEvent $event) { $request = $event->getRequest(); - $path = $this->extractPath($request); - - $path = drupal_get_normal_path($path); - + $path = $this->aliasManagerCacheDecorator->getSystemPath($path); $this->setPath($request, $path); } /** + * Ensures system paths for the request get cached. + */ + public function onKernelTerminate(PostResponseEvent $event) { + $this->aliasManagerCacheDecorator->writeCache(); + } + + /** * Resolve the front-page default path. * * @todo The path system should be objectified to remove the function calls in @@ -116,6 +125,7 @@ static function getSubscribedEvents() { $events[KernelEvents::REQUEST][] = array('onKernelRequestLanguageResolve', 150); $events[KernelEvents::REQUEST][] = array('onKernelRequestFrontPageResolve', 101); $events[KernelEvents::REQUEST][] = array('onKernelRequestPathResolve', 100); + $events[KernelEvents::TERMINATE][] = array('onKernelTerminate', 200); return $events; } diff --git a/core/lib/Drupal/Core/EventSubscriber/RequestCloseSubscriber.php b/core/lib/Drupal/Core/EventSubscriber/RequestCloseSubscriber.php index 175ce79..8d5f87a 100644 --- a/core/lib/Drupal/Core/EventSubscriber/RequestCloseSubscriber.php +++ b/core/lib/Drupal/Core/EventSubscriber/RequestCloseSubscriber.php @@ -29,7 +29,6 @@ class RequestCloseSubscriber implements EventSubscriberInterface { */ public function onTerminate(PostResponseEvent $event) { module_invoke_all('exit'); - drupal_cache_system_paths(); module_implements_write_cache(); system_run_automated_cron(); } @@ -41,7 +40,7 @@ public function onTerminate(PostResponseEvent $event) { * An array of event listener definitions. */ static function getSubscribedEvents() { - $events[KernelEvents::TERMINATE][] = array('onTerminate'); + $events[KernelEvents::TERMINATE][] = array('onTerminate', 100); return $events; } diff --git a/core/lib/Drupal/Core/ExceptionController.php b/core/lib/Drupal/Core/ExceptionController.php index 332d4b9..1ef4e35 100644 --- a/core/lib/Drupal/Core/ExceptionController.php +++ b/core/lib/Drupal/Core/ExceptionController.php @@ -101,7 +101,7 @@ public function on403Html(FlattenException $exception, Request $request) { $system_path = $request->attributes->get('system_path'); watchdog('access denied', $system_path, array(), WATCHDOG_WARNING); - $path = drupal_get_normal_path(config('system.site')->get('page.403')); + $path = $this->container->get('path.alias_manager')->getSystemPath(config('system.site')->get('page.403')); if ($path && $path != $system_path) { // Keep old path for reference, and to allow forms to redirect to it. if (!isset($_GET['destination'])) { @@ -167,7 +167,7 @@ public function on404Html(FlattenException $exception, Request $request) { $_GET['destination'] = $system_path; } - $path = drupal_get_normal_path(config('system.site')->get('page.404')); + $path = $this->container->get('path.alias_manager')->getSystemPath(config('system.site')->get('page.404')); if ($path && $path != $system_path) { // @todo Um, how do I specify an override URL again? Totally not clear. Do // that and sub-call the kernel rather than using meah(). diff --git a/core/lib/Drupal/Core/Path/AliasManager.php b/core/lib/Drupal/Core/Path/AliasManager.php new file mode 100644 index 0000000..d640576 --- /dev/null +++ b/core/lib/Drupal/Core/Path/AliasManager.php @@ -0,0 +1,299 @@ +connection = $connection; + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getSystemPath(). + */ + public function getSystemPath($path, $path_language = NULL) { + $original_path = $path; + // Lookup the path alias first. + if ($source = $this->lookupPath('source', $path, $path_language)) { + $path = $source; + } + + return $path; + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getPathAlias(). + */ + public function getPathAlias($path = NULL, $path_language = NULL) { + // If no path is specified, use the current page's path. + if ($path === NULL) { + $path = _current_path(); + } + $result = $path; + if ($alias = $this->lookupPath('alias', $path, $path_language)) { + $result = $alias; + } + return $result; + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::cacheClear(). + */ + public function cacheClear($source = NULL) { + $this->lookupMap = array(); + $this->noSource = array(); + $this->whitelist = NULL; + $this->no_aliases = array(); + $this->firstCall = TRUE; + $this->systemPaths = array(); + $this->pathAliasWhitelistRebuild($source); + } + + /** + * Implements \Drupal\Core\Path\AliasManagerInterface::getLookupMap(). + */ + public function getLookupMap() { + return $this->lookupMap; + } + + public function setSystemPaths(array $system_path_list) { + $this->systemPaths = $system_path_list; + } + + /** + * Given an alias, return its Drupal system URL if one exists. Given a Drupal + * system URL return one of its aliases if such a one exists. Otherwise, + * return FALSE. + * + * @param $action + * One of the following values: + * - wipe: delete the alias cache. + * - alias: return an alias for a given Drupal system path (if one exists). + * - source: return the Drupal system URL for a path alias (if one exists). + * @param $path + * The path to investigate for corresponding aliases or system URLs. + * @param $langcode + * Optional language code to search the path with. Defaults to the page language. + * If there's no path defined for that language it will search paths without + * language. + * + * @return + * Either a Drupal system path, an aliased path, or FALSE if no path was + * found. + */ + protected function lookupPath($action, $path = '', $langcode = NULL) { + + // Retrieve the path alias whitelist. + if (!isset($this->whitelist)) { + // @todo Inject state so we don't have this function call. + $this->whitelist= state()->get('system.path_alias_whitelist', NULL); + if (!isset($this->whitelist)) { + $this->whitelist = $this->pathAliasWhitelistRebuild(); + } + } + + // If no language is explicitly specified we default to the current URL + // language. If we used a language different from the one conveyed by the + // requested URL, we might end up being unable to check if there is a path + // alias matching the URL path. + if (!$langcode) { + // @todo Inject the language manager so we don't have this function call. + $langcode = language(LANGUAGE_TYPE_URL)->langcode; + } + + if ($action == 'wipe') { + $this->cacheClear(); + } + elseif ($this->whitelist && $path != '') { + if ($action == 'alias') { + // During the first call to this method per language, load the expected + // system paths for the page from cache. + if (!empty($this->firstLookup)) { + $this->firstLookup = FALSE; + $this->lookupMap[$langcode] = array(); + // Load system paths from cache. + if (!empty($this->systemPaths)) { + // Now fetch the aliases corresponding to these system paths. + $args = array( + ':system' => $this->systemPaths, + ':langcode' => $langcode, + ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, + ); + // Always get the language-specific alias before the language-neutral + // one. For example 'de' is less than 'und' so the order needs to be + // ASC, while 'xx-lolspeak' is more than 'und' so the order needs to + // be DESC. We also order by pid ASC so that fetchAllKeyed() returns + // the most recently created alias for each source. Subsequent queries + // using fetchField() must use pid DESC to have the same effect. + // For performance reasons, the query builder is not used here. + if ($langcode == LANGUAGE_NOT_SPECIFIED) { + // Prevent PDO from complaining about a token the query doesn't use. + unset($args[':langcode']); + $result = $this->connection->query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode = :langcode_undetermined ORDER BY pid ASC', $args); + } + elseif ($langcode < LANGUAGE_NOT_SPECIFIED) { + $result = $this->connection->query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid ASC', $args); + } + else { + $result = $this->connection->query('SELECT source, alias FROM {url_alias} WHERE source IN (:system) AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid ASC', $args); + } + $this->lookupMap[$langcode] = $result->fetchAllKeyed(); + // Keep a record of paths with no alias to avoid querying twice. + $this->noAliases[$langcode] = array_flip(array_diff_key($this->systemPaths, array_keys($this->lookupMap[$langcode]))); + } + } + // If the alias has already been loaded, return it. + if (isset($this->lookupMap[$langcode][$path])) { + return $this->lookupMap[$langcode][$path]; + } + // Check the path whitelist, if the top-level part before the first / + // is not in the list, then there is no need to do anything further, + // it is not in the database. + elseif (!isset($this->whitelist[strtok($path, '/')])) { + return FALSE; + } + // For system paths which were not cached, query aliases individually. + elseif (!isset($this->noAliases[$langcode][$path])) { + $args = array( + ':source' => $path, + ':langcode' => $langcode, + ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, + ); + // See the queries above. + if ($langcode == LANGUAGE_NOT_SPECIFIED) { + unset($args[':langcode']); + $alias = $this->connection->query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode = :langcode_undetermined ORDER BY pid DESC", $args)->fetchField(); + } + elseif ($langcode > LANGUAGE_NOT_SPECIFIED) { + $alias = $this->connection->query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid DESC", $args)->fetchField(); + } + else { + $alias = $this->connection->query("SELECT alias FROM {url_alias} WHERE source = :source AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid DESC", $args)->fetchField(); + } + $this->lookupMap[$langcode][$path] = $alias; + return $alias; + } + } + // Check $no_source for this $path in case we've already determined that there + // isn't a path that has this alias + elseif ($action == 'source' && !isset($this->noSource[$langcode][$path])) { + // Look for the value $path within the cached $map + $source = FALSE; + if (!isset($this->lookupMap[$langcode]) || !($source = array_search($path, $this->lookupMap[$langcode]))) { + $args = array( + ':alias' => $path, + ':langcode' => $langcode, + ':langcode_undetermined' => LANGUAGE_NOT_SPECIFIED, + ); + // See the queries above. + if ($langcode == LANGUAGE_NOT_SPECIFIED) { + unset($args[':langcode']); + $result = $this->connection->query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode = :langcode_undetermined ORDER BY pid DESC", $args); + } + elseif ($langcode > LANGUAGE_NOT_SPECIFIED) { + $result = $this->connection->query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode DESC, pid DESC", $args); + } + else { + $result = $this->connection->query("SELECT source FROM {url_alias} WHERE alias = :alias AND langcode IN (:langcode, :langcode_undetermined) ORDER BY langcode ASC, pid DESC", $args); + } + if ($source = $result->fetchField()) { + $this->lookupMap[$langcode][$source] = $path; + } + else { + // We can't record anything into $map because we do not have a valid + // index and there is no need because we have not learned anything + // about any Drupal path. Thus cache to $no_source. + $this->noSource[$langcode][$path] = TRUE; + } + } + return $source; + } + } + + return FALSE; + } + + /** + * Rebuild the path alias white list. + * + * @param $source + * An optional system path for which an alias is being inserted. + * + * @return + * An array containing a white list of path aliases. + */ + protected function pathAliasWhitelistRebuild($source = NULL) { + // When paths are inserted, only rebuild the whitelist if the system path + // has a top level component which is not already in the whitelist. + if (!empty($source)) { + // @todo Inject state so we don't have this function call. + $whitelist = state()->get('system.path_alias_whitelist', NULL); + if (isset($whitelist[strtok($source, '/')])) { + return $whitelist; + } + } + // For each alias in the database, get the top level component of the system + // path it corresponds to. This is the portion of the path before the first + // '/', if present, otherwise the whole path itself. + $whitelist = array(); + $result = $this->connection->query("SELECT DISTINCT SUBSTRING_INDEX(source, '/', 1) AS path FROM {url_alias}"); + foreach ($result as $row) { + $whitelist[$row->path] = TRUE; + } + // @todo Inject state so we don't have this function call. + state()->set('system.path_alias_whitelist', $whitelist); + return $whitelist; + } +} diff --git a/core/lib/Drupal/Core/Path/AliasManagerInterface.php b/core/lib/Drupal/Core/Path/AliasManagerInterface.php new file mode 100644 index 0000000..1d348d2 --- /dev/null +++ b/core/lib/Drupal/Core/Path/AliasManagerInterface.php @@ -0,0 +1,52 @@ +connection = $connection; + $this->alias_manager = $alias_manager; + } + + public function save($source, $alias, $langcode = LANGUAGE_NOT_SPECIFIED, $pid = NULL) { + + $fields = array( + 'source' => $source, + 'alias' => $alias, + 'langcode' => $langcode, + ); + + // Insert or update the alias. + if (empty($pid)) { + $query = $this->connection->insert('url_alias', array('return' => Database::RETURN_INSERT_ID)) + ->fields($fields); + $pid = $query->execute(); + $fields['pid'] = $pid; + // @todo: Find a correct place to invoke hook_path_insert(). + $hook = 'path_insert'; + } + else { + $fields['pid'] = $pid; + $query = $this->connection->update('url_alias') + ->fields($fields) + ->condition('pid', $pid); + $pid = $query->execute(); + // @todo: figure out where we can invoke hook_path_update() + $hook = 'path_update'; + } + if ($pid) { + module_invoke_all($hook, $fields); + $this->alias_manager->cacheClear(); + return $fields; + } + return FALSE; + } + + public function load($conditions) { + if (is_numeric($conditions)) { + $conditions = array('pid' => $conditions); + } + elseif (is_string($conditions)) { + $conditions = array('source' => $conditions); + } + elseif (!is_array($conditions)) { + return FALSE; + } + $select = $this->connection->select('url_alias'); + foreach ($conditions as $field => $value) { + $select->condition($field, $value); + } + return $select + ->fields('url_alias') + ->execute() + ->fetchAssoc(); + } + + public function delete($conditions) { + if (!is_array($conditions)) { + $conditions = array('pid' => $conditions); + } + $path = $this->load($conditions); + $query = $this->connection->delete('url_alias'); + foreach ($conditions as $field => $value) { + $query->condition($field, $value); + } + $deleted = $query->execute(); + $this->alias_manager->cacheClear(); + // @todo: figure out where we can invoke hook_path_delete() + return $deleted; + } +} + diff --git a/core/modules/block/block.module b/core/modules/block/block.module index 2978146..7340c6b 100644 --- a/core/modules/block/block.module +++ b/core/modules/block/block.module @@ -851,7 +851,7 @@ function block_block_list_alter(&$blocks) { if ($block->visibility < BLOCK_VISIBILITY_PHP) { // Compare the lowercase path alias (if any) and internal path. $path = current_path(); - $path_alias = drupal_strtolower(drupal_get_path_alias($path)); + $path_alias = drupal_strtolower(drupal_container()->get('path.alias_manager')->getPathAlias($path)); $page_match = drupal_match_path($path_alias, $pages) || (($path != $path_alias) && drupal_match_path($path, $pages)); // When $block->visibility has a value of 0 (BLOCK_VISIBILITY_NOTLISTED), // the block is displayed on all pages except those listed in $block->pages. diff --git a/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php b/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php index bafb620..d1ad5f0 100644 --- a/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php +++ b/core/modules/locale/lib/Drupal/locale/Tests/LocalePathTest.php @@ -109,13 +109,13 @@ function testPathLanguageConfiguration() { 'alias' => $custom_path, 'langcode' => LANGUAGE_NOT_SPECIFIED, ); - path_save($edit); - $lookup_path = drupal_lookup_path('alias', 'node/' . $node->nid, 'en'); + drupal_container()->get('path.crud')->save($edit['source'], $edit['alias'], $edit['langcode']); + $lookup_path = drupal_container()->get('path.alias_manager')->getPathAlias('node/' . $node->nid, 'en'); $this->assertEqual($english_path, $lookup_path, t('English language alias has priority.')); // Same check for language 'xx'. - $lookup_path = drupal_lookup_path('alias', 'node/' . $node->nid, $prefix); + $lookup_path = drupal_container()->get('path.alias_manager')->getPathAlias('node/' . $node->nid, $prefix); $this->assertEqual($custom_language_path, $lookup_path, t('Custom language alias has priority.')); - path_delete($edit); + drupal_container()->get('path.crud')->delete($edit); // Create language nodes to check priority of aliases. $first_node = $this->drupalCreateNode(array('type' => 'page', 'promote' => 1)); @@ -127,7 +127,7 @@ function testPathLanguageConfiguration() { 'alias' => $custom_path, 'langcode' => 'en', ); - path_save($edit); + drupal_container()->get('path.crud')->save($edit['source'], $edit['alias'], $edit['langcode']); // Assign a custom path alias to second node with LANGUAGE_NOT_SPECIFIED. $edit = array( @@ -135,7 +135,7 @@ function testPathLanguageConfiguration() { 'alias' => $custom_path, 'langcode' => LANGUAGE_NOT_SPECIFIED, ); - path_save($edit); + drupal_container()->get('path.crud')->save($edit['source'], $edit['alias'], $edit['langcode']); // Test that both node titles link to our path alias. $this->drupalGet(''); diff --git a/core/modules/menu/menu.admin.inc b/core/modules/menu/menu.admin.inc index 3b67b97..605f898 100644 --- a/core/modules/menu/menu.admin.inc +++ b/core/modules/menu/menu.admin.inc @@ -417,7 +417,7 @@ function menu_edit_item($form, &$form_state, $type, $item, $menu) { */ function menu_edit_item_validate($form, &$form_state) { $item = &$form_state['values']; - $normal_path = drupal_get_normal_path($item['link_path']); + $normal_path = drupal_container()->get('path.alias_manager')->getSystemPath($item['link_path']); if ($item['link_path'] != $normal_path) { drupal_set_message(t('The menu system stores system paths only, but will use the URL alias for display. %link_path has been stored as %normal_path', array('%link_path' => $item['link_path'], '%normal_path' => $normal_path))); $item['link_path'] = $normal_path; diff --git a/core/modules/path/lib/Drupal/path/Tests/PathAliasTest.php b/core/modules/path/lib/Drupal/path/Tests/PathAliasTest.php index 181bd72..4168275 100644 --- a/core/modules/path/lib/Drupal/path/Tests/PathAliasTest.php +++ b/core/modules/path/lib/Drupal/path/Tests/PathAliasTest.php @@ -57,9 +57,7 @@ function testPathCache() { // Visit the alias for the node and confirm a cache entry is created. cache('path')->flush(); $this->drupalGet($edit['alias']); - // @todo The alias should actually have been cached with the system path as - // the key, see the todo in drupal_cache_system_paths() in path.inc. - $this->assertTrue(cache('path')->get($edit['alias']), 'Cache entry was created.'); + $this->assertTrue(cache('path')->get($edit['source']), 'Cache entry was created.'); } /** @@ -94,7 +92,7 @@ function testAdminAlias() { $this->assertText($node1->label(), 'Changed alias works.'); $this->assertResponse(200); - drupal_static_reset('drupal_lookup_path'); + drupal_container()->get('path.alias_manager')->cacheClear(); // Confirm that previous alias no longer works. $this->drupalGet($previous); $this->assertNoText($node1->label(), 'Previous alias no longer works.'); diff --git a/core/modules/path/lib/Drupal/path/Tests/PathLanguageTest.php b/core/modules/path/lib/Drupal/path/Tests/PathLanguageTest.php index e6fd10d..83be7b5 100644 --- a/core/modules/path/lib/Drupal/path/Tests/PathLanguageTest.php +++ b/core/modules/path/lib/Drupal/path/Tests/PathLanguageTest.php @@ -82,7 +82,7 @@ function testAliasTranslation() { $this->drupalPost(NULL, $edit, t('Save')); // Clear the path lookup cache. - drupal_lookup_path('wipe'); + drupal_container()->get('path.alias_manager')->cacheClear(); // Ensure the node was created. $french_node = $this->drupalGetNodeByTitle($edit["title"]); @@ -121,7 +121,7 @@ function testAliasTranslation() { // We need to ensure that the user language preference is not taken into // account while determining the path alias language, because if this // happens we have no way to check that the path alias is valid: there is no - // path alias for French matching the english alias. So drupal_lookup_path() + // path alias for French matching the english alias. So the alias manager // needs to use the URL language to check whether the alias is valid. $this->drupalGet($english_alias); $this->assertText($english_node->label(), 'Alias for English translation works.'); @@ -145,20 +145,20 @@ function testAliasTranslation() { $this->drupalGet($french_alias); $this->assertResponse(404, 'Alias for French translation is unavailable when URL language negotiation is disabled.'); - // drupal_lookup_path() has an internal static cache. Check to see that + // The alias manager has an internal path lookup cache. Check to see that // it has the appropriate contents at this point. - drupal_lookup_path('wipe'); - $french_node_path = drupal_lookup_path('source', $french_alias, $french_node->langcode); + drupal_container()->get('path.alias_manager')->cacheClear(); + $french_node_path = drupal_container()->get('path.alias_manager')->getSystemPath($french_alias, $french_node->langcode); $this->assertEqual($french_node_path, 'node/' . $french_node->nid, 'Normal path works.'); // Second call should return the same path. - $french_node_path = drupal_lookup_path('source', $french_alias, $french_node->langcode); + $french_node_path = drupal_container()->get('path.alias_manager')->getSystemPath($french_alias, $french_node->langcode); $this->assertEqual($french_node_path, 'node/' . $french_node->nid, 'Normal path is the same.'); // Confirm that the alias works. - $french_node_alias = drupal_lookup_path('alias', 'node/' . $french_node->nid, $french_node->langcode); + $french_node_alias = drupal_container()->get('path.alias_manager')->getPathAlias('node/' . $french_node->nid, $french_node->langcode); $this->assertEqual($french_node_alias, $french_alias, 'Alias works.'); // Second call should return the same alias. - $french_node_alias = drupal_lookup_path('alias', 'node/' . $french_node->nid, $french_node->langcode); + $french_node_alias = drupal_container()->get('path.alias_manager')->getPathAlias('node/' . $french_node->nid, $french_node->langcode); $this->assertEqual($french_node_alias, $french_alias, 'Alias is the same.'); } } diff --git a/core/modules/path/path.admin.inc b/core/modules/path/path.admin.inc index 5465141..69578f2 100644 --- a/core/modules/path/path.admin.inc +++ b/core/modules/path/path.admin.inc @@ -76,7 +76,7 @@ function path_admin_overview($keys = NULL) { // If the system path maps to a different URL alias, highlight this table // row to let the user know of old aliases. - if ($data->alias != drupal_get_path_alias($data->source, $data->langcode)) { + if ($data->alias != drupal_container()->get('path.alias_manager')->getPathAlias($data->source, $data->langcode)) { $row['class'] = array('warning'); } @@ -218,7 +218,7 @@ function path_admin_form_delete_submit($form, &$form_state) { */ function path_admin_form_validate($form, &$form_state) { $source = &$form_state['values']['source']; - $source = drupal_get_normal_path($source); + $source = drupal_container()->get('path.alias_manager')->getSystemPath($source); $alias = $form_state['values']['alias']; $pid = isset($form_state['values']['pid']) ? $form_state['values']['pid'] : 0; // Language is only set if language.module is enabled, otherwise save for all @@ -250,7 +250,15 @@ function path_admin_form_submit($form, &$form_state) { // Remove unnecessary values. form_state_values_clean($form_state); - path_save($form_state['values']); + $pid = isset($form_state['values']['pid']) ? $form_state['values']['pid'] : 0; + $source = &$form_state['values']['source']; + $source = drupal_container()->get('path.alias_manager')->getSystemPath($source); + $alias = $form_state['values']['alias']; + // Language is only set if language.module is enabled, otherwise save for all + // languages. + $langcode = isset($form_state['values']['langcode']) ? $form_state['values']['langcode'] : LANGUAGE_NOT_SPECIFIED; + + drupal_container()->get('path.crud')->save($source, $alias, $langcode, $pid); drupal_set_message(t('The alias has been saved.')); $form_state['redirect'] = 'admin/config/search/path'; @@ -282,7 +290,7 @@ function path_admin_delete_confirm($form, &$form_state, $path) { */ function path_admin_delete_confirm_submit($form, &$form_state) { if ($form_state['values']['confirm']) { - path_delete($form_state['path']['pid']); + drupal_container()->get('path.crud')->delete($form_state['path']['pid']); $form_state['redirect'] = 'admin/config/search/path'; } } diff --git a/core/modules/path/path.api.php b/core/modules/path/path.api.php index f2c5ece..c69b97d 100644 --- a/core/modules/path/path.api.php +++ b/core/modules/path/path.api.php @@ -20,7 +20,7 @@ * - pid: Unique path alias identifier. * - langcode: The language code of the alias. * - * @see path_save() + * @see \Drupal\Core\Path\Path::save() */ function hook_path_insert($path) { db_insert('mytable') @@ -41,7 +41,7 @@ function hook_path_insert($path) { * - pid: Unique path alias identifier. * - langcode: The language code of the alias. * - * @see path_save() + * @see \Drupal\Core\Path\Path::save() */ function hook_path_update($path) { db_update('mytable') @@ -60,7 +60,7 @@ function hook_path_update($path) { * - pid: Unique path alias identifier. * - langcode: The language code of the alias. * - * @see path_delete() + * @see \Drupal\Core\Path\Path::delete() */ function hook_path_delete($path) { db_delete('mytable') diff --git a/core/modules/path/path.module b/core/modules/path/path.module index 942fbb6..07d4119 100644 --- a/core/modules/path/path.module +++ b/core/modules/path/path.module @@ -107,7 +107,7 @@ function path_form_node_form_alter(&$form, $form_state) { if ($node->langcode != LANGUAGE_NOT_SPECIFIED) { $conditions['langcode'] = $node->langcode; } - $path = path_load($conditions); + $path = drupal_container()->get('path.crud')->load($conditions); if ($path === FALSE) { $path = array(); } @@ -190,14 +190,13 @@ function path_form_element_validate($element, &$form_state, $complete_form) { */ function path_node_insert(Node $node) { if (isset($node->path)) { - $path = $node->path; - $path['alias'] = trim($path['alias']); + $alias = trim($node->path['alias']); // Only save a non-empty alias. - if (!empty($path['alias'])) { + if (!empty($alias)) { // Ensure fields for programmatic executions. - $path['source'] = 'node/' . $node->nid; - $path['langcode'] = isset($node->langcode) ? $node->langcode : LANGUAGE_NOT_SPECIFIED; - path_save($path); + $source = 'node/' . $node->nid; + $langcode = isset($node->langcode) ? $node->langcode : LANGUAGE_NOT_SPECIFIED; + drupal_container()->get('path.crud')->save($source, $alias, $langcode); } } } @@ -208,17 +207,17 @@ function path_node_insert(Node $node) { function path_node_update(Node $node) { if (isset($node->path)) { $path = $node->path; - $path['alias'] = trim($path['alias']); + $alias = trim($path['alias']); // Delete old alias if user erased it. if (!empty($path['pid']) && empty($path['alias'])) { - path_delete($path['pid']); + drupal_container()->get('path.crud')->delete($path['pid']); } // Only save a non-empty alias. if (!empty($path['alias'])) { // Ensure fields for programmatic executions. - $path['source'] = 'node/' . $node->nid; - $path['langcode'] = isset($node->langcode) ? $node->langcode : LANGUAGE_NOT_SPECIFIED; - path_save($path); + $source = 'node/' . $node->nid; + $langcode = isset($node->langcode) ? $node->langcode : LANGUAGE_NOT_SPECIFIED; + drupal_container()->get('path.crud')->save($source, $alias, $langcode, $path['pid']); } } } @@ -228,7 +227,7 @@ function path_node_update(Node $node) { */ function path_node_predelete(Node $node) { // Delete all aliases associated with this node. - path_delete(array('source' => 'node/' . $node->nid)); + drupal_container()->get('path.crud')->delete(array('source' => 'node/' . $node->nid)); } /** @@ -238,7 +237,7 @@ function path_form_taxonomy_term_form_alter(&$form, $form_state) { // Make sure this does not show up on the delete confirmation form. if (empty($form_state['confirm_delete'])) { $term = $form_state['controller']->getEntity($form_state); - $path = (isset($term->tid) ? path_load('taxonomy/term/' . $term->tid) : array()); + $path = (isset($term->tid) ? drupal_container()->get('path.crud')->load('taxonomy/term/' . $term->tid) : array()); if ($path === FALSE) { $path = array(); } @@ -279,7 +278,7 @@ function path_taxonomy_term_insert(Term $term) { // Ensure fields for programmatic executions. $path['source'] = 'taxonomy/term/' . $term->tid; $path['langcode'] = LANGUAGE_NOT_SPECIFIED; - path_save($path); + drupal_container()->get('path.crud')->save($path['source'], $path['alias'], $path['langcode']); } } } @@ -293,14 +292,15 @@ function path_taxonomy_term_update(Term $term) { $path['alias'] = trim($path['alias']); // Delete old alias if user erased it. if (!empty($path['pid']) && empty($path['alias'])) { - path_delete($path['pid']); + drupal_container()->get('path.crud')->delete($path['pid']); } // Only save a non-empty alias. if (!empty($path['alias'])) { + $pid = (!empty($path['pid']) ? $path['pid'] : NULL); // Ensure fields for programmatic executions. $path['source'] = 'taxonomy/term/' . $term->tid; $path['langcode'] = LANGUAGE_NOT_SPECIFIED; - path_save($path); + drupal_container()->get('path.crud')->save($path['source'], $path['alias'], $path['langcode'], $pid); } } } @@ -310,7 +310,7 @@ function path_taxonomy_term_update(Term $term) { */ function path_taxonomy_term_delete(Term $term) { // Delete all aliases associated with this term. - path_delete(array('source' => 'taxonomy/term/' . $term->tid)); + drupal_container()->get('path.crud')->delete(array('source' => 'taxonomy/term/' . $term->tid)); } /** diff --git a/core/modules/search/search.module b/core/modules/search/search.module index a32b5c3..866536e 100644 --- a/core/modules/search/search.module +++ b/core/modules/search/search.module @@ -643,7 +643,7 @@ function search_index($sid, $module, $text, $langcode) { if ($tagname == 'a') { // Check if link points to a node on this site if (preg_match($node_regexp, $value, $match)) { - $path = drupal_get_normal_path($match[1]); + $path = drupal_container()->get('path.alias_manager')->getSystemPath($match[1]); if (preg_match('!(?:node|book)/(?:view/)?([0-9]+)!i', $path, $match)) { $linknid = $match[1]; if ($linknid > 0) { diff --git a/core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutLinksTest.php b/core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutLinksTest.php index 54eeab4..ccf9834 100644 --- a/core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutLinksTest.php +++ b/core/modules/shortcut/lib/Drupal/shortcut/Tests/ShortcutLinksTest.php @@ -31,7 +31,7 @@ function testShortcutLinkAdd() { 'source' => 'node/' . $this->node->nid, 'alias' => $this->randomName(8), ); - path_save($path); + drupal_container()->get('path.crud')->save($path['source'], $path['alias']); // Create some paths to test. $test_cases = array( @@ -54,7 +54,7 @@ function testShortcutLinkAdd() { $saved_set = shortcut_set_load($set->set_name); $paths = $this->getShortcutInformation($saved_set, 'link_path'); $test_path = empty($test['path']) ? '' : $test['path']; - $this->assertTrue(in_array(drupal_get_normal_path($test_path), $paths), 'Shortcut created: '. $test['path']); + $this->assertTrue(in_array(drupal_container()->get('path.alias_manager')->getSystemPath($test_path), $paths), 'Shortcut created: '. $test['path']); $this->assertLink($title, 0, 'Shortcut link found on the page.'); } } diff --git a/core/modules/shortcut/shortcut.admin.inc b/core/modules/shortcut/shortcut.admin.inc index e805acf..18ec1c3 100644 --- a/core/modules/shortcut/shortcut.admin.inc +++ b/core/modules/shortcut/shortcut.admin.inc @@ -430,7 +430,7 @@ function _shortcut_link_form_elements($shortcut_link = NULL) { ); } else { - $shortcut_link['link_path'] = ($shortcut_link['link_path'] == '') ? '' : drupal_get_path_alias($shortcut_link['link_path']); + $shortcut_link['link_path'] = ($shortcut_link['link_path'] == '') ? '' : drupal_container()->get('path.alias_manager')->getPathAlias($shortcut_link['link_path']); } $form['shortcut_link']['#tree'] = TRUE; @@ -477,7 +477,7 @@ function shortcut_link_edit_validate($form, &$form_state) { */ function shortcut_link_edit_submit($form, &$form_state) { // Normalize the path in case it is an alias. - $shortcut_path = drupal_get_normal_path($form_state['values']['shortcut_link']['link_path']); + $shortcut_path = drupal_container()->get('path.alias_manager')->getSystemPath($form_state['values']['shortcut_link']['link_path']); if (empty($shortcut_path)) { $shortcut_path = ''; } @@ -517,7 +517,7 @@ function shortcut_link_add_submit($form, &$form_state) { */ function shortcut_admin_add_link($shortcut_link, &$shortcut_set) { // Normalize the path in case it is an alias. - $shortcut_link['link_path'] = drupal_get_normal_path($shortcut_link['link_path']); + $shortcut_link['link_path'] = drupal_container()->get('path.alias_manager')->getSystemPath($shortcut_link['link_path']); if (empty($shortcut_link['link_path'])) { $shortcut_link['link_path'] = ''; } diff --git a/core/modules/shortcut/shortcut.module b/core/modules/shortcut/shortcut.module index 9bbb12a..2a07989 100644 --- a/core/modules/shortcut/shortcut.module +++ b/core/modules/shortcut/shortcut.module @@ -616,7 +616,7 @@ function shortcut_set_title_exists($title) { */ function shortcut_valid_link($path) { // Do not use URL aliases. - $normal_path = drupal_get_normal_path($path); + $normal_path = drupal_container()->get('path.alias_manager')->getSystemPath($path); if ($path != $normal_path) { $path = $normal_path; } diff --git a/core/modules/statistics/statistics.module b/core/modules/statistics/statistics.module index f11f3ed..a1f1406 100644 --- a/core/modules/statistics/statistics.module +++ b/core/modules/statistics/statistics.module @@ -388,7 +388,7 @@ function statistics_block_view($delta = '') { * A string as a link, truncated to the width, linked to the given $path. */ function _statistics_link($path, $width = 35) { - $title = drupal_get_path_alias($path); + $title = drupal_container()->get('path.alias_manager')->getPathAlias($path); $title = truncate_utf8($title, $width, FALSE, TRUE); return l($title, $path); } diff --git a/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php b/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php index f52907f..7ce84cb 100644 --- a/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php +++ b/core/modules/system/lib/Drupal/system/Tests/Common/JavaScriptTest.php @@ -176,7 +176,7 @@ function testHeaderSetting() { $this->drupalGet('common-test/query-string'); $this->assertPattern('@