bernskioldmedia / ilmenite-cookie-consent Goto Github PK

The second parameter of get_option() is used if the option's value is empty. This results in the is_analytics_shown() function always returning true. Using an actual value for the option rather than an empty string would resolve.

// If the Analytics checkbox is left unchecked, `$hidden` will always be `false`
$hidden = '' === get_option( 'ilcc_settings_analytics_is_shown', '1' )

Not sure if this plugin is being maintained or not, but I found it and tried to install. Whenever downloading the repository, assets are missing. There are source files, but they're not built.

Hi,
We have some problems with the plugin when it comes to cashing the website, in our case we use litespeed cash.

When a visitor comes to the page, the cookie banner should appear and all scripts should be under the Blacklist in Yett until the user has approved cookies and the script then ends up under whitelist.

It now turns out that when the visitor then approves cookies and later on triggers a new cashing, the next visitor then also receives all scripts under whitelist and the person is not faced with the question of whether he wants cookies or not.

We then thought we had found a solution in Litespeed. They have a field where you can write excluded arguments. We put in ilcc_has_preferences". It now turns out that visitors who accept cookies now never get a cached version of the page, which now makes the page slow for users who accept cookies.

Do you have any suggestions on how we could solve this?

Currently the screenshots are showing the old version, causing confusion (see #18). They should be updated with new and proper ones.

It would be nice to get a feel for how well the cookie banner "performs". We're considering adding basic anonymous local tracking to get statistics on:

• How many total hits have been recorded.
• How many select each category/opt in for all/only necessary.

The important visualisation here would be the split percentages over X days. This is not meant as a replacement local statistical tool.

A filter would make auto-purging of stats after X days available.

Like in the screenshots for the plugin. I guess I can hide them with css, but does feel a bit unclean.

As the title says, the option to change the "Configure settings" text does not display in customiser.

Line 418 of class-settings.php

'section' => 'ilmenite_cookie_banner',

should be:

'section' => 'ilmenite_cookie_banner_general',

Some have reported that some scripts are blocked on subsequent visits, even if cookies have been consented to already.

Possibly, YETT is still blacklisting on these subsequent visits. Is this a plugin error, or possibly cache? If so, how can we reasonably avoid in a simple way.

i added the domains fonts.gstatic.com and www.gstatic.com to the marketing list of the tracker tool.

So i expected that this domains will be blocked until the user has allowed it. But the browser ist calling this domains on loading the page.

Both the script file and style file needs to be included as minified versions.

Include better mobile friendly CSS

Not all websites have statistics and/or marketing. We should consider better supporting the case where a user has either of the three and enable toggling in the settings.

In the case that the website has neither stats nor marketing, we should also only show one button as referenced in #18.

See https://wordpress.org/support/topic/minor-warning-in-the-logs/#post-14148590

I’m seeing this in the logs while I’m debugging another issue on a site:
ilcc_has_preferences ... class-consent.php on line 11

It’s only an annoyance rather than critical, but could that be wrapped with an isset() to keep the debug logs a bit tidier?

The policy URL must be a setting in a proper place in the admin panel, such as the general settings screen.