Invalid path error message is displayed three times in link widget for external link

Created on 26 August 2019, almost 6 years ago
Updated 26 June 2023, about 2 years ago

Add a link field to any content type with only external links allowed. Add an invalid url like `/example.com` and fill in the link text field as well. The error message is displayed 3 times.

🐛 Bug report
Status

Closed: cannot reproduce

Version

11.0 🔥

Component
Link 

Last updated 4 days ago

Created by

🇮🇳India abhaisasidharan

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Merge Requests

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • 🇺🇸United States smustgrave

    If still a valid bug please reopen updating issue summary per #24

    Thanks!

  • Status changed to Needs review about 2 years ago
  • 🇷🇴Romania amateescu

    What the issue summary didn't mention is that this is only reproducible with the Inline form errors module enabled. Updated the IS.

  • Status changed to Needs work about 2 years ago
  • 🇺🇸United States smustgrave

    Thanks for updating!

    If that's the case then the tests will need to be updated to check for that.

  • 🇺🇸United States dcam

    Added the issue summary template. Added the Needs Tests tag because the existing tests need to be updated to check for those specifically.

  • Merge request !13013Converted the patch in comment 22 to an MR → (Open) created by dcam
  • 🇺🇸United States dcam

    The patch from #22 was converted to an MR. The changes to LinkNotExistingInternalConstraintValidatorTest did not seem to be relevant anymore. No other test changes have been made.

  • Pipeline finished with Failed
    7 days ago
    Total: 240s
    #574421
  • Pipeline finished with Success
    7 days ago
    #574429
  • 🇺🇸United States dcam

    The original proposed changes must be updated because the arrayPropertyPath property was deprecated and removed. Here's the change record: https://www.drupal.org/node/3307919 .

  • Pipeline finished with Failed
    6 days ago
    Total: 285s
    #574858
  • Pipeline finished with Success
    6 days ago
    Total: 905s
    #574866
Production build 0.71.5 2024