Problem/Motivation

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'),
    );*/

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Issue fork pathauto-3418158

Command icon Show commands

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

Nikolay Shapovalov created an issue. See original summary.

jeremy1606 made their first commit to this issue’s fork.

mably’s picture

Related issues: +#3571880: Global phpcs fixes

Should also be removed by issue #3571880: Global phpcs fixes

mably’s picture

Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

mably’s picture

Status: Fixed » Closed (fixed)