Problem/Motivation

pathauto.js contains a behavior to set the summary of a vertical tab using drupalSetSummary. The selectors used in this behavior are outdated, since the form element for path has been converted from a fieldset to details in #1168246: Freedom For Fieldsets! Long Live The DETAILS.

Furthermore, the detection of the checkbox value uses the jQuery function .attr. This function was changed in jQuery 1.6. Typically .attr can now be substituted with .prop to maintain the old behavior.

Proposed resolution

  • Update the outdated javascript selectors in pathauto.js
  • Use jQuery .prop instead .attr

Remaining tasks

  1. Write a patch
  2. Review
  3. Commit

User interface changes

The vertical tab summary displays the correct Pathauto summary, in line with the current implementation in D7.

API changes

None

Data model changes

None

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

idebr created an issue. See original summary.

idebr’s picture

Assigned: idebr » Unassigned
Status: Active » Needs review
FileSize
1.11 KB

Attached patch implements the following changes:

  • Updated the outdated javascript selectors in pathauto.js based on D8/core/modules/path/path.js
  • Used jQuery .prop instead .attr