Skip to content
Snippets Groups Projects

Issue #3096092: Make "path_alias" module optional

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
5 5 version: VERSION
6 6 dependencies:
7 7 - drupal:filter
8 - drupal:path_alias
  • Michael Strelan added 4 commits

    added 4 commits

    • b3ea2f0b - Add path_alias to standard and demo_umami
    • 34b514c0 - Make DistributionProfileTest pass
    • 957e85e8 - Fix PathEncodedTest
    • 73cf5118 - Fix tests that use PathAliasTestTrait

    Compare with previous version

  • Michael Strelan added 2 commits

    added 2 commits

    • fb50edd8 - Fix LanguageSwitchingTest
    • 77dbe358 - Attempt to make RequestPath work without path_alias

    Compare with previous version

  • Michael Strelan added 3 commits

    added 3 commits

    • b388840d - Implement hook_help
    • 45125730 - Remove path_alias special handling from ConfigImportAllTest
    • d9f9e3c7 - Enable path_alias when enabling views

    Compare with previous version

  • 8 declare(strict_types = 1);
    9
    10 use Drupal\Core\Routing\RouteMatchInterface;
    11 use Drupal\path_alias\Form\SystemInformationFormAlter;
    12
    13 /**
    14 * Implements hook_help().
    15 */
    16 function path_alias_help($route_name, RouteMatchInterface $route_match): string {
    17 switch ($route_name) {
    18 case 'help.page.path_alias':
    19 $output = '';
    20 $output .= '<h3>' . t('About') . '</h3>';
    21 // @todo put something meaningful here.
    22 $output .= '<p>' . t('The Path alias module foo bar baz') . '</p>';
    23 $output .= '<p>' . t('For more information, see the <a href=":path_alias">online documentation for the Path alias module</a>.', [':path_alias' => 'https://www.drupal.org/documentation/modules/path_alias']);
  • 51 * @param \Drupal\Core\Path\PathValidatorInterface $path_validator
    42 * @param \Drupal\Core\Path\PathValidatorInterface|\Drupal\path_alias\AliasManagerInterface $path_validator
    52 43 * The path validator.
    53 * @param \Drupal\Core\Routing\RequestContext $request_context
    44 * @param \Drupal\Core\Routing\RequestContext|\Drupal\Core\Path\PathValidatorInterface $request_context
    54 45 * The request context.
    55 46 */
    56 public function __construct(ConfigFactoryInterface $config_factory, TypedConfigManagerInterface $typedConfigManager, AliasManagerInterface $alias_manager, PathValidatorInterface $path_validator, RequestContext $request_context) {
    47 public function __construct(ConfigFactoryInterface $config_factory, TypedConfigManagerInterface $typedConfigManager, PathValidatorInterface|AliasManagerInterface $path_validator, RequestContext|PathValidatorInterface $request_context) {
    57 48 parent::__construct($config_factory, $typedConfigManager);
    58 $this->aliasManager = $alias_manager;
    49 if ($path_validator instanceof AliasManagerInterface) {
    50 @trigger_error('Calling ' . __CLASS__ . '::__construct() with the $alias_manager argument is deprecated in drupal:10.2.0 and is removed in drupal:11.0.0. See https://www.drupal.org/node/3096092', E_USER_DEPRECATED);
    51 $path_validator = func_get_arg(3);
    52 $request_context = func_get_arg(4);
    53 }
  • 1 <?php
    2
    3 declare(strict_types = 1);
    4
    5 namespace Drupal\path_alias\Form;
    6
    7 use Drupal\Core\DependencyInjection\ContainerInjectionInterface;
    8 use Drupal\Core\DependencyInjection\DependencySerializationTrait;
    9 use Drupal\Core\Form\FormStateInterface;
    10 use Drupal\path_alias\AliasManagerInterface;
    11 use Symfony\Component\DependencyInjection\ContainerInterface;
    12
    13 /**
    14 * Alteration class for the site information settings form.
    15 */
    16 class SystemInformationFormAlter implements ContainerInjectionInterface {
  • 21 21 - menu_ui
    22 22 - options
    23 23 - path
    24 - path_alias
  • 36 36 'finish_url' => '/root-user',
    37 37 ],
    38 38 ],
    39 // @todo determine why we're adding a path alias in this test and if this can be removed.
    40 'install' => [
    41 'path_alias',
    42 ],
  • Michael Strelan added 1 commit

    added 1 commit

    • ec64dba9 - Remove aliasManager from RequestPath

    Compare with previous version

  • Michael Strelan added 1 commit

    added 1 commit

    • 08ca0ded - Remove aliasManager from RequestPath

    Compare with previous version

  • Michael Strelan added 92 commits

    added 92 commits

    Compare with previous version

  • Ankitha Shetty added 164 commits

    added 164 commits

    Compare with previous version

  • Michael Strelan added 158 commits

    added 158 commits

    Compare with previous version

  • Michael Strelan added 1 commit

    added 1 commit

    • a71464bc - Fix StableLibraryOverrideTestBase

    Compare with previous version

  • Michael Strelan added 2 commits

    added 2 commits

    • 9bb1d5e5 - Rename SystemInformationFormAlter to SiteInformationFormAlter
    • c2a3eaaf - Add test coverage

    Compare with previous version

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading