Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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
- Write a patch
- Review
- 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
Comment | File | Size | Author |
---|---|---|---|
#2 | pathauto-vertical_tab_summary-2882323-2.patch | 1.11 KB | idebr |
Comments
Comment #2
idebr CreditAttribution: idebr at ezCompany commentedAttached patch implements the following changes:
.prop
instead.attr
Comment #3
ralphvdhoudt CreditAttribution: ralphvdhoudt at ezCompany commentedPatch works as advertised and is conform https://www.drupal.org/node/2815083
Comment #5
BerdirThanks.