Closed (fixed)
Project:
EU Cookie Compliance (GDPR Compliance)
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Oct 2020 at 11:33 UTC
Updated:
4 Mar 2021 at 16:49 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
emek commentedIn my setup I can get rid of the message by declaring the variable with a format I have before the if-statement. But there might be a better way that could work for everyone?
Comment #3
svenryen commentedI see. What text formats do you have in your config?
Can you live with these notices? Does it work otherwise or is functionality missing or broken because the module failed to find a valid text format at install?
Comment #4
emek commentedIt seems to work, I see the text on the settings page and it shows as Full HTML, but I think that's because I have that value in my configuration.
The formats I have are basic_html and full_html.
Isn't there some way to set some default value for the format, so we can declare the variable and don't get the error message?
Comment #5
svenryen commentedIn my opinion there's a bug in
filter_default_format()that led to the workaround which, although not ideal, seems to work.I'll check again, though probably not until next week. If you could send me your config, that would be great.
Patches are also welcome.
Comment #6
svenryen commentedWe have no way of knowing in advance what formats are available, so we can't really set some default value for the format.
Comment #7
emek commentedI did create a patch for our environment but that will only work for sites with the same format we have.
If anyone else has this problem and finds this issue, then the patch basically looks like this:
Comment #8
heddnThis will get around the issue. See comments on
filter_fallback_format().Comment #10
emek commentedThe patch in #8 removes the error message for me.
Comment #11
svenryen commentedComment #12
baikho commented#8 works for me.
Tested on
1.14.0and core9.1.4.Before:

After:

Comment #13
neslee canil pintoCommitted to the dev branch. Don't know but when I had pushed my network connection went down and I can't see the commit list, but it is committed into the dev branch.
Comment #14
neslee canil pinto