Problem/Motivation

When using the content support url embed module you will not see a cookie notification if you have not accepted cookies, but a 1.

This is because the processHtmlAsString function returns a boolean and this boolean is then used in the content swap on line 84 replaceNodeContent function in the CsUrlEmbedFilter class.

Steps to reproduce

  1. Add a url embed field to a paragraph.
  2. Enter a link to a youtube video.
  3. Save and reject all cookies.
  4. A 1 is displayed.

Proposed resolution

Open for disscusion.

Remaining tasks

Decide on the approach and implement it.

User interface changes

The applicable message is displayed instead of a 1.

API changes

None

Data model changes

None

CommentFileSizeAuthor
embed-ulr-consent-support-bug.jpg28.17 KBdtfabio

Issue fork consent_support-3385300

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

dtfabio created an issue. See original summary.

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

andreasderijcke’s picture

andreasderijcke
Location Antwerpen / Gent
commented
Assigned: Unassigned » andreasderijcke

andreasderijcke opened merge request !7

  • andreasderijcke committed c45335a1 on 1.0.x 
    Issue #3385300 by andreasderijcke, dtfabio: processHtmlAsString function...
andreasderijcke’s picture

andreasderijcke
Location Antwerpen / Gent
commented
Assigned: andreasderijcke » Unassigned
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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