Remove leftover from src/PathautoWidget.php

Created on 30 January 2024, 5 months ago

Problem/Motivation

From issue πŸ“Œ Fix phpcs Drupal.WhiteSpace, Squiz.WhiteSpace issues Needs review
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

πŸ“Œ Task
Status

Active

Version

1.0

Component

Code

Created by

πŸ‡·πŸ‡ΊRussia zniki.ru

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.69.0 2024