Problem/Motivation

wildcard + matching tag: wildcard resolves into matching tag, but matching tag already supports all attributes and wildcard + matching tag: wildcard resolves into matching tag, but matching tag already supports all attributes — vice versa added in #3268983: [regression] FilterHtml throws Unsupported operand types error when * used in tag attribute should be moved to the wildcard + matching tag group.

CommentFileSizeAuthor
#2 3295935-2.patch2.82 KBlauriii
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lauriii created an issue. See original summary.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Active » Reviewed & tested by the community
FileSize
3295935-2.patch2.82 KB
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Status: Reviewed & tested by the community » Needs review

we should get a +1 from one of the other people involved in #3268983: [regression] FilterHtml throws Unsupported operand types error when * used in tag attribute

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Needs review » Reviewed & tested by the community

+1

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented
Version: 10.1.x-dev » 9.4.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 07a160117a to 10.1.x and 1a8b586718 to 10.0.x and 32426274f0 to 9.5.x and 172f53b3ef to 9.4.x. Thanks!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Acro Commerce, Thunder commented

Backported to 9.4.x to keep tests aligned.

  • alexpott committed 07a1601 on 10.1.x 
    Issue #3295935 by lauriii: Follow-up to #3268983: Move test case to...

  • alexpott committed 1a8b586 on 10.0.x 
    Issue #3295935 by lauriii: Follow-up to #3268983: Move test case to...

  • alexpott committed 3242627 on 9.5.x 
    Issue #3295935 by lauriii: Follow-up to #3268983: Move test case to...

  • alexpott committed 172f53b on 9.4.x 
    Issue #3295935 by lauriii: Follow-up to #3268983: Move test case to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.