Filter is using DOMXPath in a wrong way. Resulting extra DOM code for each time the filter is used.

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml"><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8"><meta http-equiv="Content-Type" content="text/html; charset=utf-8"><meta http-equiv="Content-Type" content="text/html; charset=utf-8">

Creating a patch...

CommentFileSizeAuthor
#3 3054550-3-Drupal DOMXPath extra HTML is not removed after using.patch8.75 KBmartijn de wit

Comments

Martijn de Wit created an issue. See original summary.

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Related issues: +#3054543: DOMXPath is rendering HTML document again.

Same issues as https://www.drupal.org/project/editor_button_link/issues/3054543

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Status: Active » Needs review
StatusFileSize
new 3054550-3-Drupal DOMXPath extra HTML is not removed after using.patch8.75 KB

Drupal DOMXPath filter was used the wrong way.

Now using the same way EditorFileReference.php is doing it.

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Assigned: martijn de wit » Unassigned

Status: Needs review » Needs work

The last submitted patch, 3: 3054550-3-Drupal DOMXPath extra HTML is not removed after using.patch, failed testing. View results

  • Martijn de Wit committed 2f22b19 on 8.x-1.x 
    Issue #3054550 by Martijn de Wit: Drupal DOMXPath extra HTML is not...
martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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