Hide on admin pages and on overlay

I agree that this is necessary, this should not be considered so much of a "feature request" as a bug report as there is no reason to display this to logged in users or administrators.

Not displaying to logged in users could be a tricky situation -- as a user may already be logged in before this module is added, and thus would never see the pop-up and consent information provided.

There's already code in the module to not render the icon/pop-up on admin pages.

if (strpos(current_path(), 'admin') === FALSE) {

I have seen the problem with things like lightbox.
Not come up with a solution to that yet.

The attached patch makes use of path_is_admin (http://api.drupal.org/api/drupal/includes%21path.inc/function/path_is_ad...), instead of checking if 'admin' is in the current path. This fixes the problem with it displaying on '/node/add' or '/node/edit/*', and other administration pages.

This doesn't solve the problem of it displaying in lightboxs though.

Chris.

The attached patch builds on the previous patch.

It adds a field to the admin page - 'Exclude Paths', in which you can add a list of paths to hide the cookie control on.

My specific use case is the cookie control is displaying in the media module's dialog iframe, (the media module doesn't declare it as an admin page), but I'm sure there are other pages where you'd want to hide it (perhaps when using the print module).

Chris.

I've applied #6 and it works for me. Thank you!

Confirming number 6 works here too. On all 10 drupal sites, across 5 different servers, 3 different OS's and all with slightly different configs. :-p

Thanks for the patch! __cj :-)

Marking as RBTC for now.

Thanks __cj committed.

Budda - any idea when we'll see a release with this patch included?

@MJCO - the 1.6 release which was rolled out on 25th feb included it.