Problem/Motivation

The code still makes use of deprecated jquery.once declarations, we should drop D8 support and only support D9 going forward.

Steps to reproduce

Add a redirect node to a menu, and check the console log on the frontend.

You should see the following:

drupal.js?v=10.0.11-dev:64 Uncaught TypeError: $(...).once is not a function
    at Object.attach (redirect_node.edit_link.js?v=10.0.11-dev:22:41)
    at drupal.js?v=10.0.11-dev:166:24
    at Array.forEach (<anonymous>)
    at Drupal.attachBehaviors (drupal.js?v=10.0.11-dev:162:34)
    at drupal.init.js?v=10.0.11-dev:32:12
    at HTMLDocument.listener (drupal.init.js?v=10.0.11-dev:20:7)

Proposed resolution

Drop support for D8, supporting >=9.2 and switching to the new APIs.

Remaining tasks

Provide issue fork.

User interface changes

N/A

Issue fork redirect_node-3380453

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:

About issue forks

Comments

codebymikey created an issue. See original summary.

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

sakthi_dev opened merge request !3

sakthi_dev’s picture

sakthi_dev CreditAttribution: sakthi_dev at Specbee for Drupal India Association commented
Status: Active » Needs review

Please review.

codebymikey’s picture

codebymikey CreditAttribution: codebymikey at Zodiac Media commented

@sakthi_dev, apologies, but had to force push my changes over your original commit since I had already implemented that as well as other issues.

core/themes/stable/images/core/icons/ffffff/pencil.svg no longer exists, and can be safely replaced with core/misc/icons/ffffff/pencil.svg

Also improved translations support.