Closed (fixed)
Project:
Pathauto
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
30 Jan 2024 at 20:24 UTC
Updated:
11 Feb 2026 at 22:34 UTC
Jump to comment: Most recent
From issue #3411792: Fix phpcs Drupal.WhiteSpace, Squiz.WhiteSpace issues
https://git.drupalcode.org/project/pathauto/-/merge_requests/62#note_256662
This could very well be a leftover that's no longer necessary. looking at the parent PathWidget, it already does some check with $form['advanced']. maybe we can just drop this whole chunk in a separate issue preferably after some manual testinng.
// Taxonomy terms do not have an actual fieldset for path settings.
// Merge in the defaults.
// @todo Impossible to do this in widget, use another solution
/*
$form['path'] += array(
'#type' => 'fieldset',
'#title' => $this->t('URL path settings'),
'#collapsible' => TRUE,
'#collapsed' => empty($form['path']['alias']),
'#group' => 'additional_settings',
'#attributes' => array(
'class' => array('path-form'),
),
'#access' => \Drupal::currentUser()->hasPermission('create url aliases') || \Drupal::currentUser()->hasPermission('administer url aliases'),
'#weight' => 30,
'#tree' => TRUE,
'#element_validate' => array('path_form_element_validate'),
);*/
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #4
mably commentedShould also be removed by issue #3571880: Global phpcs fixes
Comment #5
mably commentedComment #7
mably commented