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.Shouldn't FormErrorHandler.php be using \Link and not l() ?
elseif ($is_visible_element && $has_title && $has_id) {
$error_links[] = $this->l($title, Url::fromRoute('<none>', [], ['fragment' => $form_element['#id'], 'external' => TRUE]));
unset($errors[$name]);
}
Comments
Comment #2
Xtal CreditAttribution: Xtal as a volunteer and at Torchbox commentedComment #3
Xtal CreditAttribution: Xtal as a volunteer and at Torchbox commentedComment #5
Xtal CreditAttribution: Xtal as a volunteer and at Torchbox commentedComment #6
dev.patrick CreditAttribution: dev.patrick at TATA Consultancy Services for Pfizer, Inc. commentedI am working on this issue
Comment #7
dev.patrick CreditAttribution: dev.patrick at TATA Consultancy Services for Pfizer, Inc. commentedComment #9
dev.patrick CreditAttribution: dev.patrick at TATA Consultancy Services for Pfizer, Inc. commentedComment #10
dev.patrick CreditAttribution: dev.patrick at TATA Consultancy Services for Pfizer, Inc. commentedComment #12
swati.nuna CreditAttribution: swati.nuna as a volunteer and at InfoBeans Technologies Limited commentedHi,
Please check attached patch file.
Comment #15
andrewmacpherson CreditAttribution: andrewmacpherson as a volunteer and at Annertech commentedComment #16
chanderbhushan CreditAttribution: chanderbhushan as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commentedadded patch
Comment #19
Wim LeersComment #20
LendudeWe should use toRenderable() and not toString(), and we can strip out the LinkGenerator and the Trait because that is now not used anymore.
No idea if this is stripping too much (tests are green locally..).
Comment #21
Wim LeersSimplification FTW! 🤘
Comment #22
catchCommitted a22498f and pushed to 8.7.x. Thanks!